| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-11-30 | |||
| 11:57:26 | elodilles | bauzas: oh, uh, sry :S i've updated the patch now | |
| 11:59:01 | elodilles | bauzas: btw, i hesitated a bit about the version bump, whether the last patch worth a MINOR bump (28.1.0), but I think we are still good with PATCH bump (28.0.1) | |
| 12:12:04 | opendevreview | Merged openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | |
| 13:00:31 | opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 13:07:27 | sean-k-mooney | frickler: looking at the console output it at least loads zswap, wether swapon has been done or not i have no idea. it could be added to the init script i guess if its not already being done but ya you might be right that it will be too early for swap to have any impact | |
| 13:22:18 | opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 13:25:52 | alisafari | sean-k-mooney: Would you possibly recheck this? https://review.opendev.org/c/openstack/nova/+/902183 | |
| 16:33:56 | dvo-plv | sean-k-mooney, gibi If you are aroung and have some time, glance patch, last one from the packed ring story already pass zuul | |
| 16:33:57 | dvo-plv | https://review.opendev.org/c/openstack/glance/+/882133 | |
| 16:36:33 | opendevreview | Sylvain Bauza proposed openstack/nova master: vgpu: Allow device_addresses to not be set https://review.opendev.org/c/openstack/nova/+/902084 | |
| 16:37:52 | opendevreview | Sylvain Bauza proposed openstack/nova master: vgpu: Allow device_addresses to not be set https://review.opendev.org/c/openstack/nova/+/902084 | |
| 16:53:39 | bauzas | JayF: heh, any news about ironic-shards for Caracal timeframe ? I see john's not connectedc | |
| 16:54:50 | JayF | bauzas: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/EUP3AITYONBPAMGZHXWT7TXLF7DYKC4Y/ we basically plan on doing it, and trying to fix some bugs in ironic/nova driver | |
| 16:55:12 | JayF | bauzas: I personally will be doing some of that work, but have been extremely busy; essentially my entire december for upstream is pointed at this :) | |
| 16:56:42 | bauzas | ++:) | |
| 16:56:44 | bauzas | awesome ! | |
| 16:56:55 | JayF | https://review.opendev.org/c/openstack/nova/+/900831 is up to resolve the bug mentioned there already, has no reviews, I was not going to recheck-spam to get it V+1 without any review feedback though; failures are not-ironic-related | |
| 17:04:25 | bauzas | JayF: feel free to add your patches up in https://etherpad.opendev.org/p/nova-caracal-status#L47 I'm confused about the exact paperwork we may need for this release | |
| 17:05:01 | bauzas | I'll sort out the right section to put those patches, whether those are bugfixes or features | |
| 17:05:35 | bauzas | but I want to be sure I'm able to track that effort for this release, and that etherpad is basically my braindump | |
| 17:05:40 | JayF | It's in there, that one patch is a bugfix for sure | |
| 17:05:50 | JayF | put it in proposed at the bottom | |
| 17:05:52 | bauzas | ++ | |
| 17:06:16 | bauzas | I'll then move it to approved bugfixes once I just check that the patch is ready for reviews | |
| 18:36:08 | 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 | |
| 18:38:21 | 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 | |
| 21:18:15 | 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 | |
| 23:25:41 | 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 | |
| 23:47:57 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| #openstack-nova - 2023-12-01 | |||
| 01:48:53 | opendevreview | Merged openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| 14:48:20 | dvo-plv | Hello sean-k-mooney gibi dansmith Maybe you will have a time to review last patch from packed ring series | |
| 14:48:20 | dvo-plv | https://review.opendev.org/c/openstack/glance/+/882133 | |
| 15:13:51 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | |
| 21:37:50 | 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 | |
| #openstack-nova - 2023-12-04 | |||
| 10:44:39 | 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: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 | |