| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-01-21 | |||
| 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 | NEXT_MIN_LIBVIRT_VERSION = (4, 0, 0) | |
| 14:05:59 | kashyap | And: | |
| 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_QEMU_VERSION = (2, 8, 0) | |
| 14:10:30 | kashyap | +NEXT_MIN_LIBVIRT_VERSION = (3, 0, 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 | |
| 15:33:25 | jaypipes | sean-k-mooney: indeed it is. thine thoughts on the matter be correct. | |
| 15:34:50 | sean-k-mooney | ye is till common in ireland partly because in irish we do have distinct words for singualr and plural you so we still use the older english word "ye" for the plural form commonly | |
| 15:35:36 | jaypipes | sean-k-mooney: that's interesting, considering there are not distinct words for "three" and "tree". ;) | |
| 15:36:43 | jaypipes | I will never forget ildikov and I trying to find that town of Houth and all the Dubliners looking at us like we were morons for saying "hooth" like "tooth". | |
| 15:37:49 | jaypipes | sean-k-mooney: in any case, I blame stephenfin for all of it. | |
| 15:38:23 | sean-k-mooney | jaypipes: this might help if you are interested https://en.wikipedia.org/wiki/Hiberno-English | |
| 15:38:26 | jaypipes | (apparently stephenfin doesn't have a proper Irish accent, much to my surprise) :) | |
| 15:38:58 | ildikov | jaypipes: Ireland is a fun place :) | |
| 15:39:21 | jaypipes | ildikov: it is! especially when you miss your train stop! | |