Earlier  
Posted Nick Remark
#openstack-nova - 2021-02-21
19:24:52 portdirect https://www.irccloud.com/pastebin/pe6F6pqG/
19:25:09 portdirect but this seems rather hacky
#openstack-nova - 2021-02-22
00:09:38 fungi if brinzhang happens back by, gerrit password generation should be working again now
00:09:51 fungi feel free to find us in #opendev if it's still acting up
08:41:19 nightmare_unreal https://docs.openstack.org/api-ref/placement/
08:41:19 nightmare_unreal hello i was looking at osc-placement API , there seems to be no DEL route for the aggregates
08:44:17 gibi nightmare_unreal:
08:44:18 gibi hi
08:44:29 gibi the https://docs.openstack.org/api-ref/placement/?expanded=update-resource-provider-aggregates-detail#update-resource-provider-aggregates is the way to add or remove aggregates
08:44:51 gibi as you have to PUT all the aggregates in the request
08:45:14 gibi so you can GET the aggregates first, modify the list locally and PUT the modified list
08:45:27 nightmare_unreal hello gibi . I see , it can also be used to remove my bad. I'll try to implement it in the client then
08:46:04 nightmare_unreal or wait a min . osc-placement client can already do that ? because PUT is implemented i think
08:46:05 nightmare_unreal https://opendev.org/openstack/osc-placement/src/branch/master/osc_placement/resources/aggregate.py
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

Earlier   Later