| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-21 | |||
| 22:39:51 | jaypipes | gawd, this stuff is hard to talk about in any sort of reasonably clear language, sorry :( | |
| 22:47:22 | melwitt | jaypipes: correct that we go through the scheduler for rebuilds, as of queens. fwiw, it looks like it will say NoValidHost when filters fail https://bugs.launchpad.net/nova/+bug/1744325 but I don't recall issues with doubling up allocations for rebuilds before | |
| 22:47:24 | openstack | Launchpad bug 1744325 in OpenStack Compute (nova) pike "If a rebuild is refused by the scheduler, the instance's imageref is not rolled back" [High,Fix committed] - Assigned to melanie witt (melwitt) | |
| 22:48:05 | melwitt | (that bug shows brief explanation of rebuild going through scheduler and what happens when there's a failure to schedule) | |
| 22:50:32 | mriedem | jaypipes: pretty sure that's been fixed by cfriesen and/or hongbin | |
| 22:50:38 | mriedem | maybe you don't have the backport | |
| 22:50:39 | jaypipes | melwitt: thx Melanie :) as you know, we're on Ocata currently for VMs and I'm seeing that the "solution" internally is "don't change the image when rebuilding"... | |
| 22:51:50 | jaypipes | Have I mentioned I hate the fact that rebuild is basically a crutch because people can't stand to lose their IP addresses because they hard-code them everywhere? :( | |
| 22:52:37 | melwitt | heh, yeah, good ol rebuild | |
| 22:53:03 | melwitt | so, if this is ocata, you wouldn't be going through scheduler for rebuild right? | |
| 22:54:17 | jaypipes | melwitt: yeah, it's been backported. | |
| 22:54:35 | melwitt | ah ok. I'm looking for a bug fix like mriedem mentioned | |
| 22:54:40 | jaypipes | melwitt: well, at least according to https://bugs.launchpad.net/nova/+bug/1744325 (and we have that code merged in our local ocata branch) | |
| 22:54:41 | openstack | Launchpad bug 1744325 in OpenStack Compute (nova) pike "If a rebuild is refused by the scheduler, the instance's imageref is not rolled back" [High,Fix committed] - Assigned to melanie witt (melwitt) | |
| 22:55:46 | melwitt | jaypipes: here we go https://review.openstack.org/561015 | |
| 22:55:49 | melwitt | good memory mriedem | |
| 22:57:00 | jaypipes | melwitt: that's it! thank you! | |
| 22:57:05 | melwitt | \o/ | |
| #openstack-nova - 2019-01-22 | |||
| 00:13:53 | openstackgerrit | Merged openstack/os-vif master: do not always plug ovs ports. https://review.openstack.org/631829 | |
| 04:55:00 | openstackgerrit | Merged openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112 | |
| 06:17:20 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add minimum value in max_concurrent_live_migrations https://review.openstack.org/632240 | |
| 07:53:34 | jangutter | cdent, jaypipes: https://en.wikipedia.org/wiki/Action_Philosophers! | |
| 09:18:14 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025 | |
| 09:19:10 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add test to check os_vif.internal.command.ip.exists https://review.openstack.org/632077 | |
| 09:22:41 | jangutter | dansmith: remember I threatened some o.vo docs? https://review.openstack.org/#/c/632321 | |
| 09:28:25 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Per-instance serial number https://review.openstack.org/619953 | |
| 10:05:56 | kashyap | Anyone else seeing the StringException failures in the 'openstack-tox-lower-constraints' job? | |
| 10:06:00 | kashyap | http://logs.openstack.org/80/630980/6/check/openstack-tox-lower-constraints/aba7c59/testr_results.html.gz | |
| 10:09:25 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS" https://review.openstack.org/630980 | |
| 10:09:26 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: docs: Update references to "QEMU-native TLS" document https://review.openstack.org/631283 | |
| 10:10:28 | kashyap | gibi: stephenfin ^ When you get a moment, appreicate if you could re-ACK them. | |
| 10:14:10 | bauzas | gibi: morning | |
| 10:14:23 | bauzas | gibi: do you remember a bit of how works the affinity groups ? | |
| 10:14:50 | bauzas | gibi: I'm working on an internal bug about this, but I have a specific issue | |
| 10:15:18 | bauzas | gibi: when booting an instance asking for a group, the group UUID is just a hint | |
| 10:20:10 | gibi | kashyap: ack | |
| 10:20:31 | gibi | bauzas: yes, the group UUID is just a hint | |
| 10:21:18 | bauzas | gibi: do you remember when we transfrom the hint into some InstanceGroup object ? | |
| 10:22:45 | gibi | bauzas: https://github.com/openstack/nova/blob/f7252b586b4b9f5a098fedfa27715b8f7e662af6/nova/scheduler/utils.py#L902 | |
| 10:23:19 | bauzas | gibi: yeah, that's what I remembered but I need to check where we call it | |
| 10:23:26 | bauzas | anyway, sorry for pinging you | |
| 10:23:40 | bauzas | gibi: thanks btw. for the help on functional tests for the reshape | |
| 10:23:48 | gibi | bauzas: yeah, I'm just tracing it backwards from there | |
| 10:23:57 | bauzas | I'm really sorry to be out of upstream for the moment | |
| 10:24:55 | openstackgerrit | caoxufeng proposed openstack/nova master: oslo.config>=6.1.0 which is a dependency of nova requires rfc3986>=1.2.0 https://review.openstack.org/632391 | |
| 10:25:19 | gibi | bauzas: no worries | |
| 10:27:11 | gibi | bauzas: still not the right spot https://github.com/openstack/nova/blob/00b14a233886cbcaccf1576904571a59ca8b6535/nova/compute/api.py#L876 | |
| 10:28:07 | gibi | bauzas: this is the place https://github.com/openstack/nova/blob/00b14a233886cbcaccf1576904571a59ca8b6535/nova/compute/api.py#L1180 | |
| 10:30:57 | gibi | kashyap: done | |
| 10:32:06 | kashyap | gibi: Thank you very much. :-) | |
| 13:04:51 | jaypipes | jangutter: ha! that's awesome :) | |
| 13:07:06 | jangutter | jaypipes: I've got the hardcover, it most definitely is. | |
| 13:07:18 | jaypipes | :) | |
| 13:55:22 | sean-k-mooney | jaypipes: hi regarding teh libvirt min version. looking at https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix#Nova_release_min_version we had planned to set it to somthing in the 3.y.z seriese in stien. do you think setting it to 3.2.0 would be resonable? | |
| 13:58:42 | mriedem | ask kashyap he usually handles that now | |
| 13:58:55 | mriedem | and socializes the plan in the ML | |
| 13:59:15 | sean-k-mooney | well i also can avoid the need to increase the minium versions | |
| 13:59:27 | kashyap | Just finishing up late lunch; be here in 5 min | |
| 13:59:53 | jaypipes | sean-k-mooney: not up to me :) I'm cool with that particular patch, just thought it was worth mentioning that there is no specific MIN_VERSION_XXX for indicating which version is needed for the whole "can set mac to 00:00:00..." ability | |
| 14:01:25 | sean-k-mooney | jaypipes: just replied inline also https://review.openstack.org/#/c/624842/4/nova/network/linux_utils.py@a74 | |
| 14:01:36 | jaypipes | sean-k-mooney: ack | |
| 14:01:40 | sean-k-mooney | i can skip removing the code and it will work | |
| 14:01:58 | sean-k-mooney | we can do the removal later when we have a suffient min version | |
| 14:04:15 | kashyap | sean-k-mooney: Hey, so back here. | |
| 14:04:36 | kashyap | sean-k-mooney: To set it to 3.2.0, we need to ensure it is the "LCD"... | |
| 14:05:00 | kashyap | sean-k-mooney: Check out this Nova commit: 489b5f762e2c | |
| 14:05:59 | kashyap | And: | |
| 14:05:59 | kashyap | NEXT_MIN_LIBVIRT_VERSION = (4, 0, 0) | |
| 14:06:00 | kashyap | NEXT_MIN_QEMU_VERSION = (2, 11, 0) | |
| 14:06:29 | kashyap | sean-k-mooney: I'm not sure how did you arrive on libvirt 3.2.0 | |
| 14:06:40 | sean-k-mooney | i was askign for stien | |
| 14:07:26 | kashyap | For Stein: 28d337b -- Pick next minimum libvirt / QEMU versions for "Stein" | |
| 14:07:31 | sean-k-mooney | 3.2.0 is needed to resovle a specific sriov issue but i can avoid the version bump by not cleaning up some code and allowing it to fail | |
| 14:08:11 | kashyap | sean-k-mooney: Okay, can do the version compat check across, and let you know in a bit | |
| 14:08:35 | kashyap | But it's normally not nice to "deviate from the plan", as we announced it to the mailing list as well | |
| 14:09:22 | sean-k-mooney | true but the wiki still states it as a draft and i had no idea it was tracked anywhere else | |
| 14:10:29 | kashyap | sean-k-mooney: So, actually I should bump it right now to: | |
| 14:10:30 | kashyap | +NEXT_MIN_LIBVIRT_VERSION = (3, 0, 0) | |
| 14:10:30 | kashyap | +NEXT_MIN_QEMU_VERSION = (2, 8, 0) | |
| 14:10:33 | sean-k-mooney | kashyap: anyway as i said in my reply to jay here https://review.openstack.org/#/c/624842/4/nova/network/linux_utils.py@a74 its not needed | |
| 14:11:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Share snapshot image membership with instance owner https://review.openstack.org/630769 | |
| 14:11:34 | kashyap | Thanks for the reminder! I lost this TODO item from my Org-Mode file, damn | |
| 14:12:24 | tosky | hi all, did anyone notice an issue when attaching volumes on a devstack/python3/bionic environment? (yes, it may be a cinder issue, but the relevant code is in lib/nova) | |
| 14:12:31 | tosky | I proposed a fix in https://review.openstack.org/#/c/632452/ | |
| 14:14:24 | sean-k-mooney | tosky: we have been haveing some issue yes but not sure if its related to that | |
| 14:21:59 | mriedem | stephenfin: can you push through this simple change https://review.openstack.org/#/c/628205/ | |
| 14:22:08 | mriedem | that will get the approved bp change on top merged | |
| 14:30:44 | stephenfin | mriedem: I can | |
| 14:44:02 | openstackgerrit | Elod Illes proposed openstack/nova stable/pike: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/632480 | |
| 15:19:27 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:21:34 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:23:41 | openstackgerrit | sean mooney proposed openstack/os-vif master: make kuryr-kubernetes-tempest-daemon-octavia non voting https://review.openstack.org/632484 | |
| 15:26:18 | sean-k-mooney | stephenfin: jaypipes ^ can ye take a look at that. 2 in every 3 ci runs seam to be failing for the k8s job and that is not really a new occurance | |
| 15:27:12 | sean-k-mooney | i have treatened to do this many times in the past and its finally at the point where it has annoyed me enought to right the patch. | |
| 15:29:52 | jaypipes | sean-k-mooney: is "ye" a reference to the fact that I am old(e) English? | |
| 15:30:43 | sean-k-mooney | no its the pluar of you | |
| 15:31:20 | jangutter | sean-k-mooney: y'all is singular, all y'all is plural | |
| 15:31:50 | jaypipes | sean-k-mooney: I'm not entirely sure what pluar is, but I have left a comment on said patch. :P | |
| 15:32:10 | jaypipes | sean-k-mooney: thou are most welcome. | |
| 15:32:26 | sean-k-mooney | y'all presuablly is a contraction of you all then | |