| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-11-29 | |||
| 20:19:34 | melwitt | I think I can do it given the examples you linked. but yeah sounds good, if I do it wrong feel free to redo it or whatever tomorrow | |
| 20:21:12 | sean-k-mooney | we in theory have nested virt capablity for 2 vexhost clouds and ovh so i dont really feel as bad about using in in nova next as i did whne it was only one provider | |
| 20:22:53 | melwitt | ah ok | |
| #openstack-nova - 2023-11-30 | |||
| 03:01: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 | |
| 06:40:30 | melwitt | sean-k-mooney: there was an attempt ... https://review.opendev.org/c/openstack/nova/+/902217 but it's super not working 😆 | |
| 06:40:30 | melwitt | sean-k-mooney: there was an attempt ... https://review.opendev.org/c/openstack/nova/+/902217 but it's super not working 😆 | |
| 07:54:29 | opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 09:26:14 | dvo-plv | gibi, dansmith Hello, gentlemens? Maybe you will have a time to review spec and code | |
| 09:26:18 | dvo-plv | https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 09:26:22 | dvo-plv | https://review.opendev.org/c/openstack/nova/+/876075 | |
| 09:34:12 | gibi | bauzas: https://review.opendev.org/c/openstack/nova/+/901656 merged so I bobcat point release is good to go for me :) | |
| 09:34:26 | bauzas | cool | |
| 09:34:37 | bauzas | elodilles: can you please update the bobcat release patch then , | |
| 09:36:43 | 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:56 | 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:37:24 | gibi | elodilles: fixed your comment in https://review.opendev.org/c/openstack/nova/+/901567 | |
| 09:50:44 | dvo-plv | gibi, thanks. Should I do something else to get +1 workflow for patch ? | |
| 09:50:47 | frickler | melwitt: yes, you need to add more things to your nodeset definition, let me update | |
| 09:51:59 | gibi | dvo-plv: I'm waiting for the spec to land | |
| 09:52:03 | 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:30 | dvo-plv | I see | |
| 10:17:44 | elodilles | bauzas: done: https://review.opendev.org/c/openstack/releases/+/899602 | |
| 10:17:55 | elodilles | gibi: ACK, will review it again then :) | |
| 10:19:29 | 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:20:22 | 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:22:38 | bauzas | elodilles: -1 for https://review.opendev.org/c/openstack/releases/+/899602 | |
| 10:22:42 | bauzas | looks you squashed two things | |
| 10:34:42 | opendevreview | alisafari proposed openstack/nova master: Fix traits to cpu flags mapping https://review.opendev.org/c/openstack/nova/+/902183 | |
| 11:08:01 | 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:35 | sean-k-mooney[m] | from a nova perspective its testing two very slightly different code paths | |
| 11:09:28 | 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:12:46 | 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:35:20 | frickler | all jobs should have swap already, or are you talking about swap for the cirros instance? | |
| 11:40:11 | 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:20 | sean-k-mooney[m] | espically if its a very minor failure | |
| 11:40:42 | sean-k-mooney[m] | but we can add a small amount of swap to the flavor say 64 mb | |
| 11:41:11 | sean-k-mooney[m] | cirroz has zswap too so a small amount will go a long way | |
| 11:46:00 | 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: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 | https://review.opendev.org/c/openstack/glance/+/882133 | |
| 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 | |
| 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 | |