| 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 | 03:01 |
|---|---|---|
| melwitt | sean-k-mooney: there was an attempt ... https://review.opendev.org/c/openstack/nova/+/902217 but it's super not working 😆 | 06:40 |
| opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | 07:54 |
| dvo-plv | gibi, dansmith Hello, gentlemens? Maybe you will have a time to review spec and code | 09:26 |
| dvo-plv | https://review.opendev.org/c/openstack/nova-specs/+/895924 | 09:26 |
| dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075 | 09:26 |
| *** tobias-urdin is now known as tobias-urdin-pto | 09:34 | |
| gibi | bauzas: https://review.opendev.org/c/openstack/nova/+/901656 merged so I bobcat point release is good to go for me :) | 09:34 |
| bauzas | cool | 09:34 |
| bauzas | elodilles: can you please update the bobcat release patch then , | 09:34 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Do not manage CPU0's state https://review.opendev.org/c/openstack/nova/+/901567 | 09:36 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: Allow enabling cpu_power_management with 0 dedicated CPUs https://review.opendev.org/c/openstack/nova/+/901660 | 09:36 |
| gibi | elodilles: fixed your comment in https://review.opendev.org/c/openstack/nova/+/901567 | 09:37 |
| dvo-plv | gibi, thanks. Should I do something else to get +1 workflow for patch ? | 09:50 |
| frickler | melwitt: yes, you need to add more things to your nodeset definition, let me update | 09:50 |
| gibi | dvo-plv: I'm waiting for the spec to land | 09:51 |
| opendevreview | Merged openstack/nova-specs master: Re-propose using VirtIO PackedRing Configuration support for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895924 | 09:52 |
| dvo-plv | I see | 09:52 |
| elodilles | bauzas: done: https://review.opendev.org/c/openstack/releases/+/899602 | 10:17 |
| elodilles | gibi: ACK, will review it again then :) | 10:17 |
| opendevreview | Dr. Jens Harbott proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | 10:19 |
| frickler | sean-k-mooney: melwitt: ^^ fwiw I would rather not depend on the nested virt nodes, maybe just changing the image type would be good enough already? | 10:20 |
| bauzas | elodilles: -1 for https://review.opendev.org/c/openstack/releases/+/899602 | 10:22 |
| bauzas | looks you squashed two things | 10:22 |
| opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | 10:34 |
| sean-k-mooney[m] | frickler: the concern is we dont want to change the image type everywhere we want to have at least one full disk image | 11:08 |
| sean-k-mooney[m] | from a nova perspective its testing two very slightly different code paths | 11:08 |
| sean-k-mooney[m] | so we would only be enabling nested virt on nova-next which would be the only job to test the full disk image that we are using today | 11:09 |
| sean-k-mooney[m] | frickler: if we can have the ci stable without nested virt, either with tweaks to the image or the flavor (adding swap for exampel) we could do that instead for that job or use a diffent full disk image in that job | 11:12 |
| frickler | all jobs should have swap already, or are you talking about swap for the cirros instance? | 11:35 |
| sean-k-mooney[m] | yep swap for the cirros image. i dont think it reasonable to keep bumping the ram we keep allocating to cirros | 11:40 |
| sean-k-mooney[m] | espically if its a very minor failure | 11:40 |
| sean-k-mooney[m] | but we can add a small amount of swap to the flavor say 64 mb | 11:40 |
| sean-k-mooney[m] | cirroz has zswap too so a small amount will go a long way | 11:41 |
| frickler | hmm. will it activate that while still in initramfs phase? but I guess it can't hurt trying it out. I'll make a devstack patch | 11:46 |
| elodilles | bauzas: oh, uh, sry :S i've updated the patch now | 11:57 |
| 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) | 11:59 |
| opendevreview | Merged openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075 | 12:12 |
| opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | 13:00 |
| 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:07 |
| opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | 13:22 |
| alisafari | sean-k-mooney: Would you possibly recheck this? https://review.opendev.org/c/openstack/nova/+/902183 | 13:25 |
| 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 |
| dvo-plv | https://review.opendev.org/c/openstack/glance/+/882133 | 16: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:36 |
| 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 |
| bauzas | JayF: heh, any news about ironic-shards for Caracal timeframe ? I see john's not connectedc | 16:53 |
| 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:54 |
| 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:55 |
| bauzas | ++:) | 16:56 |
| bauzas | awesome ! | 16:56 |
| 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 | 16:56 |
| *** tkajinam is now known as Guest8666 | 17:02 | |
| 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:04 |
| bauzas | I'll sort out the right section to put those patches, whether those are bugfixes or features | 17:05 |
| 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 |
| JayF | It's in there, that one patch is a bugfix for sure | 17:05 |
| JayF | put it in proposed at the bottom | 17:05 |
| bauzas | ++ | 17:05 |
| bauzas | I'll then move it to approved bugfixes once I just check that the patch is ready for reviews | 17:06 |
| 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:36 |
| 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 |
| 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 |
| 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 |
| opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!