Earlier  
Posted Nick Remark
#openstack-nova - 2021-02-22
08:47:39 gibi yes you can do openstack resource provider aggregate set <rp-uuid> --aggregate <agg1> --aggregate <agg2> ...
09:01:03 nightmare_unreal thanks gibi
09:57:32 openstackgerrit Stephen Finucane proposed openstack/nova master: compute: Report COMPUTE_SECURITY_UEFI_SECURE_BOOT https://review.opendev.org/c/openstack/nova/+/776679
09:57:33 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Switch to libvirt's firmware auto-selection https://review.opendev.org/c/openstack/nova/+/775691
09:57:33 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Translate secure boot requests to trait https://review.opendev.org/c/openstack/nova/+/776680
09:57:34 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Report secure boot support to scheduler https://review.opendev.org/c/openstack/nova/+/775690
09:57:34 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Wire up 'os_secure_boot' property https://review.opendev.org/c/openstack/nova/+/776681
09:57:35 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove duplicated 'start_compute' helper https://review.opendev.org/c/openstack/nova/+/776683
09:57:35 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add functional tests for UEFI, secure boot https://review.opendev.org/c/openstack/nova/+/776682
09:57:36 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Document UEFI secure boot feature https://review.opendev.org/c/openstack/nova/+/776684
09:58:43 bauzas gibi: stephenfin: routed networks series has been respinned on yesterday, I'd appreciate some reviews for closing this one so I could move to reviews + the RPC API bump story
09:59:12 bauzas s/yesterday/friday ftw
10:00:51 gibi bauzas: ack, I will re-review the routed net series
10:00:55 gibi today
10:01:11 bauzas gibi: thanks, and I promise I'll look at your own change ;)
10:01:27 gibi :)
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

Earlier   Later