| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-12-04 | |||
| 10:46:37 | jras | I am unsure whether I should open a bug for this. Please advise. | |
| 11:05:37 | alisafari | ratailor_ Hey man, Could you check this out whenever you have the time? https://review.opendev.org/c/openstack/nova/+/902183 | |
| 11:17:05 | ratailor_ | alisafari, done. Thanks! | |
| 11:40:22 | alisafari | ratailor_ Thanks, Appreciate your time and effort | |
| 11:57:52 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 12:38:39 | 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:44:55 | sean-k-mooney | others need to review it | |
| 12:45:09 | sean-k-mooney | it need 2 core reviewers to approve it | |
| 12:45:38 | sean-k-mooney | bauzas: ^ its a pretty trivial review if you want to take a look | |
| 12:46:06 | sean-k-mooney | its a clear bug with a simple fix | |
| 12:52:58 | alisafari | sean-k-mooney Thanks, And also sorry. I didn't know that. | |
| 12:53:06 | sean-k-mooney | no worries | |
| 13:57:44 | dvo-plv | Hello sean-k-mooney gibi | |
| 13:57:56 | dvo-plv | Could you pelase review last patch from packer ring series https://review.opendev.org/c/openstack/glance/+/882133 ? | |
| 13:58:23 | sean-k-mooney | sure i can do it again | |
| 13:58:31 | sean-k-mooney | i was previously fine with it | |
| 13:58:44 | dvo-plv | great, thanks | |
| 14:04:12 | LarsErikP | is there any SRU planned to include this in zed? https://review.opendev.org/c/openstack/nova/+/882914 | |
| 14:04:39 | sean-k-mooney | this is not thr correct place to ask that | |
| 14:04:52 | LarsErikP | ok. where should I ask? =) | |
| 14:05:15 | sean-k-mooney | that a question for cannoical we do not generally discuss that in this irc channel | |
| 14:05:25 | LarsErikP | ok | |
| 14:06:17 | 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:25 | 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:49 | LarsErikP | is that something that kan be adressed here? :P | |
| 14:07:05 | sean-k-mooney | yep we just did a yoga release as awe are moving it to unmaintaiend | |
| 14:07:16 | sean-k-mooney | we normally do a release of all the new branches first | |
| 14:07:32 | sean-k-mooney | i assuem the patch just has not been approved in the release repo yet | |
| 14:08:24 | sean-k-mooney | LarsErikP: https://review.opendev.org/c/openstack/releases/+/899604 | |
| 14:08:51 | LarsErikP | aha | |
| 14:08:57 | LarsErikP | so. it is on its way | |
| 14:09:42 | sean-k-mooney | bauzas: ^ you mentioned you had a concern with the bobcat one in october but not waht it was | |
| 14:09:59 | sean-k-mooney | LarsErikP: we can bring this up in the team meeting tomorrow | |
| 14:10:03 | sean-k-mooney | and see fi we can unblock this | |
| 14:10:07 | LarsErikP | thanks! | |
| 14:10:27 | LarsErikP | We were hoping to upgrade from yoga to zed before christmas, but that.. may not happen now :P | |
| 14:10:28 | bauzas | sean-k-mooney: we need to land the RPC 6.x backport fix in Zed | |
| 14:10:32 | sean-k-mooney | the bobcat release now has the +1 form bauzas 899602 | |
| 14:10:44 | sean-k-mooney | bauzas: ah ok | |
| 14:11:03 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/899375 | |
| 14:11:21 | bauzas | https://review.opendev.org/c/openstack/nova/+/900341 that one | |
| 14:11:45 | sean-k-mooney | greade failed for soem reason | |
| 14:11:53 | sean-k-mooney | im quickly going to see why and or recheck | |
| 14:12:23 | sean-k-mooney | weried it failsed on a vnc console issue | |
| 14:13:02 | bauzas | yeah it's a known issue | |
| 14:13:16 | sean-k-mooney | a know issue that is now fixed? | |
| 14:13:26 | sean-k-mooney | or just a random failure | |
| 14:13:28 | bauzas | https://review.opendev.org/c/openstack/grenade/+/900257 | |
| 14:13:53 | bauzas | this is the vnc fix ^ | |
| 14:14:08 | bauzas | we landed it in master but bobcat and antelope have the problem | |
| 14:14:23 | sean-k-mooney | ok i see so we need to land that before we can land the nova stabel branch change | |
| 14:14:28 | bauzas | in my series, I only change the CI job in the last patch | |
| 14:14:37 | bauzas | so we can merge the fixes | |
| 14:14:43 | sean-k-mooney | ack | |
| 14:14:43 | bauzas | and we can release | |
| 14:15:06 | bauzas | we can even -1 the grenade changes if we want for stable branches | |
| 14:15:12 | bauzas | those are not really needed | |
| 14:15:22 | bauzas | this is just for closing a coverage gap we had | |
| 14:15:29 | bauzas | (and why rebuild was broken) | |
| 14:15:33 | sean-k-mooney | ok cool so you approvced the zed code fix | |
| 14:15:44 | sean-k-mooney | so we just need to wait for that to merge then update the sha | |
| 14:15:47 | bauzas | yup | |
| 14:15:50 | sean-k-mooney | and LarsErikP then we can do the release | |
| 14:16:22 | LarsErikP | great! | |
| 14:16:28 | 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:46 | LarsErikP | and then I have to wait for canonical and the SRU of course.. fun times :P | |
| 14:18:11 | sean-k-mooney | LarsErikP well they can do release of patches without use doign the release | |
| 14:18:24 | sean-k-mooney | so they may do an async/hotfix for you if your one of there customers | |
| 14:18:29 | LarsErikP | they can.. but will they? :P | |
| 14:18:35 | LarsErikP | I'm not paying :P | |
| 14:18:46 | sean-k-mooney | heheh then proably not | |
| 14:18:53 | sean-k-mooney | hopefully this will get resovled this week | |
| 14:18:55 | LarsErikP | *sad trumpet noises* | |
| 14:19:12 | sean-k-mooney | if it does not keep letting us knwo until we get it done | |
| 14:19:39 | LarsErikP | will do =) | |
| 14:20:27 | LarsErikP | thanks for all the help! We found it a bit.. surprising that the bug was reintroduced when we did the upgrade.. :P | |
| 14:21:33 | obrest | Luckily we have a dedicated production-environment! So that we can fail safely in test :P | |
| 14:40:01 | sean-k-mooney | LarsErikP: normally the yoga release sould not have happend until after all the rest | |
| 14:40:28 | sean-k-mooney | its movign from stable/yoga to unmaintained/yoga and that was really ment ot have happend back in septemeber | |
| 14:40:46 | LarsErikP | aha.. | |
| 14:41:10 | sean-k-mooney | well octopber i guess it was ment to have happend after the 2023.2 release was officlaly made | |
| 14:55:29 | 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) | |
| 15:00:10 | sean-k-mooney | we could i guess | |
| 15:00:17 | sean-k-mooney | im not sure its required but we can | |
| 15:01:03 | sean-k-mooney | it does not actully disable the driver or anythign it just adds the warnign on start up | |
| 15:01:29 | 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:04:06 | bauzas | sean-k-mooney: I added a comment so the release folks will tell us whether we need it | |
| 15:04:15 | sean-k-mooney | cool | |
| 15:32:45 | 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:57:28 | opendevreview | Merged openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 17:14:23 | 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:24 | 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:25 | opendevreview | Elod Illes proposed openstack/nova stable/zed: Adding server actions tests to grenade-multinode https://review.opendev.org/c/openstack/nova/+/900342 | |
| 17:18:29 | elodilles | bauzas: sorry, i did not realize that the functional test failure was valid, please review the updated test & that my thinking is correct ^^^ | |
| 23:14:08 | 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:53 | JayF | bauzas: we're trying hard to land as many of these cleanups as possible before sharding this time :D | |
| 23: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? | |
| #openstack-nova - 2023-12-05 | |||
| 03:37:48 | opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | |
| 07:25:01 | opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | |
| 08:47:42 | sean-k-mooney[m] | gibi: melwitt can either of ye take a look at https://review.opendev.org/c/openstack/nova/+/883682 | |