Posted	Nick	Remark
#openstack-nova - 2021-02-19
17:12:45	sean-k-mooney	so proably {"subnets":[]}
17:12:53	sean-k-mooney	bauzas: it does not
17:13:06	sean-k-mooney	bauzas:which is why i said we just need to keep the extension test
17:13:13	sean-k-mooney	*check
17:13:21	bauzas	sean-k-mooney: yup, I just changed one liner
17:13:29	bauzas	ie. the Neutron client call
17:13:34	gibi	bauzas: and this is the case when the subnet has no segment_id set http://paste.openstack.org/show/802842/
17:13:59	sean-k-mooney	oh its null
17:14:10	sean-k-mooney	{"subnets":[{"segment_id":null}]}
17:14:12	bauzas	perfect
17:14:23	gibi	so you get a list of subnets and for each subnet you get a segment_id which is either null or a uuid
17:14:24	bauzas	the client will then return None
17:14:35	bauzas	right, it's not a list
17:14:41	openstackgerrit	Lee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent https://review.opendev.org/c/openstack/nova/+/720769
17:14:50	bauzas	because there is a 1:1 mapping segment/subnet
17:14:57	gibi	yepp
17:14:58	sean-k-mooney	well its a dict os with a list of dicts
17:15:05	bauzas	all good, touching my UTs and I'm done
17:15:06	sean-k-mooney	one per subnet on the network
17:15:29	bauzas	yeah, I also changed the comprehension list
17:15:52	bauzas	but we're all good
17:17:18	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add 'nova.virt.libvirt.utils.get_arch' stub to fixture https://review.opendev.org/c/openstack/nova/+/774904
17:17:18	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary mock https://review.opendev.org/c/openstack/nova/+/774905
17:17:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Poison os.uname https://review.opendev.org/c/openstack/nova/+/775415
17:17:19	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Parse the 'os' element from domainCapabilities https://review.opendev.org/c/openstack/nova/+/673790
17:17:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Stop passing around virt_type, caps https://review.opendev.org/c/openstack/nova/+/775689
17:17:20	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add missing type hints https://review.opendev.org/c/openstack/nova/+/775688
17:17:22	openstackgerrit	Stephen Finucane proposed openstack/nova master: hardware: Start parsing 'os_secure_boot' https://review.opendev.org/c/openstack/nova/+/682628
17:17:23	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Switch to libvirt's firmware auto-selection https://review.opendev.org/c/openstack/nova/+/775691
17:17:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Report secure boot support to scheduler https://review.opendev.org/c/openstack/nova/+/775690
17:17:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Add firmware auto-configuration support https://review.opendev.org/c/openstack/nova/+/776678
17:17:26	openstackgerrit	Stephen Finucane proposed openstack/nova master: compute: Report COMPUTE_SECURITY_UEFI_SECURE_BOOT https://review.opendev.org/c/openstack/nova/+/776679
17:17:27	openstackgerrit	Stephen Finucane proposed openstack/nova master: scheduler: Translate secure boot requests to trait https://review.opendev.org/c/openstack/nova/+/776680
17:17:29	openstackgerrit	Stephen Finucane proposed openstack/nova master: libvirt: Wire up 'os_secure_boot' property https://review.opendev.org/c/openstack/nova/+/776681
17:17:33	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Add functional tests for UEFI, secure boot https://review.opendev.org/c/openstack/nova/+/776682
17:17:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: tests: Remove duplicated 'start_compute' helper https://review.opendev.org/c/openstack/nova/+/776683
17:17:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: doc: Document UEFI secure boot feature https://review.opendev.org/c/openstack/nova/+/776684
17:18:04	stephenfin	well gdi, didn't I time that well
17:18:32	bauzas	sean-k-mooney: gibi: someone fancy reporting a neutron bug against the segments API ?
17:18:59	bauzas	I'm adding a fat comment explaining this is borked and ideally I should mention the bug report
17:19:46	sean-k-mooney	bauzas: ya although there wont be a performace hit so less of an issue
17:20:19	stephenfin	Anyone want to give me a one line about what I've missed RE: some segment issue?
17:20:24	gibi	bauzas, sean-k-mooney: I'm not sure I understand why returning a segment from a network is a bad thing especially as the API is created to list segments per network
17:21:03	sean-k-mooney	stephenfin: /segments?network=<id> does not give you the routed segement for the subnets on the network
17:21:03	gibi	stephenfin: routed net patch had issue with collecting segment_ids from neutron
17:21:25	sean-k-mooney	gibi: the segmentation id is ment to be admin only
17:21:35	gibi	bauzas, sean-k-mooney: so I let you report a bug
17:21:45	sean-k-mooney	gibi: the provider:segmenat_id that is
17:21:51	stephenfin	hmm, so how do we get it? Can we?
17:22:02	stephenfin	I'm guessing we need to do the filtering ourselves?
17:22:03	sean-k-mooney	stephenfin: ya via the subnets endpoint
17:22:08	stephenfin	Ah
17:22:49	openstackgerrit	Lee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent https://review.opendev.org/c/openstack/nova/+/720769
17:23:00	sean-k-mooney	so /subnets?network_id=<id>&fields=segment_id
17:23:51	sean-k-mooney	stephenfin: i think we are all good bauzas is updating test now
17:24:00	bauzas	I'm 99% done
17:24:05	bauzas	just running UTs
17:24:10	bauzas	for regression testing
17:24:16	bauzas	all good, uploading
17:25:18	bauzas	stephenfin: that reminds me, we haven't discussed on the exception handling you'd like
17:25:28	bauzas	could you tl;dr your concerns ?N
17:26:22	stephenfin	I'd like us to go back to raising a generic NovaException-derived exception, and simply capture/transform that into something scheduler-specific inside the pre-filter
17:27:32	bauzas	ack ok
17:28:01	bauzas	so, leave the exceptions be neutron-somehow specific and just catch and reraise in the prefilter, gotcha
17:28:08	stephenfin	yup, please :)
17:28:14	bauzas	I can do it
17:28:34	bauzas	whoaw, that's now 2 mins I'm uploading the series
17:28:51	bauzas	800kps up
17:28:58	bauzas	kbps even
17:29:30	bauzas	with small caps for the 'b'
17:29:31	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Add net & utils methods for routed nets & segments https://review.opendev.org/c/openstack/nova/+/773976
17:29:40	bauzas	wow
17:29:40	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068
17:29:56	bauzas	gibi: you're free to leave, it's late your time
17:30:09	bauzas	here, I'm stuck at home until 6am
17:30:22	bauzas	stephenfin: working on the FUP
17:30:51	gibi	bauzas, sean-k-mooney: I will file a bug on the inventory handling as I think I understand why inventory is not updated when the deferred port is bound, there is some logic error in the code
17:30:55	gibi	here https://opendev.org/openstack/neutron/src/branch/master/neutron/services/segments/plugin.py#L446-L450
17:55:30	gibi	bauzas, sean-k-mooney: the ip allocation bug https://bugs.launchpad.net/neutron/+bug/1916276
17:55:31	openstack	Launchpad bug 1916276 in neutron "IP address inventory handling for routed net does not work for ip_allocation=deferrred " [Undecided,New]
17:56:45	gibi	sean-k-mooney: could you please file a bug for the segment listing bug, I don't feel to have the knowledge to do that
17:58:35	bauzas	gibi: sean-k-mooney: I spotted an issues with my functest as list_subnets mock doesn't filter
17:58:39	bauzas	so I'll need to respin
17:58:56	bauzas	gibi: thanks for the bug report
18:06:01	sean-k-mooney	ill try and do it on monday. i need to get a few things done today before i finish and its getting a little late
18:11:32	gibi	sean-k-mooney: no problem, and thank you
18:13:53	gibi	stephenfin: does this make sense to you? https://review.opendev.org/c/openstack/nova/+/772381/5/nova/virt/libvirt/driver.py#257
18:15:38	stephenfin	gibi: It does. I hadn't paid attention to the detach-specific nature of the handler
18:16:15	gibi	stephenfin: do you mean the device specific nature?
18:16:26	stephenfin	Yes, sorry :)
18:16:34	gibi	np it is late Friday :)
18:16:45	gibi	stephenfin: OK, I can do that generalization
18:16:55	stephenfin	cool :)
18:17:54	gibi	thanks for the suggestion :)
18:18:06	gibi	but now I leave for the weekend
18:18:11	gibi	have a nice one everyone!
18:18:13	gibi	o/
18:18:15	stephenfin	o/
18:23:37	bauzas	I'm also done now \o
18:23:44	bauzas	just uploading the series

Earlier   Later