| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-08-22 | |||
| 18:26:26 | sean-k-mooney | they do not use microversion | |
| 18:26:32 | sean-k-mooney | so its hard to say | |
| 18:26:33 | dansmith | gmann: but... where does it come from, or where is it honored? that's not a real-world header? | |
| 18:27:08 | sean-k-mooney | it comes for the keystone auth midelware i think? | |
| 18:27:15 | gmann | dansmith: yeah, it is just for test in NoAuthMiddleware used by tests | |
| 18:27:22 | gmann | https://github.com/openstack/placement/blob/c8255d70c89f60cf85676ac5cfbb313ea44645c0/placement/auth.py#L35C7-L35C23 | |
| 18:27:44 | 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:28:05 | dansmith | gmann: hrm, okay | |
| 18:28:22 | sean-k-mooney | melwitt: well the cli has been deprecated for removal for 2 years now | |
| 18:28:43 | sean-k-mooney | melwitt: and as of a few weeks ago nova client is nolonger used by the osc | |
| 18:28:50 | melwitt | I knowwwww :) just mentioning the thing that I noticed | |
| 18:29:03 | sean-k-mooney | melwitt: so i wonder if its time to remove the cli or stop testing it | |
| 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 | |