| melwitt | gmann: there is a patch to unblock the python-novaclient gate https://review.opendev.org/c/openstack/python-novaclient/+/901621 that I have +2ed that you might want to look at. it's a very weird thing with failing when NEUTRON_ENFORCE_SCOPE = true | 01:05 |
|---|---|---|
| gmann | melwitt: checking | 02:54 |
| gmann | melwitt: done, yeah strange error. seeing from old and new rule existing token should pass either it is admin or member role token | 03:01 |
| opendevreview | Merged openstack/python-novaclient master: Disable NEUTRON_ENFORCE_SCOPE at function job https://review.opendev.org/c/openstack/python-novaclient/+/901621 | 04:36 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Detect maximum number of SEV guests automatically https://review.opendev.org/c/openstack/nova/+/899381 | 07:58 |
| tkajinam | sean-k-mooney, bauzas ^^^ I've created as bp https://blueprints.launchpad.net/nova/+spec/libvirt-detect-sev-max-guests and link it from the change, as suggested | 08:00 |
| tkajinam | I've not created any bp for a while so lmk if I made something wrong ... | 08:00 |
| gibi | tkajinam: good to see you around :) The BP looks good to me | 08:41 |
| gibi | bauzas: ^^ do we need to wait for the next nova meeting to approve that or we already agreed about the feature and this is just paperwork? | 08:42 |
| tkajinam | gibi, hey. good morning and hope everything is good with you. thanks for checking the bp :-D | 08:44 |
| bauzas | lgtm, I can stamp the paperwork | 08:49 |
| bauzas | tkajinam: gibi: sean-k-mooney: bp approved as specless, good to go for reviews | 08:51 |
| gibi | bauzas: cool | 08:56 |
| gibi | tkajinam: yeah all good here. Hope all good on your side too | 08:57 |
| bauzas | tkajinam: btw. saw your email on tooz | 08:57 |
| bauzas | fwiw, I agree with the concerns, we rather need tooz to continue to verify the older API | 08:58 |
| bauzas | even if we say to our operators that they should first upgrade their environment to the last minor .y version before SLURP upgrading, some of them could not do that for tooz | 08:58 |
| bauzas | given it's a library | 08:59 |
| SvenKieske | anybody has seen this qemu issue anywhere? https://gitlab.com/qemu-project/qemu/-/issues/1069 | 08:59 |
| tkajinam | gibi, all good on my side as well :-) | 09:08 |
| tkajinam | bauzas, thanks for the approval ! | 09:09 |
| tkajinam | bauzas, hmm the limitation caused by hard-coded api version can't be fixed in older tooz unfortunately (because of the fact tooz is branchless) though the proposed change should ease the pain caused by etcd versions. | 09:13 |
| tkajinam | bauzas, and I agree that we need tooz to continue to verify the older API. The proposed API detection mechanism should work for older tooz but we probably have to have a few functional tests to verify it with older etcd | 09:14 |
| bauzas | to be clear, grenade and devstack are just coal canaries | 09:15 |
| bauzas | the problem is the SLURP upgrade, which implies that you don't need to lockstep both the openstack upgrade and the OS upgrade | 09:15 |
| bauzas | well, that's not a problem, that's rather a tenet | 09:15 |
| bauzas | operators aren't forced to upgrade their OSes during openstack upgrade itself | 09:16 |
| tkajinam | I agree | 09:16 |
| bauzas | that's why we have PTIs that continue to support old OS releases during 2 SLURP cadences | 09:16 |
| tkajinam | yes | 09:17 |
| bauzas | the problem is that tooz is branchless | 09:18 |
| bauzas | but new tooz (which would be upgraded during the SLURP release) still needs to talk like the old tooz was doing, against an old etcd API | 09:18 |
| bauzas | we don't need to backport anything | 09:19 |
| bauzas | or ask the operators to minor-uprgade their libs before the SLURP upgrade | 09:19 |
| gibi | bauzas: the SEV slot autodetection impl patch is fairly simple and I'm +2 on it, so I think it is an easy win :) | 09:21 |
| tkajinam | I'm not too sure how many users may maintain tooz independently from OpenStack. usually it's bundled to OpenStack in distros for example. (The tooz package comes from UCA or RDO afair) | 09:21 |
| tkajinam | but | 09:21 |
| tkajinam | bauzas, I think your concern should be cleared by the change I proposed to tooz. the old tooz was able to talk with old etcd without api version override. now with the change new tooz is able to talk with old etcd without api version override so the default behavior is same before/after updating tooz | 09:22 |
| bauzas | ++ | 09:23 |
| tkajinam | gibi, thanks :-D | 09:24 |
| opendevreview | Haidong Pang proposed openstack/nova master: Corrected allocation deletion error during live migration. https://review.opendev.org/c/openstack/nova/+/901721 | 09:33 |
| *** jakob is now known as grandchild | 10:15 | |
| sean-k-mooney | tkajinam: i approved the patch gibi thanks for reviewing | 10:36 |
| sean-k-mooney | tkajinam: specifically https://review.opendev.org/c/openstack/nova/+/899381 thanks for paying donw that technial debt | 10:38 |
| dvo-plv | sean-k-mooney, gibi Do you have a time today to review this proposal ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | 11:52 |
| dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075 | 11:52 |
| sean-k-mooney | i readded my +2 to the spec | 12:18 |
| dvo-plv | thanks | 12:18 |
| sean-k-mooney | im not sure ill have time to review the code today but ill see how my day goes | 12:18 |
| sean-k-mooney | if i dont get to it today feel free to ping me on monday (im on pto tomorrow) | 12:19 |
| sean-k-mooney | i was largely happy with the implemation before so if you have adress gibis previous comments in the nova patch | 12:19 |
| sean-k-mooney | then its likely that nova patch is more or less ready to merge unless we see anything else on this revions | 12:20 |
| dvo-plv | sure, thanks | 12:22 |
| auniyal6 | Hi dansmith, just a reminderto review https://review.opendev.org/c/openstack/nova/+/865336 | 17:01 |
| auniyal6 | you said earlier "will chat with sean-k-mooney about it when he's back before I review/approve" | 17:03 |
| auniyal6 | this only walaby is pending https://review.opendev.org/q/topic:%22bug/1970467%22 | 17:04 |
| opendevreview | Merged openstack/nova master: Detect maximum number of SEV guests automatically https://review.opendev.org/c/openstack/nova/+/899381 | 18:13 |
| sean-k-mooney | artom: Uggla_ if you get around to revisiting https://review.opendev.org/c/openstack/nova/+/877773 i left a -1 and some chagnes that would be required fro that to be backporable. | 18:21 |
| artom | sean-k-mooney, you're insane :) | 18:31 |
| artom | I mean that in the best way possible | 18:31 |
| artom | At least you wrapped it in a @property | 18:32 |
| *** mtreinish_ is now known as mtreinish | 18:47 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!