| *** mhen_ is now known as mhen | 01:53 | |
| LarsErikP | " | 07:25 |
|---|---|---|
| sean-k-mooney | LarsErikP: " | 08:11 |
| sean-k-mooney | :) | 08:11 |
| LarsErikP | sean-k-mooney: fat-finger-tuesday? :P | 08:11 |
| sean-k-mooney | cant just leave un closed quots by them selves they coudl get lonely | 08:12 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: Fix fill_metadata usage for the ImagePropertiesWeigher https://review.opendev.org/c/openstack/nova/+/963358 | 08:12 |
| LarsErikP | fair point :P | 08:12 |
| bauzas | sean-k-mooney: thanks for the help on autopep8, I'll look at the alt keys indeed before committing | 08:12 |
| sean-k-mooney | bauzas: i reallised it worse for you | 08:12 |
| sean-k-mooney | so you have a artzy keyboard right | 08:13 |
| sean-k-mooney | so your # us ib 3 | 08:13 |
| sean-k-mooney | *is on | 08:13 |
| bauzas | maybe because I'm using a french kb, yes | 08:13 |
| bauzas | yeah, Alt-Gr + 3 | 08:13 |
| sean-k-mooney | ya so you press altgr 3 to type hash | 08:13 |
| bauzas | correct | 08:13 |
| bauzas | but I had no issues before AFAICR | 08:13 |
| sean-k-mooney | ha so for me and its just the key above shift and for the use i think its shift 3 | 08:14 |
| bauzas | lemme write some code and cat the file | 08:14 |
| bauzas | maybe that's a VSCode/Codium issue | 08:14 |
| sean-k-mooney | no its a local kmap issue | 08:14 |
| bauzas | I was having the same issue with Codium and not only Cursor | 08:14 |
| sean-k-mooney | once i relicated it i coudl do it in emacs as well | 08:15 |
| sean-k-mooney | probably other editor too | 08:15 |
| sean-k-mooney | oh well they may have other key binds that intercept it | 08:15 |
| sean-k-mooney | i can only do it with altgr because left alt is bound ot other things in vscode | 08:15 |
| sean-k-mooney | oh no for me alt space is grabbed by kde | 08:17 |
| sean-k-mooney | its the applicaton laucher | 08:17 |
| bauzas | just created a new file | 08:17 |
| bauzas | https://paste.opendev.org/show/btgI58rkgJvf4i6XkWcO/ | 08:18 |
| bauzas | I wasn't seeing any keys when catting | 08:18 |
| bauzas | but lemme check the chars | 08:18 |
| sean-k-mooney | do cat -vet nova/poc_wip.py | 08:19 |
| sean-k-mooney | and you wil see the unicode | 08:19 |
| bauzas | interesting https://paste.opendev.org/show/bd91pr4uhj1SrmkHRjry/ | 08:21 |
| sean-k-mooney | ya so M-BM- | 08:22 |
| sean-k-mooney | is what a non breaking space looks like | 08:22 |
| bauzas | lemme try with vim to write another file | 08:22 |
| sean-k-mooney | there are plugins by the way for vscode to remove "irrgualar spaces" | 08:23 |
| sean-k-mooney | this is apprently a common enough thing that people took the time to write one | 08:23 |
| sean-k-mooney | i didnt want to recomend any specific one sicne i have never tested them or veted them | 08:23 |
| sean-k-mooney | "Fix Irregular Whitespace" is the most downloaded one | 08:25 |
| bauzas | can also reproduce it with vim | 08:25 |
| opendevreview | Taketani Ryo proposed openstack/nova-specs master: Adjust the SEV/SEV-ES code for multiple cc support https://review.opendev.org/c/openstack/nova-specs/+/962578 | 08:25 |
| bauzas | so that's not an editor issue, definitely a keyboard mapping problem | 08:26 |
| bauzas | oh fuck I found why | 08:27 |
| bauzas | that's PEBKAC | 08:27 |
| bauzas | I'm typing too fast so I wasn't releasing the altgr key when hitting space | 08:28 |
| bauzas | so I was typing 'hashtag' (altgr-3) then 'alternative-mapping' (alt-gr space) | 08:28 |
| bauzas | doh, I should set altgr-space to write a space | 08:29 |
| opendevreview | Bogdan Dobrelya proposed openstack/nova master: Determine the device type from configuration first https://review.opendev.org/c/openstack/nova/+/963702 | 08:33 |
| sean-k-mooney | bauzas: ya that is what i was assuming and because you have # on altgr 3 unlike the rest of us where it etiehr does not hava modifyier or its shift 3 | 08:37 |
| sean-k-mooney | you are just more likely ot pres altgr space then the rest of us | 08:37 |
| bauzas | yeah, I tested : shift-space produces a space, while altgr-space produces a non-visible combination of "M-BM- + space" | 08:40 |
| * gibi intrigued by bauzas' keyboard mapping | 08:59 | |
| bauzas | checked Gnome and my mapping is French (alt.) : confirmed that it hits nobreakspace | 09:02 |
| bauzas | that probably changed in a fedora upgrade | 09:02 |
| bauzas | changed my mapping to French, now it should be fixed | 09:09 |
| bauzas | thanks all | 09:09 |
| opendevreview | Bogdan Dobrelya proposed openstack/nova master: Determine the device type from configuration first https://review.opendev.org/c/openstack/nova/+/963702 | 09:42 |
| tkajinam | hmm it seems functional tests in osc-placement is failing due to a warning log added to placement recently https://bugs.launchpad.net/placement/+bug/2127848 | 11:23 |
| tkajinam | gibi, I wonder if that log should be really a warning or can it be debug ? https://opendev.org/openstack/placement/blame/commit/af09d65c3e24f214c43aebcb9cd069703ee87050/placement/objects/allocation_candidate.py#L827 | 11:24 |
| tkajinam | (by the way we should use warning instead of warn... | 11:35 |
| opendevreview | Takashi Kajinami proposed openstack/placement master: Replace LOG.warn https://review.opendev.org/c/openstack/placement/+/963948 | 11:35 |
| sean-k-mooney | tkajinam: it depend on if its actionable or not | 11:45 |
| sean-k-mooney | it proably shoudl be info level | 11:46 |
| sean-k-mooney | with that said osc-placemetn shoudl not be asserting anyting related to placmenet logging | 11:47 |
| sean-k-mooney | so this feels more like a osc placement bug in the test setup | 11:47 |
| sean-k-mooney | the placemetn logs form the placemetn fixture shoudl be capsture in the est setup and filtered out of that assertion | 11:48 |
| sean-k-mooney | in a real envonment that log woudl not be aviabel to the client | 11:49 |
| sean-k-mooney | tkajinam: so the proper fix woudl be to install a log filter int eh osc-palcement functional tests i think to null route the placement logs | 11:50 |
| tkajinam | sean-k-mooney, yeah I agree the issue and the lo level should be separate... | 11:56 |
| tkajinam | sean-k-mooney, I looked into the logging setting but it looks tricky to me. the core problem is that the test does not use actual nova api but uses placement api fixture and filtering all placement logs mean we lost all api logs https://opendev.org/openstack/osc-placement/src/branch/master/osc_placement/tests/functional/base.py#L77 | 11:57 |
| tkajinam | the test does not use actual *placement* api, I mean | 11:57 |
| tkajinam | maybe we could raise it to ERROR ? | 11:57 |
| sean-k-mooney | tkajinam: correct we shoudl nto have access to any of the api logs in osc-placement | 12:00 |
| sean-k-mooney | osc-placmenet is the openstack client plugin and in real code it will only have access to the api respocne not hte logs | 12:01 |
| tkajinam | it may be correct for assertions but I'm a bit afraid of loosing debuggability in test failures | 12:01 |
| sean-k-mooney | so if any test is relying on the logs then that test is invlaid | 12:01 |
| sean-k-mooney | we can keep the logs for the fixture but we shoudl not be loging the placment logs to stdout or stderr in generall but we shoudl atttach teh logs to the test output | 12:02 |
| sean-k-mooney | we can proably do somethign like this https://github.com/openstack/nova/blob/master/nova/config.py#L56-L89 | 12:05 |
| sean-k-mooney | add a log filt to the root loger that will intercept and redirect the logging else where but ill admit i have not tired | 12:05 |
| tkajinam | ok | 12:05 |
| tkajinam | let me give it some tries but I might just end up with bumping the log level filter to error | 12:06 |
| sean-k-mooney | perhaps like this https://github.com/openstack/nova/blob/8b81b5f91ffe1f9c38a483d151b82316d443dbf6/nova/tests/fixtures/nova.py#L190-L199 | 12:07 |
| sean-k-mooney | we may be able ot patch where placmenet or the placment fixutrer setup up logging iwth our own funciton | 12:08 |
| sean-k-mooney | tkajinam: i assume this is repoducable if we just clone master and run the tests? | 12:08 |
| tkajinam | yup | 12:09 |
| tkajinam | replace openstack-placement>=1.0.0 in tox.ini by git+https://opendev.org/openstack/placement#egg=openstack-placement | 12:09 |
| tkajinam | so that it uses non-released version | 12:09 |
| tkajinam | (that's what you need after clone, before tox -e functional | 12:09 |
| sean-k-mooney | ack. ill see if i can play around with it locally too | 12:09 |
| sean-k-mooney | tanks for reporting this | 12:09 |
| sean-k-mooney | im on pto the rest of the week and i dont have too much time today so i cant promis ill make much progress but i have a littel time between meeting today | 12:10 |
| gibi | tkajinam: that log will disappeare once we land a fix https://review.opendev.org/c/openstack/placement/+/962776 . I (wrongly) assumed that we will land the two patches close together so did not worried about the extra logs in the reproduce commit | 12:11 |
| sean-k-mooney | gibi: well it found a bug as a result but that also a good pont it shoudl be resovle when the actual fix lands | 12:13 |
| gibi | it was a log I added to prove that root cause of the bug | 12:14 |
| gibi | it is not necessary in production | 12:15 |
| gibi | I can spin up a patch that removes it independently from https://review.opendev.org/c/openstack/placement/+/962776 | 12:15 |
| gibi | so we can restore the sanity of the osc-placement's gate | 12:15 |
| * gibi goes spinning | 12:15 | |
| gibi | I did not load context on why osc-placement test depends on placmenet api logs. But that can be (and probably should be) fixed separately from unblocking the gate | 12:17 |
| tkajinam | gibi, ah, thanks ! | 12:19 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Remove troubleshooting logs https://review.opendev.org/c/openstack/placement/+/963952 | 12:22 |
| gibi | sean-k-mooney: , tkajinam ^^ | 12:23 |
| opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Filter out placement api logs in functional tests https://review.opendev.org/c/openstack/osc-placement/+/963953 | 12:23 |
| tkajinam | ^^^ sean-k-mooney gibi: this is the initial try to filter out these logs in osc-placement functional tests | 12:24 |
| tkajinam | gibi, thanks ! | 12:24 |
| gibi | tkajinam: I'm OK with that. If CI is green then I think we can land your test setup fix | 12:25 |
| gibi | that will probably land faster than my placement log removal patch | 12:26 |
| sean-k-mooney | tkajinam: i think that is fine we could proably be more granular but im not oppose to that | 12:26 |
| tkajinam | yeah we could probable filter more specific module but I tend to try with that wider filtering and see if that kills out life | 12:27 |
| tkajinam | * our life | 12:27 |
| tkajinam | my concern with narrow filtering is that we may face similar problems frequently in the future | 12:27 |
| tkajinam | so far it worked in my local reproducer but let's see how CI passes... | 12:28 |
| sean-k-mooney | so what i was going to propsoe is lsightly diffent. i was going to try and capture the placment api logs into a seperate stringio buffer and make them aviabel in tests but only log them if you run the test in debug mode | 12:28 |
| sean-k-mooney | so similar to how https://github.com/openstack/placement/blob/master/placement/tests/functional/fixtures/capture.py works internally | 12:30 |
| sean-k-mooney | or rather similar ot how https://github.com/openstack/oslotest/blob/9c1cb8e6df468a459783b49d59c60eee3cf83d4d/oslotest/log.py#L35 works internaly | 12:31 |
| sean-k-mooney | currently the way our test are setup teh python loing module is set up to log to stdout and stderr | 12:33 |
| sean-k-mooney | then https://github.com/openstack/oslotest/blob/9c1cb8e6df468a459783b49d59c60eee3cf83d4d/oslotest/output.py#L64-L71 | 12:33 |
| sean-k-mooney | intercept that if we enabel that fixture | 12:33 |
| sean-k-mooney | so what i want to try and do is see if i can redirect the placement loggers to a stringstream or stringio buffer instead and add them to the base test class as self.placment_api_log | 12:35 |
| sean-k-mooney | tkajinam: gibi just to be clear im fine with proceexcing with tkajinam's patch if it passes ci | 12:35 |
| gibi | cool. I think tkajinam's fix can land fast to unblock the gate | 12:36 |
| gibi | there is at least on green functional run already in 963953 | 12:37 |
| gibi | I mean in https://zuul.opendev.org/t/openstack/status?change=963953 | 12:37 |
| tkajinam | sean-k-mooney, I didn't try that way first, because in these functional tests of osc-placement we have a single root logger which receives both placement log and osc-placmenet's log | 12:41 |
| tkajinam | we can technically add different handlers and select these according to log module name but that's what current fixtures in oslotest and placement are designed for | 12:41 |
| tkajinam | (I'm not saying we can't/shouldn't do it but I'm saying we need additional consideration for this specific use case | 12:42 |
| sean-k-mooney | ya. lets not rush to make that work but if we can split it cleanly later it coudl be nice | 12:42 |
| sean-k-mooney | i am +2w on your current patch so if ci is green it will merge | 12:42 |
| gibi | sean-k-mooney: thanks | 12:43 |
| gibi | it passed check and on the gate now. So I'm abandoning https://review.opendev.org/c/openstack/placement/+/963952 | 12:49 |
| opendevreview | Merged openstack/osc-placement master: Filter out placement api logs in functional tests https://review.opendev.org/c/openstack/osc-placement/+/963953 | 12:52 |
| opendevreview | Takashi Kajinami proposed openstack/osc-placement master: Replace remaining py39 target https://review.opendev.org/c/openstack/osc-placement/+/963809 | 12:53 |
| tkajinam | \o/ | 12:54 |
| *** haleyb|out is now known as haleyb | 12:59 | |
| gibi | that was a quick one. Thanks folks | 13:01 |
| sean-k-mooney | gibi: i got ai to implement the other approch as well im just cleaning it up and makeing sure work properly | 13:06 |
| * bauzas reading back, gibi sean-k-mooney anything I can help to review ? | 13:13 | |
| sean-k-mooney | bauzas: no we are good we merged the quick fix to resolve the gate issue | 13:17 |
| ratailor | sean-k-mooney, gibi bauzas could you please review https://review.opendev.org/c/openstack/nova/+/937534 https://review.opendev.org/c/openstack/nova/+/938054 and https://review.opendev.org/c/openstack/nova/+/934294 | 13:19 |
| bauzas | sean-k-mooney: cool sorry I wasn't looking at the channel | 13:25 |
| opendevreview | sean mooney proposed openstack/osc-placement master: Improve functional test debuggability with log capture https://review.opendev.org/c/openstack/osc-placement/+/963962 | 13:30 |
| sean-k-mooney | gibi: tkajinam i need to join a call but ^ shoudl work | 13:30 |
| sean-k-mooney | ... | 13:30 |
| sean-k-mooney | after i fix the whitespace issues | 13:30 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | 13:40 |
| gibi | sean-k-mooney: dansmith: ^^ fixed up based on Dan's comments | 13:40 |
| dansmith | bauzas: if you're looking for reviews, we're still hoping you can +W the bottom two patches in the vtpm series | 13:44 |
| dansmith | and of course, contribute to the reviews up the stack | 13:44 |
| bauzas | shit, my brain forgor :) | 13:44 |
| bauzas | forgot* | 13:44 |
| dansmith | bauzas: https://review.opendev.org/c/openstack/nova/+/961734/4 | 13:45 |
| opendevreview | sean mooney proposed openstack/osc-placement master: Improve functional test debuggability with log capture https://review.opendev.org/c/openstack/osc-placement/+/963962 | 14:17 |
| Uggla | Nova meeting in ~30mn | 15:28 |
| opendevreview | Merged openstack/nova master: Add functional reproducer for bug 2125030 https://review.opendev.org/c/openstack/nova/+/961734 | 15:30 |
| *** gmaan_pto is now known as gmaan | 16:00 | |
| Uggla | #startmeeting nova | 16:01 |
| opendevmeet | Meeting started Tue Oct 14 16:01:59 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
| opendevmeet | The meeting name has been set to 'nova' | 16:01 |
| gibi | o/ | 16:02 |
| bauzas | o/ | 16:02 |
| nicolairuckel | o/ | 16:02 |
| gmaan | o/ | 16:02 |
| opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2026.1 Gazpacho https://review.opendev.org/c/openstack/nova/+/961896 | 16:02 |
| * gibi is on a parallel meeting | 16:02 | |
| fwiesel | o/ | 16:02 |
| Uggla | hey gmaan welcome back ;) | 16:03 |
| gibi | gmaan: oooh hi! | 16:04 |
| sean-k-mooney | o/ | 16:04 |
| gmaan | hi | 16:04 |
| Uggla | Let's start slowly. | 16:05 |
| elodilles | o/ | 16:05 |
| Uggla | #topic Bugs (stuck/critical) | 16:05 |
| Uggla | #info No Critical bug | 16:05 |
| Uggla | #topic Gate status | 16:05 |
| Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:06 |
| Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:06 |
| Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | 16:06 |
| Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:06 |
| Uggla | #info Please try to provide a meaningful comment when you recheck | 16:06 |
| Uggla | shame on me I have not looked at the gate. | 16:06 |
| Uggla | Have you seen something bad with it ? | 16:07 |
| bauzas | I saw some failures but the ones I was knowing | 16:08 |
| sean-k-mooney | bauzas: volume issue and kernel panics? | 16:08 |
| bauzas | ssh timeout at least | 16:09 |
| gibi | I saw guest ssh timeouts in couple of times in different test cases but did not dig deep | 16:09 |
| sean-k-mooney | ack | 16:09 |
| bauzas | eg. https://zuul.opendev.org/t/openstack/build/83dac09bed7947dead180357cb14d357 | 16:09 |
| gibi | ohh we have matching observation | 16:09 |
| sean-k-mooney | that ocationaly happens but if its increaseing we may need to debug | 16:09 |
| bauzas | we'll see with my rechecks :) | 16:10 |
| sean-k-mooney | gibi: was the issue with some vms geting multipel port fixed do you know? | 16:10 |
| sean-k-mooney | im not saying that related but just wondering if you new | 16:11 |
| gibi | I don't recall the issue | 16:11 |
| sean-k-mooney | *knew | 16:11 |
| sean-k-mooney | ack no worries | 16:11 |
| gibi | anyhow or the guest ssh I will keep an eye on it | 16:11 |
| sean-k-mooney | one of the provider was allcoating mulitple port to nodpoolvms which sometimes broke the network connectivy on the computes | 16:11 |
| sean-k-mooney | i assume that has been fixed | 16:11 |
| sean-k-mooney | it wasnt a nova problem | 16:11 |
| gibi | ohh now I recall it. I haven't see that appearing recently | 16:11 |
| sean-k-mooney | ack | 16:11 |
| sean-k-mooney | that job only has one port as expected so as i said not related | 16:12 |
| sean-k-mooney | we can proably move on unless anyone else has somethign | 16:13 |
| gibi | jepp, lets move on | 16:14 |
| Uggla | I agree | 16:14 |
| Uggla | Skipping next point, as I want gmaan to come back smoothly. | 16:14 |
| Uggla | #topic Release Planning | 16:14 |
| Uggla | #link https://releases.openstack.org/gazpacho/schedule.html | 16:14 |
| Uggla | #info PTG etherpad for 2026.1 is available: https://etherpad.opendev.org/p/nova-2026.1-ptg | 16:14 |
| Uggla | #info Please add the topics you would like to discuss in the above document. | 16:15 |
| Uggla | #info PTG meetings (to be confirmed) will happen room grizzly 15:00UTC -> 17:00UTC from Tuesday to Friday | 16:15 |
| Uggla | Note, I'm currently crafting the PTG etherpad defining the timeslots. | 16:16 |
| gibi | Uggla: 15-16, 16-17, or also 17-18? | 16:16 |
| gibi | I mean the last starting time is 17UTC or the finish time is 17 UTC? | 16:16 |
| sean-k-mooney | yep ^ its an inclusive range of startimes | 16:16 |
| sean-k-mooney | so its really 15:00-18:00 | 16:16 |
| Uggla | yep 15:00 --> 18:00 CET | 16:17 |
| gibi | ack, so 3 hours for 4 days | 16:17 |
| gibi | works forme | 16:17 |
| sean-k-mooney | based on https://ptg.opendev.org/ptg.html | 16:17 |
| Uggla | yep as usual, starting (Tue) | 16:17 |
| sean-k-mooney | well you need to adjust that actully | 16:17 |
| sean-k-mooney | it curernly show all 5 hours not just the fine 3 in the slot | 16:18 |
| Uggla | sean-k-mooney yes I reserved a bit larger at the first shot | 16:20 |
| bauzas | at least the slots are booked | 16:21 |
| Uggla | yep :) | 16:21 |
| bauzas | if we need to continue to discuss, we can book another slot... or we can unbook another one :) | 16:21 |
| Uggla | I'will arrange that in the coming days | 16:22 |
| Uggla | ok moving on | 16:24 |
| Uggla | #topic OpenAPI | 16:25 |
| Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:25 |
| Uggla | #info still 28 remaining atm. | 16:25 |
| Uggla | #topic Stable Branches | 16:25 |
| * Uggla giving the voice to elodilles | 16:25 | |
| elodilles | thanks o/ | 16:26 |
| elodilles | #info stable branches (stable/2025.* and stable/2024.*) seem to be in OK state | 16:26 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:26 |
| elodilles | #info 2024.1 Caracal (stable/2024.1) moves to Unmaintained at the end of next week - final stable Caracal release should be proposed and merged ASAP | 16:26 |
| elodilles | #info elod will propose the release patches tomorrow | 16:26 |
| elodilles | and a question: | 16:27 |
| elodilles | does anyone have any important patch to land to stable/2024.1 before we release? (list of open patches: https://etherpad.opendev.org/p/nova-stable-caracal-eom ) | 16:27 |
| Uggla | - | 16:27 |
| gibi | - | 16:27 |
| elodilles | thanks for the reviews so far, btw! | 16:27 |
| elodilles | Uggla: gibi: ACK, then i'll propose the review patches right away (flamingo, epoxy, dalmatian, caracal) | 16:28 |
| elodilles | (though caracal is the most important for now o:)) | 16:28 |
| elodilles | and that's all from me about stable for now | 16:29 |
| gibi | poor caracal, I will miss it | 16:29 |
| elodilles | :] | 16:29 |
| Uggla | rip caracal | 16:29 |
| elodilles | you don't have to dig it, it won't go to EOL, just Unmaintained o:) | 16:29 |
| elodilles | so people can still backport bug fixes to unmaintained/2024.1 ;) | 16:29 |
| sean-k-mooney | elodilles: as an aside there is only ment to be one unmtianed slurp release at a time | 16:30 |
| gibi | yeah basicaly kicked out to the streets nice elodilles :) | 16:30 |
| sean-k-mooney | elodilles: im fine with not enforce that as long as you still care about the rest | 16:30 |
| elodilles | sean-k-mooney: in case there is no volunteer to keep open unmaintained/2023.1 | 16:30 |
| sean-k-mooney | but in thory 2023.1 is ment to go eol when 2024.1 moves to unmainted | 16:30 |
| gmaan | antelope is still unmaintained, any plan to move to EOL before caracal moving to unmaintained ? | 16:31 |
| gmaan | sean-k-mooney: yeah | 16:31 |
| sean-k-mooney | so i think elodilles is still interested in keeping it alive | 16:31 |
| gmaan | because based on that we need to see if we can continue running grenade job on unmaintained caracal | 16:31 |
| sean-k-mooney | so as long as the gates are green that is fine | 16:31 |
| sean-k-mooney | well grenade is not ment to test upgrades form unmaintained | 16:32 |
| gmaan | as long as it run green then no harm to test it but if previous base relaese is not maintained or broken then yes we can remove | 16:32 |
| elodilles | sean-k-mooney: yes, i want to step up as a volunteer to keep it open | 16:32 |
| sean-k-mooney | or rahter the oldest stable is not ment to test upgrading form unmaintaiend but you coudl run it your self i guess | 16:33 |
| elodilles | sean-k-mooney: yes, the oldest stable is officially not supported, so can be disabled, though if it works then i'd rather ask for keeping it, and also help maintaining it as long as it possible | 16:34 |
| sean-k-mooney | well the counter point to that | 16:35 |
| sean-k-mooney | is we want to evntually be able to remove old disto images and mirrors | 16:35 |
| sean-k-mooney | so im not objecting to keepign it for a release or two bvut we shoudl nto keep it indefinetly | 16:36 |
| elodilles | of course if it blocks the gate then it can be removed (though i'd rather set it as non-voting if not completely unfixable or broken forever) | 16:36 |
| elodilles | sean-k-mooney: +1 | 16:36 |
| sean-k-mooney | 2023.1 is using ubuntu 22.04 which is supprot until 2027 | 16:36 |
| sean-k-mooney | that kind of the upper bound on testing that IMO | 16:36 |
| sean-k-mooney | so it has 18months+ runway i think | 16:37 |
| sean-k-mooney | anyway we can likely move on since its not being EOLd this cycle but we could talk about it more at the ptg if poepel wanted | 16:37 |
| elodilles | yepp | 16:38 |
| gmaan | I am only concern about the running grenade skip job on Epoxy which will be from caracal to Epoxy and if we can continue running it it will be great | 16:39 |
| gmaan | and it seems like we will do that until it is broken so I am good here | 16:39 |
| elodilles | gmaan: ++ | 16:39 |
| Uggla | ok so moving on. | 16:40 |
| Uggla | thx elodilles | 16:40 |
| Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:40 |
| fwiesel | No updates from my side | 16:40 |
| Uggla | fwiesel thanks | 16:41 |
| Uggla | #topic Gibi's news about eventlet removal | 16:41 |
| gibi | o/ | 16:41 |
| Uggla | gibi anything on your side ? | 16:41 |
| gibi | then first couple of patches are reviewable on https://review.opendev.org/c/openstack/nova/+/956089/ | 16:42 |
| gibi | and the ptg etherpad as the longer term plan | 16:42 |
| gibi | Also please report any strange test failures from nova-next, since last week we run conduct there in threading mode as well | 16:42 |
| gibi | (top of scheduler api, and metadata) | 16:43 |
| gibi | that is ti | 16:43 |
| gibi | it | 16:43 |
| sean-k-mooney | hum on the rbd thread pool i guess everually we will wnat to use the asinc api instead | 16:43 |
| sean-k-mooney | so we can actully pass timeouts ectra | 16:43 |
| sean-k-mooney | but i guess the intial plan is to just not swapn it on a tread pool at all which i guess is ok | 16:44 |
| gibi | sean-k-mooney: this will not prevent doing that later when we actaully go and translate the nova-compute | 16:44 |
| sean-k-mooney | ack ya just chekcign that still the medium term plan | 16:44 |
| gibi | the patches on that series are for unblocking unit testing | 16:44 |
| opendevreview | Merged openstack/nova master: Move cleanup of vTPM secret from driver to compute https://review.opendev.org/c/openstack/nova/+/962007 | 16:44 |
| sean-k-mooney | ah ok | 16:45 |
| gibi | I think we can move on | 16:47 |
| Uggla | thx gibi | 16:47 |
| Uggla | #topic Open discussion | 16:47 |
| Uggla | Just a reminder for the poll: Pool for the new schedule of upstream meeting: https://framadate.org/ao12zK6wR3K3nfbF | 16:47 |
| Uggla | Anything you want to discuss ? | 16:48 |
| nicolairuckel | I worked on a patch to preserve the NVRAM after reboots that got mentioned a few weeks ago. | 16:48 |
| nicolairuckel | I'm not sure if that's the correct place to talk about that. | 16:48 |
| Uggla | yes | 16:48 |
| nicolairuckel | https://review.opendev.org/c/openstack/nova/+/959682 | 16:48 |
| sean-k-mooney | looks like it needs a rebase | 16:49 |
| nicolairuckel | The only caveat is that at the moment you have to set the permissions for the NVRAM file manually. I wasn't able to figure out if there's a way to change the permissions through libvirt. | 16:49 |
| nicolairuckel | sean-k-mooney, I'll do that! | 16:50 |
| sean-k-mooney | why do you need to do that? | 16:50 |
| nicolairuckel | If you don't add read permissions, there will be a permissions error on a reboot and it won't be preserved. | 16:51 |
| nicolairuckel | The libvirt people told me that the permissions are like than intentionally. It's just a bit weird to me that the disk file has read permissions. | 16:52 |
| sean-k-mooney | wait why are you copying the file to the instnce dir an back | 16:52 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/959682/3/nova/virt/libvirt/driver.py#10919 | 16:52 |
| nicolairuckel | The file contains the UEFI variables that I want to preserve. | 16:52 |
| sean-k-mooney | i tought we were goign to preseve this by passign the libvirt flag to the doamin cleanup to preserve the file | 16:52 |
| nicolairuckel | You mean I could skip copying the file? | 16:53 |
| sean-k-mooney | yes | 16:53 |
| nicolairuckel | I see. I'm going to try that. | 16:53 |
| sean-k-mooney | so for the vtpm data we can pass a flag to tell libvirt not to delete it when the domein is undefiend | 16:53 |
| sean-k-mooney | i belive we can do the same for the nvram | 16:54 |
| sean-k-mooney | i could be mistaken but i belive there was a flag for the nvram as well | 16:54 |
| nicolairuckel | That would be a better solution for sure. | 16:54 |
| sean-k-mooney | so we would not need ot copy the file | 16:54 |
| nicolairuckel | I'm going to look into that, thank you. | 16:54 |
| nicolairuckel | How can I find a reviewer when that is done? Should I just bring it up again next week? | 16:55 |
| sean-k-mooney | you can also ping on irc | 16:56 |
| nicolairuckel | okay | 16:56 |
| sean-k-mooney | we will proably see it updated but you can reach out if we dont | 16:57 |
| Uggla | nicolairuckel, ping on the channel | 16:57 |
| nicolairuckel | thanks | 16:57 |
| nicolairuckel | that would be all from my side then | 16:57 |
| gibi | nicolairuckel: thanks for working on it | 16:57 |
| sean-k-mooney | VIR_DOMAIN_UNDEFINE_KEEP_NVRAM | 16:58 |
| sean-k-mooney | https://github.com/libvirt/libvirt/blob/b42a12174c787b99cd6fcb29b44e4b13bd64ee58/include/libvirt/libvirt-domain.h#L2373 | 16:58 |
| sean-k-mooney | that the flag we need to pass to not delete it when the domain is undeifed | 16:58 |
| Uggla | nicolairuckel, yes thanks for working on it | 16:58 |
| Uggla | time to close | 17:00 |
| Uggla | Thank you for participating in this meeting. | 17:00 |
| Uggla | #endmeeting | 17:00 |
| opendevmeet | Meeting ended Tue Oct 14 17:00:39 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.html | 17:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.txt | 17:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.log.html | 17:00 |
| elodilles | thanks o/ | 17:01 |
| gibi | hanks | 17:03 |
| gibi | thanks | 17:03 |
| gibi | :) | 17:03 |
| elodilles | tom? o:) | 17:06 |
| gibi | nemtom (sorry this only funny in hungarian) | 17:10 |
| sean-k-mooney | by the way im on pto retruning on tusday | 17:18 |
| sean-k-mooney | just an fyi that i wont be here for the rest of the week | 17:18 |
| gibi | sean-k-mooney: have a nice one | 17:19 |
| opendevreview | Merged openstack/nova master: Switch nova-conductor to use global executor https://review.opendev.org/c/openstack/nova/+/962478 | 18:45 |
| opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | 20:09 |
| opendevreview | Fernando Ferraz proposed openstack/nova master: [DNM] Get processutils logs from unprivileged_convert_image https://review.opendev.org/c/openstack/nova/+/964095 | 22:01 |
| opendevreview | melanie witt proposed openstack/nova master: Add handling for vTPM secret permission error https://review.opendev.org/c/openstack/nova/+/963648 | 23:22 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!