| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-22 | |||
| 11:00:45 | openstackgerrit | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add ephemeral encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 11:00:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | |
| 11:00:46 | openstackgerrit | Lee Yarwood proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 11:00:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: func: Allow compute_driver to be set and used by _IntegratedTestBase https://review.opendev.org/c/openstack/nova/+/764484 | |
| 11:00:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 11:00:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: scheduler: Add an ephemeral encryption pre-filter https://review.opendev.org/c/openstack/nova/+/760456 | |
| 11:00:48 | openstackgerrit | Lee Yarwood proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 13:39:59 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [OVN] Adapt the live-migration job scripts to work with OVN https://review.opendev.org/c/openstack/nova/+/776419 | |
| 13:40:00 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set grenade job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776934 | |
| 14:16:52 | openstackgerrit | Ghanshyam proposed openstack/nova stable/stein: DNM: testing stein cap https://review.opendev.org/c/openstack/nova/+/776816 | |
| 14:35:16 | bauzas | oh man, I know I only have DSL connection but I'm getting mad at Gerrit slowness when writing comments | |
| 14:36:09 | bauzas | (and I'm pretty sure slowness isn't coming from my side) | |
| 14:40:56 | bauzas | gibi: I +Wd https://review.opendev.org/c/openstack/nova/+/756530/ with a concern about notifying the ops that they have to run the audit command if we failed | |
| 14:41:36 | bauzas | we already leak some allocations, that's not a reason for adding more :p | |
| 14:41:54 | bauzas | or, in other words, if we can't make sure we don't leak, ask the plumber | |
| 14:51:15 | gibi | bauzas: thanks. I will look at your comments on it later | |
| 15:07:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: libvirt: Define and emit DeviceRemovedEvent and DeviceRemovalFailedEvent https://review.opendev.org/c/openstack/nova/+/749929 | |
| 15:09:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: libvirt: add AsyncDeviceEventsHandler https://review.opendev.org/c/openstack/nova/+/772381 | |
| 15:10:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: libvirt: allow querying devices from the persistent domain https://review.opendev.org/c/openstack/nova/+/772383 | |
| 15:12:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: libvirt: parse alias out from device config https://review.opendev.org/c/openstack/nova/+/772384 | |
| 15:13:09 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944 | |
| 15:14:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Replace blind retry with libvirt event waiting in detach https://review.opendev.org/c/openstack/nova/+/770246 | |
| 15:14:41 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/c/openstack/nova/+/776945 | |
| 15:17:37 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944 | |
| 15:17:38 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/c/openstack/nova/+/776945 | |
| 15:21:48 | gibi | bauzas, sean-k-mooney: fyi the tempest tests https://review.opendev.org/c/openstack/tempest/+/665155 for route net shows that that the VM boot case works \o/ but it also shows by neutron bug https://bugs.launchpad.net/neutron/+bug/1916276 that the IP inventory is not updated | |
| 15:21:50 | openstack | Launchpad bug 1916276 in neutron "IP address inventory handling for routed net does not work for ip_allocation=deferrred " [Medium,Confirmed] | |
| 15:39:27 | sean-k-mooney | gibi: lol so one step forward and one step back | |
| 15:39:47 | sean-k-mooney | gibi: this is at least a little better then previously right | |
| 15:39:54 | sean-k-mooney | because move operation wont break | |
| 15:40:09 | sean-k-mooney | but we can still have a late failure on the compute node if the ip address cant be allocated | |
| 15:40:21 | gibi | sean-k-mooney: yes | |
| 15:40:27 | sean-k-mooney | so i guess for now we can document that as a known issue untille the neutron bug is resolved | |
| 15:40:48 | sean-k-mooney | which ultimatly shoudl be done via a port resouce request so we can scedule on it | |
| 15:43:55 | gibi | sean-k-mooney: the neutron bug either resolved in the segments plugin by really updating the IP inventory for deferred ports (not just for imediate ports) or it can be resolved by adding IP resource to the the resource request | |
| 15:44:55 | sean-k-mooney | yep both would owrk but updateing the inventory wont help with schduling | |
| 15:45:20 | sean-k-mooney | since nova is not asking plamcnent for an ip allocatoin haveign reserved=total wont prevent it from returning a host in that aggreate | |
| 15:45:40 | gibi | true | |
| 15:45:49 | sean-k-mooney | so really the apprcoh that neutorn is currenly taking wont really do anything useful | |
| 15:46:10 | sean-k-mooney | its not in the allcoation so it might as well not exist | |
| 15:46:25 | bauzas | yup, what sean-k-mooney said | |
| 15:46:30 | bauzas | it's not related to my series | |
| 15:46:44 | bauzas | the scheduler doesn't verify this resource class | |
| 15:47:21 | gibi | sean-k-mooney: I don't know if the segments plugin do anyithing special when the reserved == total, in theory it can remove computes from the aggregate to prevent scheduling | |
| 15:47:47 | gibi | but I agree, the ultimate solution is to use resource_request | |
| 15:48:04 | sean-k-mooney | it could but i dont think it does | |
| 15:50:05 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: DO NOT REVIEW: Test OVN devstack module https://review.opendev.org/c/openstack/nova/+/776945 | |
| 15:50:05 | openstackgerrit | Lucas Alvares Gomes proposed openstack/nova master: [OVN] Explicitly set nova-next job to ML2/OVS https://review.opendev.org/c/openstack/nova/+/776944 | |
| 15:51:48 | openstackgerrit | Ghanshyam proposed openstack/nova stable/stein: DNM: testing stein cap https://review.opendev.org/c/openstack/nova/+/776816 | |
| 15:52:03 | gibi | sean-k-mooney: it might does: https://github.com/openstack/neutron/blob/787444ac9ecbc6db123294c33549b97cafe72fd5/neutron/services/segments/plugin.py#L323-L325 but I did not verified it | |
| 15:53:40 | bauzas | stephenfin: you had some changes that I should be reviewing, right? | |
| 15:53:49 | bauzas | gibi: heh | |
| 15:54:15 | stephenfin | Yes please. Either the secure boot series or the API DB compaction series | |
| 15:54:30 | stephenfin | I'll get to the routed networks patches before EOD, promise :) | |
| 15:54:42 | bauzas | gibi: fwiw, we should just support routed networks in Nova now, and in case people see problems with Neutron, they would test it at least | |
| 15:54:54 | bauzas | stephenfin: no worries | |
| 15:55:13 | gibi | bauzas, stephenfin: I'm on the routed net series as we speek | |
| 15:55:25 | bauzas | (also doing some downstream stuff but meh ;) ) | |
| 15:55:40 | bauzas | stephenfin: patches ? | |
| 15:55:44 | gibi | bauzas: agree about releasing the nova support for routed net and deal with the fallout later | |
| 15:56:15 | bauzas | I guess at least https://review.opendev.org/c/openstack/nova/+/774904/6 and the above | |
| 15:58:06 | bauzas | and https://review.opendev.org/c/openstack/nova/+/771419/ then | |
| 16:03:02 | stephenfin | bauzas: https://review.opendev.org/c/openstack/nova/+/776679 above and below | |
| 16:27:33 | gibi | bauzas: I'm +2 on both route net patches now. I have two small nits in https://review.opendev.org/c/openstack/nova/+/773976 for a follow up | |
| 16:27:52 | bauzas | on a meeting but thanks, will look | |
| 16:31:11 | openstackgerrit | Merged openstack/nova master: Support interface attach with QoS ports https://review.opendev.org/c/openstack/nova/+/756530 | |
| 16:31:22 | gibi | \o/ ^^ | |
| 16:33:33 | gibi | who has good grasp on lower constarints bumps? We have this trivial hyperv feature but the lower-constraint change become pretty hairy https://review.opendev.org/c/openstack/nova/+/763550 | |
| 16:38:44 | bauzas | gibi: I worked on it a bit, what's your concern ?N | |
| 16:39:24 | gibi | bauzas: the amount of lower-constraint change that is proposed there to satisfy the pip solver | |
| 16:39:30 | bauzas | hah | |
| 16:40:15 | bauzas | gibi: wow | |
| 16:40:23 | sean-k-mooney | gibi: that a lot of change to lc | |
| 16:40:31 | gibi | yes, hence I raising my flag | |
| 16:40:53 | sean-k-mooney | bumping os-brick and os-win | |
| 16:41:03 | sean-k-mooney | then that transitivly is bumping the rest | |
| 16:41:25 | sean-k-mooney | oh they are coming form os-brick | |
| 16:41:51 | gibi | I'm not even sure all these bumps are needed or it was just _a_ solution but not a solution with _minimal_ amount of bumps | |
| 16:42:27 | sean-k-mooney | right | |
| 16:42:45 | sean-k-mooney | the commit message say syncing os-brick and nova miniums | |
| 16:43:15 | bauzas | could we ask the report to split the change and bump l-c for each release ? | |
| 16:43:30 | lyarwood | so I looked at that review and then ran away | |
| 16:43:33 | sean-k-mooney | we cant bump lc in stable | |
| 16:43:50 | lyarwood | but something to keep in mind is that the release of os-brick that is pulling in has a regression | |
| 16:43:55 | bauzas | like, if 80% of the lib bumps come from the same os-brick upgrade, then it should be a separate patch to ease reviews (and potential revert) | |
| 16:43:59 | lyarwood | not that it changes the bump but still | |
| 16:44:30 | bauzas | given the amount of bumps, I'd certainly like to have a conservative approach | |
| 16:44:32 | sean-k-mooney | well nova really should have it direct deps not all the transitive ones | |
| 16:44:44 | bauzas | right | |
| 16:45:04 | sean-k-mooney | but also im not sure that os-brick is nessally using its real min deps | |
| 16:45:05 | bauzas | but i'm not an expert of transitivity for l-c | |
| 16:47:17 | gibi | yeah, me neither | |
| 16:47:23 | sean-k-mooney | os-brick is an optional dep of nova too its only need if you have cinder and os-win is only needed if you have hyperv | |
| 16:47:35 | sean-k-mooney | its seams incorrect to sync the os-brick deps into nova | |
| 16:48:57 | sean-k-mooney | im going to just bump the two libs os-brick and os-win and run the lc tox env locally | |
| 16:49:33 | gibi | sean-k-mooney: I expect that you will recreate the error https://review.opendev.org/c/openstack/nova/+/763550/12#message-2f6b7331b58c77ee91a0a69d0ee187ded003d623 | |
| 16:50:30 | gibi | from that it seems that the minimum oslo.context needed by nova is different from the minimum oslo.context needed by os-brick. and the pep solver detects this | |
| 16:51:05 | sean-k-mooney | ya this is because of the new resolver | |
| 16:51:31 | sean-k-mooney | the old resolver used the first requirement for a dep | |
| 16:51:41 | sean-k-mooney | the new one looks at all of them | |