| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-15 | |||
| 18:33:50 | eandersson | It looks like the value is never updated | |
| 18:33:56 | melwitt | eandersson: oh, okay. sorry I didn't understand that part :) | |
| 18:36:25 | eandersson | No worries - I understand very little of this in general hah | |
| 18:38:15 | eandersson | and thaks for the help melwitt | |
| 18:40:56 | melwitt | np. I'm concerned if that test is busted | |
| 18:41:33 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Deactivate powervm compute until ready https://review.openstack.org/471773 | |
| 18:47:32 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: [WIP] Implement virt driver set_host_enabled https://review.openstack.org/472794 | |
| 18:49:45 | melwitt | eandersson: from what I see in the debug logging, you're right that memory_mb_used stays stuck at 512. it never increases after each instance is booted | |
| 18:51:31 | eandersson | Maybe that calculation has moved since I created that patch. | |
| 18:52:32 | eandersson | Or maybe I messed up somewhere! :D | |
| 18:52:52 | eandersson | Then again not sure how all those unit-tests could fail if that was so | |
| 18:53:12 | eandersson | *could not | |
| 18:53:19 | melwitt | hang on, I'm also seeing free_ram_mb not changing. | |
| 19:06:25 | mriedem | sdague: huzzah! | |
| 19:06:58 | mriedem | always have to remember my old thunderbird bug trick - after an update, threading is broken, and you have to select "threading with unread" to fix threading, and then do "threading > all" to get threading on read and unread | |
| 19:07:01 | mriedem | yay for bugs | |
| 19:07:51 | melwitt | my thunderbird trick is not using threading (I didn't know it could thread) | |
| 19:08:07 | mriedem | i need threading | |
| 19:08:27 | melwitt | I might need it but I don't know it yet | |
| 19:09:54 | mriedem | we already had like 10lb of brown sugar in the house but since laura couldn't find it immediately she decided that we needed another 5lbs | |
| 19:10:02 | mriedem | so now i'm swimming in gd brown sugar | |
| 19:10:16 | openstackgerrit | Merged openstack/nova master: Stop caching compute nodes in the request https://review.openstack.org/474382 | |
| 19:11:42 | melwitt | heh | |
| 19:12:12 | melwitt | you have a lot of baked goods in your future | |
| 19:13:03 | mriedem | i do | |
| 19:13:14 | mriedem | i asked for homemade oatmeal raisin cookies for fathers day | |
| 19:13:19 | mriedem | sans chocolate chips | |
| 19:13:29 | mriedem | b/c if i wanted chocolate chip cookies, i'd have asked for those | |
| 19:14:16 | melwitt | haha | |
| 19:14:24 | clarkb | mriedem: as a cookie eater I'm happy that people like you don't eat the chocolate chip cookies so that there are more for me | |
| 19:14:28 | clarkb | :P | |
| 19:14:48 | mriedem | oatmeal raisin is a cookie too clark | |
| 19:14:52 | eandersson | melwitt, I think self.memory_mb_used may be missing here https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L276 | |
| 19:16:22 | melwitt | eandersson: hm, yeah looks like that could be a problem | |
| 19:17:35 | melwitt | it's weird that I also see free_ram_mb not changing with the patches. I don't know how that could be. without the patches, I see both free_ram_mb and memory_mb_used changing | |
| 19:17:56 | melwitt | I'm gonna try again with those set_nodes() on and see what that does | |
| 19:19:54 | eandersson | I added the following self.memory_mb_used += ram_mb and everything looks good | |
| 19:20:18 | eandersson | without any modifications to the test_server_group | |
| 19:20:32 | melwitt | though I will say the test isn't doing what it thinks it's doing. it's supposed to be booting two instances on the same host and then expecting the third one to fail. but it's really booting one server on each host and then failing the third | |
| 19:23:15 | openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984 | |
| 19:24:23 | eandersson | Will need additional review on that component, because I thought I understood how that part worked. ^ | |
| 19:26:08 | eandersson | Actually the above is very confusing in general, because it looks like free_mb can go negative, and the assumption was that it shouldn't be able to go negative | |
| 19:28:01 | melwitt | eandersson: I noticed the old code comment said it can go negative, but it didn't explain why. and I can't remember why it can or was supposed to be able to | |
| 19:28:21 | eandersson | The reason overcommit broke was because they changed it so that it couldn't go negative | |
| 19:29:25 | melwitt | I see | |
| 19:29:48 | eandersson | but maybe in some circumstances it's still possible | |
| 19:30:47 | eandersson | https://github.com/openstack/nova/commit/016b810f675b20e8ce78f4c82dc9c679c0162b7a | |
| 19:46:49 | openstackgerrit | Merged openstack/nova master: Use CONF.host for powervm nodename https://review.openstack.org/473975 | |
| 20:07:45 | rickd | Hi, Anyone aware of a bug within libvirtd v2.5.0 and nested virtulization. Having issues running Ocata on a VMware guest : ( | |
| 20:08:41 | rickd | Booting from hardisk ... GRUB... | |
| 20:09:52 | clarkb | rickd: openstack infra's experience is that it is quite buggy. We disable it by default as a result | |
| 20:09:55 | clarkb | it being nested virt | |
| 20:11:17 | rickd | clarkb I didnt release the infras had issues as well (?) | |
| 20:11:26 | rickd | Thought it would just be the computes... | |
| 20:11:43 | clarkb | rickd: sorry the infra team, eg the group that runs all the CI testing systems | |
| 20:11:49 | rickd | When you say you disabled it. Do you mean within Libvirt | |
| 20:11:58 | clarkb | rickd: we test with nova and qemu not nova and kvm | |
| 20:12:39 | rickd | ie you test nova and qemu for nested? | |
| 20:12:50 | clarkb | also when I dug into it I found the linux kernel does not enable nested virt by default on Intel CPUs because it is considered too buggy. It is enabled by default with amd though | |
| 20:13:16 | clarkb | rickd: yes, beacuse the testing itself runs on virtual machines we have the nested nova compute use qemu not kvm otherwise things fail in weird ways | |
| 20:14:04 | rickd | ok but for prod, its Nova + KVM? | |
| 20:14:25 | clarkb | yes the cloud we run tests on use kvm. I'm just saying we don't do nested virtualization | |
| 20:14:33 | clarkb | (cloud we run runs on baremetal so no nesting) | |
| 20:15:07 | rickd | ah ok, just trying to understand if there is a way around this. | |
| 20:15:31 | rickd | But it looks like libvirt has pulled support for nested anyway | |
| 20:15:31 | clarkb | rickd: according to the linux kernel use amd cpus... (I haven't been able to test reliability on amd though) | |
| 20:16:05 | clarkb | I don't think its a libvirt issue, it is a kvm thing | |
| 20:17:04 | rickd | from what I can see its was buggy before libvirt 2.5.0. Then nested support was fully pulled. | |
| 20:17:11 | rickd | I could be wrong though.. | |
| 20:18:03 | clarkb | where do you see that? my understanding is its purely a kvm option and libvirt will just do whatever with kvm being configured the way it is | |
| 20:19:26 | openstack | Launchpad bug 1689370 in devstack "using ubuntu cloud archive gets a version of libvirtd which doesn't do nested virtualization" [Undecided,Fix released] - Assigned to Amrith Kumar (amrith) | |
| 20:19:26 | rickd | ah yer see your point - https://bugs.launchpad.net/devstack/+bug/1689370 | |
| 20:22:30 | rickd | I wonder if I will be ok with a physical compute and virtual controller nodes. Think I would be fine. | |
| 20:41:41 | eandersson | melwitt, so yea looks like it was an issue with the code | |
| 20:41:51 | eandersson | odd though that the tests sometimes passes even when the scheduler fails completelt | |
| 20:46:42 | mriedem | melwitt: small things within https://review.openstack.org/#/c/446245/21 | |
| 20:56:53 | mriedem | jaypipes: i've got an hour to make a difference in your life | |
| 20:57:01 | mriedem | jaypipes: what placement change do you want me to rubber stamp? | |
| 20:59:21 | openstackgerrit | Merged openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999 | |
| 21:05:28 | jaypipes | mriedem: I don't want you to rubberstamp anything :) | |
| 21:07:42 | mriedem | just tell me what to approve | |
| 21:08:03 | mriedem | i need to redeem this lost afternoon of mailing list reading | |
| 21:08:45 | clarkb | mriedem: are you saying the thing you are looking at for me doesn't have redeeming qualities? :) | |
| 21:11:23 | melwitt | eandersson: yeah, it seems like something is weird there | |
| 21:11:40 | melwitt | mriedem: cool, thanks | |
| 21:12:26 | mriedem | clarkb: heh, your thing just got more complicated | |
| 21:38:32 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/474672 | |
| #openstack-nova - 2017-06-16 | |||
| 00:08:01 | mikal | . | |
| 00:09:06 | mriedem | .~O~. | |
| 00:09:26 | mikal | You need an ascii hat and then you can be the over the fence guy from home improvement | |
| 00:09:49 | mriedem | they locked that guy up for being too nosey | |
| 00:09:54 | mikal | Heh | |
| 00:12:49 | openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/nova master: Re-worked ram filter and weights to support overcommit https://review.openstack.org/390984 | |
| 00:23:21 | mriedem1 | alex_xu: the current pagination_parameters parameter type for limit and marker uses multi_params but i'm assuming that's for backward compat? for a new microversion would we want to use single_param for validation? | |
| 00:23:29 | mriedem1 | so you can't send like marker=A&marker=B | |
| 00:53:29 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix title for cinder v3 API spec https://review.openstack.org/474819 | |
| 01:01:20 | openstackgerrit | Merged openstack/nova-specs master: Fix title for cinder v3 API spec https://review.openstack.org/474819 | |
| 01:22:56 | alex_xu | mriedem: yes, that is a vision. I'm thinking maybe worth a email to get wide agreement on that kind of strict validation? | |
| 01:25:38 | alex_xu | mriedem: I pointed the same thing out at https://review.openstack.org/#/c/459483/13/nova/api/openstack/compute/schemas/server_migrations.py also | |
| 01:34:09 | mriedem | yeah i basically have the same thing in my os-hypervisors 2.49 change now | |