Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-19
17:09:43 bauzas thanks
17:09:47 bauzas I really need to get some env
17:10:27 sean-k-mooney http://paste.openstack.org/show/802840/
17:11:37 gibi bauzas: the above GET results in http://paste.openstack.org/show/802841/ if the subnet has a segment_id set
17:11:38 sean-k-mooney so it looks like if the extnsion is not there its an error
17:11:48 gibi bauzas: will check the no segment_id case in a sec
17:11:49 sean-k-mooney but we have the check for that so it should be fine
17:12:06 sean-k-mooney gibi: im expecting an empty list
17:12:42 bauzas sean-k-mooney: your paste probably doesn't have the extension
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: Remove unnecessary mock https://review.opendev.org/c/openstack/nova/+/774905
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: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:19 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Poison os.uname https://review.opendev.org/c/openstack/nova/+/775415
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: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: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: compute: Report COMPUTE_SECURITY_UEFI_SECURE_BOOT https://review.opendev.org/c/openstack/nova/+/776679
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: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 gibi stephenfin: routed net patch had issue with collecting segment_ids from neutron
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: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 openstackgerrit Sylvain Bauza proposed openstack/nova master: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068
17:29:40 bauzas wow
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 :)

Earlier   Later