| opendevreview | melanie witt proposed openstack/nova master: Add [workarounds]unified_limits_permissive_enforcement https://review.opendev.org/c/openstack/nova/+/924025 | 00:58 |
|---|---|---|
| *** bauzas_ is now known as bauzas | 01:17 | |
| opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add flavor scanning to migrate_to_unified_limits https://review.opendev.org/c/openstack/nova/+/924110 | 01:51 |
| opendevreview | Ghanshyam proposed openstack/placement master: Update test to use service role https://review.opendev.org/c/openstack/placement/+/926870 | 03:51 |
| opendevreview | Ghanshyam proposed openstack/placement master: DNM: testing latest oslo.policy https://review.opendev.org/c/openstack/placement/+/926088 | 03:52 |
| *** bauzas_ is now known as bauzas | 04:10 | |
| *** bauzas_ is now known as bauzas | 06:29 | |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: Add config option to require secure SPICE. https://review.opendev.org/c/openstack/nova/+/922544 | 06:36 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | 06:36 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | 06:36 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console object changes https://review.opendev.org/c/openstack/nova/+/926876 | 06:36 |
| opendevreview | Michael Still proposed openstack/nova master: libvirt: direct SPICE console database changes https://review.opendev.org/c/openstack/nova/+/926877 | 06:36 |
| opendevreview | Viktor Křivák proposed openstack/nova master: Ensure claims to placement are freed after evacuate complete Improve resource_tracker delete_allocation_for_evacuated_instance to handle non-cached compute hosts Fix test to accept this change https://review.opendev.org/c/openstack/nova/+/926292 | 07:21 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926521 | 07:42 |
| opendevreview | Amit Uniyal proposed openstack/nova master: VMware: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926522 | 07:42 |
| opendevreview | Amit Uniyal proposed openstack/nova master: zvm: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926523 | 07:42 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Ironic: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926524 | 07:42 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/909611 | 08:38 |
| mikal | sean-k-mooney: I've split up that first patch BTW. | 08:58 |
| *** __ministry is now known as Guest1136 | 09:00 | |
| songwenping | hi team, i live migrate vgpu vm with lateast code by devstack and nova-scheduler except with 'No valid host was found, The instance has complex allocations on the source host so move cannot be forced.' | 12:04 |
| erlon | Hey Nova core team, I brought [this patch series](https://review.opendev.org/q/topic:%22sg-shared-filter%22) for discussion a couple months ago, and we got the understanding that we will accept this feature for this release. Can you guys please review and approve the change? | 12:18 |
| *** bauzas_ is now known as bauzas | 12:48 | |
| opendevreview | Merged openstack/nova master: Fix wrong nova-manage command in upgrade check https://review.opendev.org/c/openstack/nova/+/880819 | 15:35 |
| elodilles | hi nova team o/ it seems that python-novaclient's gate is broken: https://review.opendev.org/c/openstack/python-novaclient/+/926334 | 15:55 |
| elodilles | with python-novaclient-functional job | 15:55 |
| elodilles | as some tests reports error, with "Multiple possible networks found, use a Network ID to be more specific. (HTTP 409)" error message | 15:56 |
| elodilles | does anyone know from top off the head what this is related to? | 15:57 |
| melwitt | elodilles: I don't know the cause but there is a bug open that reported it https://bugs.launchpad.net/python-novaclient/+bug/2077168 | 16:44 |
| opendevreview | Ghanshyam proposed openstack/placement master: Update test to use service role https://review.opendev.org/c/openstack/placement/+/926870 | 16:48 |
| sean-k-mooney | elodilles: ya i saw but didnt have a change to look into wy | 16:49 |
| sean-k-mooney | elodilles: i assume its a change in devstack or somethign else that is now resulting in new networks or we have a test interaction issue | 16:50 |
| sean-k-mooney | elodilles: i firsted notcied that on https://review.opendev.org/c/openstack/python-novaclient/+/926532 | 16:52 |
| opendevreview | Ghanshyam proposed openstack/placement master: Bump oslo.policy version to enable new RBAC by default https://review.opendev.org/c/openstack/placement/+/926935 | 16:53 |
| sean-k-mooney | there are 2 netowrks aviable the public and private network. i know there was a bug in neutron at one point where external networks(public) would not be visiable unless also marked as shared or something like that | 16:54 |
| sean-k-mooney | so maybe that was fixed | 16:54 |
| sean-k-mooney | it would be more helpful if the error actully listed the networks | 16:54 |
| sean-k-mooney | neutron have been modifygin there rback policy around networks https://github.com/openstack/neutron/commit/f22f7ae012e75b34051945fcac29f955861896ab | 16:59 |
| elodilles | thanks melwitt and sean-k-mooney , i'll try to look into these next week (though secretly hoping that the issue is familiar to someone and there will be a fix by then o:)) | 17:25 |
| sean-k-mooney | it sound like some of the issue we ocational have with tempest | 17:44 |
| sean-k-mooney | but i dont recally anything sepficic | 17:45 |
| sean-k-mooney | so maybe the neutron folks would know | 17:45 |
| melwitt | I see there is code for handling multiple networks (https://github.com/openstack/python-novaclient/blob/2bd135c13793da48af5fbad5208b00aaba7e39dc/novaclient/tests/functional/v2/test_servers.py#L229) however that means test_boot_server_with_auto_network will be skipped. so even if we added a self.multiple_networks check to the failing test, we presumably have lost coverage of test_boot_server_with_auto_network | 18:19 |
| gmann | sean-k-mooney: dansmith: melwitt: can you please check these two placement review, it is needed for oslo.policy 4.4.0 which enable new defaults by default https://review.opendev.org/c/openstack/placement/+/926870/2 https://review.opendev.org/c/openstack/placement/+/926935/1 | 18:19 |
| melwitt | I have not been able to find the cause of the change in networks available in the test env | 18:19 |
| opendevreview | Dan Smith proposed openstack/nova master: Use format_inspector from oslo https://review.opendev.org/c/openstack/nova/+/925025 | 18:20 |
| dansmith | gmann: "X_ROLES" really? | 18:21 |
| sean-k-mooney | melwitt: the test that is failing is expresssly testing booting without specifyign any network info | 18:23 |
| sean-k-mooney | melwitt: on micorversion 2.1 | 18:23 |
| sean-k-mooney | melwitt: which shoudl use the default network | 18:23 |
| sean-k-mooney | or rahter any network if there only one | 18:23 |
| melwitt | I know | 18:24 |
| sean-k-mooney | im asking about htis on neutron too and they also suggested perhaps https://github.com/openstack/neutron/commit/f22f7ae012e75b34051945fcac29f955861896ab | 18:24 |
| sean-k-mooney | which is what i was wondering about | 18:24 |
| sean-k-mooney | so i might try pinning the neutron version to the commit before that | 18:24 |
| melwitt | I'm just saying there is a different test that exists in the test suite that intends to test network auto behavior. and that test will skip itself if multiple networks are found. and I don't have a passing job run to look at whether the test used to run because the last successful run was months ago | 18:25 |
| gmann | dansmith: its a common header in test class and to add service role I need to do that | 18:25 |
| *** bauzas_ is now known as bauzas | 18:26 | |
| sean-k-mooney | melwitt: ack im wonderign if this was an unintetional change in behvior on the nturon side however | 18:26 |
| sean-k-mooney | they do not use microversion | 18:26 |
| sean-k-mooney | so its hard to say | 18:26 |
| dansmith | gmann: but... where does it come from, or where is it honored? that's not a real-world header? | 18:26 |
| sean-k-mooney | it comes for the keystone auth midelware i think? | 18:27 |
| gmann | dansmith: yeah, it is just for test in NoAuthMiddleware used by tests | 18:27 |
| gmann | https://github.com/openstack/placement/blob/c8255d70c89f60cf85676ac5cfbb313ea44645c0/placement/auth.py#L35C7-L35C23 | 18:27 |
| melwitt | sean-k-mooney: yeah. I agree a change happened elsewhere. I was just trying to point out that even if we fix it by "if self.multiple_networks" we might have lost coverage on the other test. do we care? maybe not because it's novaclient. but I just wanted to point it out | 18:27 |
| dansmith | gmann: hrm, okay | 18:28 |
| sean-k-mooney | melwitt: well the cli has been deprecated for removal for 2 years now | 18:28 |
| sean-k-mooney | melwitt: and as of a few weeks ago nova client is nolonger used by the osc | 18:28 |
| melwitt | I knowwwww :) just mentioning the thing that I noticed | 18:28 |
| sean-k-mooney | melwitt: so i wonder if its time to remove the cli or stop testing it | 18:29 |
| melwitt | yeah let's bin it | 18:29 |
| sean-k-mooney | taht would also fix the fact it does not run on 3.11+ | 18:29 |
| melwitt | heheh | 18:30 |
| sean-k-mooney | i.e. https://review.opendev.org/c/openstack/python-novaclient/+/926532 | 18:30 |
| sean-k-mooney | so what would you prefer to do | 18:30 |
| sean-k-mooney | just remove the job or actully drop the shell module and the correspondign tests | 18:30 |
| gmann | dansmith: thanks, and this one also which reflect the oslo.policy new version in requirements.txt https://review.opendev.org/c/openstack/placement/+/926935 | 18:31 |
| dansmith | gmann: yeah I'm looking through tests right now | 18:31 |
| dansmith | just checking that failure | 18:31 |
| sean-k-mooney | im personally hoping we can entirly drop nova client in 2025.2 if other project update to the sdk by then | 18:31 |
| gmann | dansmith: ohk | 18:31 |
| dansmith | gmann: so don't we need to remove some default override in placement? | 18:32 |
| dansmith | or was placement not defaulting to new RBAC like others? | 18:32 |
| gmann | dansmith: yeah, it was not defaulted to new RBAC and doing it in oslo.policy does it for placement with 4.4.0 | 18:33 |
| gmann | with oslo.policy 4.4.0 | 18:33 |
| dansmith | gmann: okay, but presumably this is almost a non-issue for placement since it's admin only? | 18:33 |
| gmann | dansmith: yeah, except reshaper operation which is service' | 18:33 |
| gmann | *service role only | 18:34 |
| melwitt | sean-k-mooney: I dunno I was just being silly without thinking about it. I think it would be easy to make the job pass for now. not sure what to do longer term. probably need to get a sanity check at a nova meeting or something, not sure what direction most would want | 18:34 |
| dansmith | gmann: okay | 18:34 |
| sean-k-mooney | melwitt: ya i considered proposing a check on the number of networks and skiping the test if there was more then 1 for example | 18:35 |
| sean-k-mooney | but im not sure we shoudl be investing too much effort in fixing these anymore | 18:36 |
| melwitt | yeah, I hear you. I agree this is not sustainable | 18:36 |
| sean-k-mooney | im going to finsih for the week soon but ill try an propsoe a DNM to try pinnign to the older neutron and see if that passes or not | 18:40 |
| sean-k-mooney | and in paralle ill propsoe a quick patch to make the functional job nonvoting to unblock it | 18:41 |
| sean-k-mooney | then ill check back on those on monday | 18:41 |
| melwitt | ok cool | 18:43 |
| opendevreview | Dan Smith proposed openstack/nova master: DNM: Test nova against LUKS inspector https://review.opendev.org/c/openstack/nova/+/925026 | 18:49 |
| dansmith | melwitt: ^ | 18:50 |
| dansmith | expecting nova and glance to reject that luks image currently | 18:50 |
| melwitt | 👀 | 19:00 |
| sean-k-mooney | melwitt: i actully think i tmight be related to https://review.opendev.org/c/openstack/neutron/+/920174 and this https://launchpad.net/bugs/2066369 | 19:15 |
| opendevreview | sean mooney proposed openstack/python-novaclient master: [DNM] testing job failure https://review.opendev.org/c/openstack/python-novaclient/+/926950 | 19:18 |
| opendevreview | sean mooney proposed openstack/python-novaclient master: make python-novaclient-functional non voting https://review.opendev.org/c/openstack/python-novaclient/+/926951 | 19:22 |
| sean-k-mooney | https://review.opendev.org/c/openstack/python-novaclient/+/926951 shoudl make it non voting | 19:22 |
| melwitt | sean-k-mooney: huh. not sure how that could result in the multiple networks but the time frame lines up | 19:36 |
| *** melwitt is now known as jgwentworth | 19:38 | |
| *** jgwentworth is now known as melwitt | 19:38 | |
| sean-k-mooney | melwitt: because i think it got a 404 for any networks before | 19:38 |
| sean-k-mooney | i.e. before it worked because it did not see any networks | 19:39 |
| melwitt | ?! | 19:39 |
| melwitt | I would not expected create with auto to work if no networks found? | 19:39 |
| melwitt | weird | 19:39 |
| sean-k-mooney | so now its seeing two netwroks and failing | 19:40 |
| sean-k-mooney | but i think before it might now have been seeing any networks and bootign without a network | 19:40 |
| sean-k-mooney | thats just a guess but that was the only neutron patch that seamed to line up | 19:40 |
| melwitt | yeah.. I didn't think that "auto" would include 'none' but I don't know that level of detail about it | 19:41 |
| sean-k-mooney | i have never booted a nova instance wituout passsing a network or explictly passing none | 19:42 |
| melwitt | same | 19:42 |
| sean-k-mooney | so im not a good judge of what shoudl work in that regard | 19:42 |
| melwitt | yeah. I was just thinking superficially, "get me a network" I would not have expected no network to be an option 😂 | 19:43 |
| melwitt | get me a not a network | 19:43 |
| sean-k-mooney | well this test is not testing that functionatliy | 19:44 |
| dansmith | "get me an airgap" | 19:44 |
| sean-k-mooney | also i didnt know "self.run_hooks('modify_body_for_create', body, **kwargs)" was a thing, now can i unlearn that the nova clinet allows you to supply hooks that change its behvior | 19:45 |
| melwitt | lol | 19:46 |
| sean-k-mooney | the 6% ginger beer i have in the fridge might help with that | 19:49 |
| sean-k-mooney | i should go test this theory o/ | 19:49 |
| sean-k-mooney | melwitt: by the way this test is usign the 2.1 microverison | 19:50 |
| sean-k-mooney | so its not using the autoacocate behvior | 19:50 |
| sean-k-mooney | and its falling back to the older behavior | 19:50 |
| melwitt | hm ok | 19:51 |
| sean-k-mooney | """Required parameter when there are multiple networks defined for the tenant. When you do not specify the networks parameter, the server attaches to the only network created for the current tenant. """ | 19:51 |
| sean-k-mooney | so my theory is before they fixed the bug there was no network visiable and now there are 2 | 19:52 |
| melwitt | ahhh | 19:55 |
| sean-k-mooney | i think we are taking this return https://github.com/openstack/nova/blob/3bdb298e685417ed0ecabbba12062d148ad58826/nova/network/neutron.py#L1181-L1189 | 20:05 |
| sean-k-mooney | i wanted ot see if i could find that quickly but i think that how it worked in the end | 20:06 |
| sean-k-mooney | unfortunetly we dont have logs form the last succesfuly run | 20:06 |
| sean-k-mooney | but if my revert passses we shoudl see that debug message for the instnace if im right | 20:06 |
| sean-k-mooney | we have had that skip since quantom support was being added https://github.com/openstack/nova/commit/8f3b3db9b04bb5ff7b9486dc6fc211745b983855 | 20:14 |
| melwitt | I see. I will watch for the result | 20:15 |
| sean-k-mooney | i hit the same issue | 20:16 |
| sean-k-mooney | https://zuul.openstack.org/stream/0effd51a31b34c03b8a9f8beef7a8cf8?logfile=console.log | 20:16 |
| sean-k-mooney | so i guesss thats not it... | 20:16 |
| sean-k-mooney | that of neutron is not a required proejct | 20:17 |
| sean-k-mooney | oh wrong zuul job | 20:19 |
| sean-k-mooney | that the non voting one i was waiting for it to pass/fail before i finsihed | 20:20 |
| sean-k-mooney | that explains why it was using amster | 20:20 |
| sean-k-mooney | it still used master in the otherone however so i dont think its set up to make depend on work | 20:21 |
| sean-k-mooney | its eaiser to see that hwen the job finshes howeer so im going to leave it urn | 20:22 |
| melwitt | and you think that might be bc the job definition doesn't required-projects neutron? | 20:23 |
| melwitt | I hadn't thought that's needed though | 20:24 |
| melwitt | dansmith: looks like compute correctly rejected the luks image, but glance skipped bc of config? | 21:35 |
| opendevreview | melanie witt proposed openstack/python-novaclient master: [DNM] testing job failure https://review.opendev.org/c/openstack/python-novaclient/+/926950 | 22:14 |
| melwitt | sean-k-mooney: I think you were right about required-projects, I did an edit on ^ I hope you don't mind | 22:15 |
| *** bauzas_ is now known as bauzas | 22:59 | |
| melwitt | hm, that didn't help. it looks like the Depends-On worked regardless of required-projects. in the previous run: | 23:22 |
| melwitt | 2024-08-22 19:25:30.213604 | controller | opendev.org/openstack/neutron checked out to: | 23:22 |
| melwitt | 2024-08-22 19:25:30.213727 | controller | b007d1c19e9a6a9ea309dd414448030d9fa6fb72 Merge 'refs/changes/48/926948/1' | 23:22 |
| melwitt | that's the proposed DNM change for neutron | 23:23 |
| dansmith | melwitt: because you can't set disk_format=luks in glance so it knows it needs to skip it else it'll just fail outright | 23:26 |
| melwitt | that's what I was thinking, ok | 23:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!