| *** ykarel|away is now known as ykarel | 04:48 | |
| jras | Hello - A recent update to Nova (Yoga release) removed the default value for cputune shares from libvirt driver. This results in out of bounds values for flavors with more than 10 vCPU. Libvirt uses cgroups v2 which allows a maximum of 10k for this value. We see values >10k being added now. | 10:44 |
|---|---|---|
| jras | I am unsure whether I should open a bug for this. Please advise. | 10:46 |
| alisafari | ratailor_ Hey man, Could you check this out whenever you have the time? https://review.opendev.org/c/openstack/nova/+/902183 | 11:05 |
| ratailor_ | alisafari, done. Thanks! | 11:17 |
| alisafari | ratailor_ Thanks, Appreciate your time and effort | 11:40 |
| opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | 11:57 |
| alisafari | sean-k-mooney Do you think this is good to be merged or requires further changes? https://review.opendev.org/c/openstack/nova/+/902183 | 12:38 |
| sean-k-mooney | others need to review it | 12:44 |
| sean-k-mooney | it need 2 core reviewers to approve it | 12:45 |
| sean-k-mooney | bauzas: ^ its a pretty trivial review if you want to take a look | 12:45 |
| sean-k-mooney | its a clear bug with a simple fix | 12:46 |
| alisafari | sean-k-mooney Thanks, And also sorry. I didn't know that. | 12:52 |
| sean-k-mooney | no worries | 12:53 |
| dvo-plv | Hello sean-k-mooney gibi | 13:57 |
| dvo-plv | Could you pelase review last patch from packer ring series https://review.opendev.org/c/openstack/glance/+/882133 ? | 13:57 |
| sean-k-mooney | sure i can do it again | 13:58 |
| sean-k-mooney | i was previously fine with it | 13:58 |
| dvo-plv | great, thanks | 13:58 |
| LarsErikP | is there any SRU planned to include this in zed? https://review.opendev.org/c/openstack/nova/+/882914 | 14:04 |
| sean-k-mooney | this is not thr correct place to ask that | 14:04 |
| LarsErikP | ok. where should I ask? =) | 14:04 |
| sean-k-mooney | that a question for cannoical we do not generally discuss that in this irc channel | 14:05 |
| LarsErikP | ok | 14:05 |
| sean-k-mooney | assumign they have a similar import process form upstream as redhat it will probaly get pulled into a future update | 14:06 |
| LarsErikP | Another related question.. It seems like there is no tagged version of nova in zed which includes that fix. But - it is tagged and released for Yoga.. Which means that whne you upgrade from yoga to zed, the bug is reintroduced.. | 14:06 |
| LarsErikP | is that something that kan be adressed here? :P | 14:06 |
| sean-k-mooney | yep we just did a yoga release as awe are moving it to unmaintaiend | 14:07 |
| sean-k-mooney | we normally do a release of all the new branches first | 14:07 |
| sean-k-mooney | i assuem the patch just has not been approved in the release repo yet | 14:07 |
| sean-k-mooney | LarsErikP: https://review.opendev.org/c/openstack/releases/+/899604 | 14:08 |
| LarsErikP | aha | 14:08 |
| LarsErikP | so. it is on its way | 14:08 |
| sean-k-mooney | bauzas: ^ you mentioned you had a concern with the bobcat one in october but not waht it was | 14:09 |
| sean-k-mooney | LarsErikP: we can bring this up in the team meeting tomorrow | 14:09 |
| sean-k-mooney | and see fi we can unblock this | 14:10 |
| LarsErikP | thanks! | 14:10 |
| LarsErikP | We were hoping to upgrade from yoga to zed before christmas, but that.. may not happen now :P | 14:10 |
| bauzas | sean-k-mooney: we need to land the RPC 6.x backport fix in Zed | 14:10 |
| sean-k-mooney | the bobcat release now has the +1 form bauzas 899602 | 14:10 |
| sean-k-mooney | bauzas: ah ok | 14:10 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/899375 | 14:11 |
| bauzas | https://review.opendev.org/c/openstack/nova/+/900341 that one | 14:11 |
| sean-k-mooney | greade failed for soem reason | 14:11 |
| sean-k-mooney | im quickly going to see why and or recheck | 14:11 |
| sean-k-mooney | weried it failsed on a vnc console issue | 14:12 |
| bauzas | yeah it's a known issue | 14:13 |
| sean-k-mooney | a know issue that is now fixed? | 14:13 |
| sean-k-mooney | or just a random failure | 14:13 |
| bauzas | https://review.opendev.org/c/openstack/grenade/+/900257 | 14:13 |
| bauzas | this is the vnc fix ^ | 14:13 |
| bauzas | we landed it in master but bobcat and antelope have the problem | 14:14 |
| sean-k-mooney | ok i see so we need to land that before we can land the nova stabel branch change | 14:14 |
| bauzas | in my series, I only change the CI job in the last patch | 14:14 |
| bauzas | so we can merge the fixes | 14:14 |
| sean-k-mooney | ack | 14:14 |
| bauzas | and we can release | 14:14 |
| bauzas | we can even -1 the grenade changes if we want for stable branches | 14:15 |
| bauzas | those are not really needed | 14:15 |
| bauzas | this is just for closing a coverage gap we had | 14:15 |
| bauzas | (and why rebuild was broken) | 14:15 |
| sean-k-mooney | ok cool so you approvced the zed code fix | 14:15 |
| sean-k-mooney | so we just need to wait for that to merge then update the sha | 14:15 |
| bauzas | yup | 14:15 |
| sean-k-mooney | and LarsErikP then we can do the release | 14:15 |
| LarsErikP | great! | 14:16 |
| bauzas | sean-k-mooney: if you want a fun puzzle, you'll like that https://review.opendev.org/c/openstack/nova/+/902084/comments/4e8056fe_b4b6a789 | 14:16 |
| LarsErikP | and then I have to wait for canonical and the SRU of course.. fun times :P | 14:16 |
| sean-k-mooney | LarsErikP well they can do release of patches without use doign the release | 14:18 |
| sean-k-mooney | so they may do an async/hotfix for you if your one of there customers | 14:18 |
| LarsErikP | they can.. but will they? :P | 14:18 |
| LarsErikP | I'm not paying :P | 14:18 |
| sean-k-mooney | heheh then proably not | 14:18 |
| sean-k-mooney | hopefully this will get resovled this week | 14:18 |
| LarsErikP | *sad trumpet noises* | 14:18 |
| sean-k-mooney | if it does not keep letting us knwo until we get it done | 14:19 |
| LarsErikP | will do =) | 14:19 |
| LarsErikP | thanks for all the help! We found it a bit.. surprising that the bug was reintroduced when we did the upgrade.. :P | 14:20 |
| obrest | Luckily we have a dedicated production-environment! So that we can fail safely in test :P | 14:21 |
| sean-k-mooney | LarsErikP: normally the yoga release sould not have happend until after all the rest | 14:40 |
| sean-k-mooney | its movign from stable/yoga to unmaintained/yoga and that was really ment ot have happend back in septemeber | 14:40 |
| LarsErikP | aha.. | 14:40 |
| sean-k-mooney | well octopber i guess it was ment to have happend after the 2023.2 release was officlaly made | 14:41 |
| bauzas | sean-k-mooney: do you think we should have a minor release (.y) for https://review.opendev.org/c/openstack/releases/+/899603 ? (due to the hyverv experimental backport) | 14:55 |
| sean-k-mooney | we could i guess | 15:00 |
| sean-k-mooney | im not sure its required but we can | 15:00 |
| sean-k-mooney | it does not actully disable the driver or anythign it just adds the warnign on start up | 15:01 |
| sean-k-mooney | if we took the neutron approch of not loadign experimntal drivers without an explict opt in it woudl be another matter | 15:01 |
| bauzas | sean-k-mooney: I added a comment so the release folks will tell us whether we need it | 15:04 |
| sean-k-mooney | cool | 15:04 |
| gibi | sean-k-mooney: could you check the two backports of power management fixes https://review.opendev.org/c/openstack/nova/+/901567 https://review.opendev.org/c/openstack/nova/+/901660/2 (both has +2 from elodilles already) | 15:32 |
| opendevreview | Merged openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | 15:57 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: add a regression test for all compute RPCAPI 6.x pinnings for rebuild https://review.opendev.org/c/openstack/nova/+/900307 | 17:14 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: Fix rebuild compute RPC API exception for rolling-upgrades https://review.opendev.org/c/openstack/nova/+/900341 | 17:14 |
| opendevreview | Elod Illes proposed openstack/nova stable/zed: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900342 | 17:14 |
| elodilles | bauzas: sorry, i did not realize that the functional test failure was valid, please review the updated test & that my thinking is correct ^^^ | 17:18 |
| *** seebaer is now known as seba | 19:26 | |
| JayF | bauzas: for https://review.opendev.org/c/openstack/nova/+/659691 and other patches related to the SDK migration for ironic<>nova driver, is there a place you'd like me to link these, or some way you'd like me to tag them, once Ironic cores are happy w/em? | 23:14 |
| JayF | bauzas: we're trying hard to land as many of these cleanups as possible before sharding this time :D | 23:14 |
| JayF | stephenfin: https://review.opendev.org/c/openstack/nova/+/867799 (and the chain) needs a rebase; I was going to review, get this tested in Ironic CI, and try to help land this -- do you want to rebase this or would you like me to? | 23:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!