| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-22 | |||
| 18:29:12 | melwitt | yeah let's bin it | |
| 18:29:33 | sean-k-mooney | taht would also fix the fact it does not run on 3.11+ | |
| 18:30:08 | melwitt | heheh | |
| 18:30:10 | sean-k-mooney | i.e. https://review.opendev.org/c/openstack/python-novaclient/+/926532 | |
| 18:30:18 | sean-k-mooney | so what would you prefer to do | |
| 18:30:40 | sean-k-mooney | just remove the job or actully drop the shell module and the correspondign tests | |
| 18:31:08 | 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:20 | dansmith | gmann: yeah I'm looking through tests right now | |
| 18:31:25 | dansmith | just checking that failure | |
| 18:31:30 | 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:33 | gmann | dansmith: ohk | |
| 18:32:14 | dansmith | gmann: so don't we need to remove some default override in placement? | |
| 18:32:27 | dansmith | or was placement not defaulting to new RBAC like others? | |
| 18:33:17 | 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:33 | gmann | with oslo.policy 4.4.0 | |
| 18:33:36 | dansmith | gmann: okay, but presumably this is almost a non-issue for placement since it's admin only? | |
| 18:33:57 | gmann | dansmith: yeah, except reshaper operation which is service' | |
| 18:34:04 | gmann | *service role only | |
| 18:34:23 | 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:31 | dansmith | gmann: okay | |
| 18:35:54 | 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:36:10 | sean-k-mooney | but im not sure we shoudl be investing too much effort in fixing these anymore | |
| 18:36:43 | melwitt | yeah, I hear you. I agree this is not sustainable | |
| 18:40:37 | 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:41:00 | sean-k-mooney | and in paralle ill propsoe a quick patch to make the functional job nonvoting to unblock it | |
| 18:41:11 | sean-k-mooney | then ill check back on those on monday | |
| 18:43:04 | melwitt | ok cool | |
| 18:49:50 | opendevreview | Dan Smith proposed openstack/nova master: DNM: Test nova against LUKS inspector https://review.opendev.org/c/openstack/nova/+/925026 | |
| 18:50:05 | dansmith | melwitt: ^ | |
| 18:50:33 | dansmith | expecting nova and glance to reject that luks image currently | |
| 19:00:08 | melwitt | 👀 | |
| 19:15:07 | 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:18:42 | opendevreview | sean mooney proposed openstack/python-novaclient master: [DNM] testing job failure https://review.opendev.org/c/openstack/python-novaclient/+/926950 | |
| 19:22:28 | 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:49 | sean-k-mooney | https://review.opendev.org/c/openstack/python-novaclient/+/926951 shoudl make it non voting | |
| 19:36:23 | melwitt | sean-k-mooney: huh. not sure how that could result in the multiple networks but the time frame lines up | |
| 19:38:51 | sean-k-mooney | melwitt: because i think it got a 404 for any networks before | |
| 19:39:06 | sean-k-mooney | i.e. before it worked because it did not see any networks | |
| 19:39:14 | melwitt | ?! | |
| 19:39:38 | melwitt | I would not expected create with auto to work if no networks found? | |
| 19:39:52 | melwitt | weird | |
| 19:40:05 | sean-k-mooney | so now its seeing two netwroks and failing | |
| 19:40:20 | sean-k-mooney | but i think before it might now have been seeing any networks and bootign without a network | |
| 19:40:42 | sean-k-mooney | thats just a guess but that was the only neutron patch that seamed to line up | |
| 19:41:18 | melwitt | yeah.. I didn't think that "auto" would include 'none' but I don't know that level of detail about it | |
| 19:42:00 | sean-k-mooney | i have never booted a nova instance wituout passsing a network or explictly passing none | |
| 19:42:22 | melwitt | same | |
| 19:42:26 | sean-k-mooney | so im not a good judge of what shoudl work in that regard | |
| 19:43:21 | melwitt | yeah. I was just thinking superficially, "get me a network" I would not have expected no network to be an option 😂 | |
| 19:43:34 | melwitt | get me a not a network | |
| 19:44:51 | sean-k-mooney | well this test is not testing that functionatliy | |
| 19:44:58 | dansmith | "get me an airgap" | |
| 19:45:42 | 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:46:24 | melwitt | lol | |
| 19:49:05 | sean-k-mooney | the 6% ginger beer i have in the fridge might help with that | |
| 19:49:18 | sean-k-mooney | i should go test this theory o/ | |
| 19:50:14 | sean-k-mooney | melwitt: by the way this test is usign the 2.1 microverison | |
| 19:50:27 | sean-k-mooney | so its not using the autoacocate behvior | |
| 19:50:35 | sean-k-mooney | and its falling back to the older behavior | |
| 19:51:05 | melwitt | hm ok | |
| 19:51:39 | 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:52:02 | sean-k-mooney | so my theory is before they fixed the bug there was no network visiable and now there are 2 | |
| 19:55:13 | melwitt | ahhh | |
| 20:05:54 | sean-k-mooney | i think we are taking this return https://github.com/openstack/nova/blob/3bdb298e685417ed0ecabbba12062d148ad58826/nova/network/neutron.py#L1181-L1189 | |
| 20:06:17 | 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:38 | sean-k-mooney | unfortunetly we dont have logs form the last succesfuly run | |
| 20:06:55 | sean-k-mooney | but if my revert passses we shoudl see that debug message for the instnace if im right | |
| 20:14:58 | sean-k-mooney | we have had that skip since quantom support was being added https://github.com/openstack/nova/commit/8f3b3db9b04bb5ff7b9486dc6fc211745b983855 | |
| 20:15:13 | melwitt | I see. I will watch for the result | |
| 20:16:36 | sean-k-mooney | i hit the same issue | |
| 20:16:37 | sean-k-mooney | https://zuul.openstack.org/stream/0effd51a31b34c03b8a9f8beef7a8cf8?logfile=console.log | |
| 20:16:47 | sean-k-mooney | so i guesss thats not it... | |
| 20:17:06 | sean-k-mooney | that of neutron is not a required proejct | |
| 20:19:46 | sean-k-mooney | oh wrong zuul job | |
| 20:20:05 | sean-k-mooney | that the non voting one i was waiting for it to pass/fail before i finsihed | |
| 20:20:14 | sean-k-mooney | that explains why it was using amster | |
| 20:21:44 | sean-k-mooney | it still used master in the otherone however so i dont think its set up to make depend on work | |
| 20:22:39 | sean-k-mooney | its eaiser to see that hwen the job finshes howeer so im going to leave it urn | |
| 20:23:24 | melwitt | and you think that might be bc the job definition doesn't required-projects neutron? | |
| 20:24:10 | melwitt | I hadn't thought that's needed though | |
| 21:35:07 | melwitt | dansmith: looks like compute correctly rejected the luks image, but glance skipped bc of config? | |
| 22:14:09 | opendevreview | melanie witt proposed openstack/python-novaclient master: [DNM] testing job failure https://review.opendev.org/c/openstack/python-novaclient/+/926950 | |
| 22:15:47 | melwitt | sean-k-mooney: I think you were right about required-projects, I did an edit on ^ I hope you don't mind | |
| 23:22:42 | melwitt | hm, that didn't help. it looks like the Depends-On worked regardless of required-projects. in the previous run: | |
| 23:22:52 | melwitt | 2024-08-22 19:25:30.213727 | controller | b007d1c19e9a6a9ea309dd414448030d9fa6fb72 Merge 'refs/changes/48/926948/1' | |
| 23:22:52 | melwitt | 2024-08-22 19:25:30.213604 | controller | opendev.org/openstack/neutron checked out to: | |
| 23:23:12 | melwitt | that's the proposed DNM change for neutron | |
| 23:26:04 | 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:22 | melwitt | that's what I was thinking, ok | |
| #openstack-nova - 2024-08-23 | |||
| 06:58:34 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: Fix retried resize operation failing with FileNotFound https://review.opendev.org/c/openstack/nova/+/926251 | |
| 09:47:06 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 09:47:07 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 14:05:14 | frickler | Uggla: auniyal: bauzas: please check the os-vif release patch https://review.opendev.org/c/openstack/releases/+/926768 | |
| 14:48:58 | opendevreview | Rajesh Tailor proposed openstack/nova master: Rephrase the help message https://review.opendev.org/c/openstack/nova/+/925207 | |
| 18:26:10 | gmann | melwitt: sean-k-mooney: if you are around, can you please check this, need 2nd +2 https://review.opendev.org/c/openstack/placement/+/926870/2 | |
| 22:40:07 | opendevreview | Merged openstack/placement master: Update test to use service role https://review.opendev.org/c/openstack/placement/+/926870 | |
| #openstack-nova - 2024-08-24 | |||
| 02:04:30 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |
| 02:04:31 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:56:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add extra spec for sound device. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 07:56:19 | opendevreview | Michael Still proposed openstack/nova master: libvirt: allow direct SPICE connections to qemu https://review.opendev.org/c/openstack/nova/+/924844 | |