opendevreview | Rajesh Tailor proposed openstack/nova-specs master: Show finish_time field in instance action show https://review.opendev.org/c/openstack/nova-specs/+/929780 | 07:10 |
---|---|---|
opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing instance-action finish_time https://review.opendev.org/c/openstack/nova/+/928933 | 07:47 |
opendevreview | Friedrich Hiekel proposed openstack/nova master: Fix nova-scheduler placement error https://review.opendev.org/c/openstack/nova/+/951936 | 07:55 |
stephenfin | sean-k-mooney: any chance you could grab https://review.opendev.org/c/openstack/nova/+/950867 again? | 09:21 |
opendevreview | Merged openstack/nova master: Fix small documentation issue https://review.opendev.org/c/openstack/nova/+/952151 | 09:22 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with remote consoles APIs https://review.opendev.org/c/openstack/nova/+/952213 | 09:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs https://review.opendev.org/c/openstack/nova/+/952214 | 09:53 |
stephenfin | Uggla: Follow-ups for your comments on the openapi series are there, if and when you have time ^ | 10:17 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with instance actions API https://review.opendev.org/c/openstack/nova/+/951941 | 10:51 |
sean-k-mooney | stephenfin: yep added +2 | 12:10 |
sean-k-mooney | by the way im still.... seeing random kernel panics related to cinder volume detach. i know we never fixed that but im wondering if we shoudl consider creating a decorator in tempest to detect that and retry the test. i.e. if the console output form tempest has "end Kernel panic" in it just retry the test once more and only fail if that second try also fails | 12:14 |
Uggla | Hi stephenfin, I will review it thx. | 12:21 |
sean-k-mooney | stephenfin: replied on https://review.opendev.org/c/openstack/openstacksdk/+/948584 thansk for the review, i might not get back to it for a few days but ill let you knwo when i have a new revision up | 12:33 |
stephenfin | sean-k-mooney: ack | 12:36 |
*** haleyb|out is now known as haleyb | 12:50 | |
Uggla | Nova meeting in ~1h. | 15:03 |
cardoe | I wanted to follow up on https://review.opendev.org/c/openstack/nova-specs/+/471815 and ask what my next steps are to advance that. | 15:04 |
opendevreview | Merged openstack/os-vif master: OVS Trunk: Add bridge_name to external_ids https://review.opendev.org/c/openstack/os-vif/+/949736 | 15:08 |
opendevreview | Stephen Finucane proposed openstack/nova master: db: Resolve alembic deprecation warning https://review.opendev.org/c/openstack/nova/+/952266 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | 15:21 |
opendevreview | Stephen Finucane proposed openstack/nova master: api-ref: Fix server topology API docs https://review.opendev.org/c/openstack/nova/+/952274 | 15:41 |
stephenfin | Uggla: That last one is for you | 15:42 |
Uggla | stephenfin, thanks for fixing this. | 15:50 |
Uggla | Nova meeting in ~10mn | 15:50 |
*** ChanServ sets mode: +o dansmith | 15:52 | |
*** maeda was kicked by dansmith (Your behavior is not conducive to the desired environment.) | 15:53 | |
*** dansmith sets mode: +b *!~maeda@240a:61:1196:9753:b065:99e5:8c7:29a2 | 15:56 | |
*** maeda was kicked by dansmith (Your behavior is not conducive to the desired environment.) | 15:56 | |
bauzas | tick tock | 16:00 |
Uggla | #startmeeting nova | 16:01 |
opendevmeet | Meeting started Tue Jun 10 16:01:12 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 |
Uggla | Hello everyone | 16:01 |
fwiesel | o/ | 16:02 |
bauzas | o/ | 16:02 |
dansmith | o/ | 16:03 |
*** ChanServ sets mode: -o dansmith | 16:03 | |
tkajinam | o/ | 16:04 |
Uggla | Let's start | 16:04 |
Uggla | #topic Bugs (stuck/critical) | 16:04 |
Uggla | #info No Critical bug | 16:04 |
gmaan | o/ | 16:05 |
* gibi lurks from pto | 16:05 | |
Uggla | #info https://review.opendev.org/c/openstack/nova/+/922140 bauzas working on a patch to enable mdev test again. | 16:05 |
bauzas | had no time to look at it :( | 16:06 |
Uggla | bauzas no worries. | 16:06 |
Uggla | If you wish I can remove it from this topic. | 16:07 |
Uggla | gibi, are you crazy. ;) | 16:07 |
Uggla | #topic Gate status | 16:08 |
Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:08 |
Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:08 |
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:08 |
Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:08 |
Uggla | #info Please try to provide meaningful comment when you recheck | 16:08 |
Uggla | Note: I have not checked periodics this week. | 16:09 |
Uggla | #topic tempest-with-latest-microversion job status | 16:09 |
Uggla | #link https://zuul.opendev.org/t/openstack/builds?job_name=tempest-with-latest-microversion&skip=0 | 16:09 |
Uggla | gmaan do you want to add something abut ^ | 16:10 |
gmaan | did not get chance to do any progress on it. | 16:10 |
gmaan | nothing to share from me | 16:10 |
Uggla | ok, no pb, moving on. | 16:10 |
Uggla | #topic Release Planning | 16:10 |
Uggla | #link https://releases.openstack.org/flamingo/schedule.html | 16:11 |
Uggla | #info Nova Spec review day will be next week. | 16:11 |
Uggla | #info IMHO, we should start reviewing specs. | 16:11 |
Uggla | #topic Review priorities | 16:12 |
Uggla | #link https://etherpad.opendev.org/p/nova-2025.2-status | 16:12 |
Uggla | I will make sure it is up2date with current reviews for specs. | 16:12 |
Uggla | But that should be not too bad. I think almost every spec is part of the document. | 16:13 |
Uggla | #topic OpenAPI | 16:14 |
Uggla | #link: https://review.opendev.org/q/topic:%22openapi%22+(project:openstack/nova+OR+project:openstack/placement)+-status:merged+-status:abandoned | 16:14 |
Uggla | #info 18 patches remaining lots of them in the pipe ready to be merged. | 16:14 |
Uggla | Thanks sean-k-mooney and gmann. Let's continue this way. | 16:15 |
Uggla | gmaan* | 16:15 |
sean-k-mooney | i think stephen has ~ anouther 10 or so after the current set | 16:15 |
sean-k-mooney | but they are holding them until the current batch are in | 16:15 |
sean-k-mooney | but ya we are makeing progress | 16:16 |
Uggla | Yep there are follow up patches. | 16:16 |
Uggla | btw, I'm counting only the ones for nova. I know there are many for cinder, etc... | 16:16 |
gmaan | I did not check the new one but I was planning to finish the series today | 16:17 |
sean-k-mooney | so was i :) | 16:17 |
Uggla | 👍 | 16:17 |
sean-k-mooney | gmaan: ack i proably wont get to them until next week but i may have time to look on friday | 16:17 |
gmaan | ack | 16:18 |
Uggla | I'm gonna skip stable branch topic because Elod is on pto. | 16:18 |
Uggla | #topic vmwareapi 3rd-party CI efforts Highlights | 16:19 |
Uggla | fwiesel, something on your side ? | 16:19 |
fwiesel | Hi, no updates from my side | 16:19 |
Uggla | 👍 | 16:19 |
Uggla | #topic Gibi's news about eventlet removal. | 16:19 |
gibi | o/ | 16:19 |
Uggla | #link Blog: https://gibizer.github.io/categories/eventlet/ | 16:19 |
Uggla | #link nova-scheduler series is ready for core review, starting at https://review.opendev.org/c/openstack/nova/+/947966 | 16:20 |
gibi | I'v booked a weekly video call to sync up | 16:20 |
gibi | Eventlet weekly sync Wednesday 14:30 UTC https://meet.google.com/bcy-uqoz-hje | 16:20 |
gibi | I might or might not be able to host it tomorrow due to PTO, but if I cannot you can still try to sync there. | 16:20 |
Uggla | ok. | 16:21 |
gibi | that is all from me | 16:21 |
sean-k-mooney | gibi: i tought you were on pto today as well | 16:21 |
Uggla | sean-k-mooney, he is. | 16:21 |
gibi | I'm :) | 16:21 |
sean-k-mooney | gibi: ok then go enjoy :) | 16:22 |
gibi | sure. thanks | 16:22 |
Uggla | yep gibi time to disconnect. :) | 16:22 |
Uggla | #topic Open discussion | 16:22 |
Uggla | there is nothing in the agenda. | 16:23 |
* gibi retreats to the shadows.. | 16:23 | |
Uggla | Someone wants to discuss about something ? | 16:23 |
opendevreview | Merged openstack/nova master: update pre-commit version pins https://review.opendev.org/c/openstack/nova/+/949291 | 16:23 |
opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | 16:23 |
tkajinam | just a quick question but is anyone working on cleaning up some code after libvirt min version bump ? | 16:23 |
Uggla | tkajinam yep I have a WIP patch for this. | 16:24 |
tkajinam | Uggla, ah, ok. I'll search it | 16:24 |
tkajinam | I have a few items in my local TODO (mostly related to TPM feature detection) so will look into it but I didn't want to duplicate the effort if the core part is covered | 16:25 |
Uggla | I left some TODO's for removal in the code, so this patch will be to remove what is not useful anymore. | 16:25 |
tkajinam | yeah I saw these comments | 16:26 |
sean-k-mooney | tkajinam: i should put that back on my review list | 16:26 |
sean-k-mooney | tkajinam: if i recall it was pretty close before FF | 16:26 |
sean-k-mooney | tkajinam: is the tpm seriese rebased and passing? | 16:27 |
tkajinam | sean-k-mooney, are you talking about sev-es series, right / | 16:27 |
tkajinam | ? | 16:27 |
tkajinam | I guess it has been kept for some time without rebase. I'll rebase it later after the meeting | 16:27 |
sean-k-mooney | the detection one. but the sev serise is also intersting | 16:27 |
sean-k-mooney | ack | 16:27 |
tkajinam | hm ok. I believe the detection feature was merged in the past but I'll recheck if there is anything remaining | 16:28 |
sean-k-mooney | i know we merged the swtpm and version detection | 16:28 |
sean-k-mooney | i dont recall if we also merged the patch to auto detect the capsity | 16:28 |
sean-k-mooney | although maybe we did | 16:28 |
tkajinam | I believe so but I'll double-check it | 16:29 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with keypairs APIs https://review.opendev.org/c/openstack/nova/+/952214 | 16:29 |
sean-k-mooney | ack, so your next priority would be the patches to extend that to SEV-ES right | 16:29 |
tkajinam | these are not really related but yeah sev-es would be the next one I may want to have. | 16:29 |
sean-k-mooney | ack | 16:30 |
tkajinam | that helps pending sev-snp as well as arm-cca which some folks from fujitsu have been working on | 16:30 |
tkajinam | I was supposed to submit spec for sev-snp but I'm not very sure if I can push one quite soon | 16:30 |
sean-k-mooney | no worries just askign for context | 16:31 |
tkajinam | :-) | 16:31 |
Uggla | I guess nobody expects SEV-SNP this cycle. | 16:32 |
tkajinam | yeah, though I have got a few local pings about my plan about it. | 16:33 |
tkajinam | anyway we can follow up separately about sev thing and can wrap up the meeting unless anyone else have their topics. (assuming not many people may be interested in such nich feature :-P ) | 16:33 |
Uggla | yep, something else before moving to bug scrubbing. | 16:34 |
tkajinam | ahh yes | 16:34 |
Uggla | #topic Bug scrubbing | 16:35 |
Uggla | #link: https://etherpad.opendev.org/p/nova-bug-selection-for-triaging#L4 | 16:35 |
Uggla | #link: https://truc.uggla.fr/ I kicked off this draft page to follow open bugs. | 16:35 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server group APIs https://review.opendev.org/c/openstack/nova/+/952281 | 16:35 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for removed APIs https://review.opendev.org/c/openstack/nova/+/952282 | 16:35 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for networks API https://review.opendev.org/c/openstack/nova/+/952283 | 16:35 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs https://review.opendev.org/c/openstack/nova/+/952284 | 16:35 |
Uggla | First: | 16:36 |
Uggla | https://bugs.launchpad.net/nova/+bug/2112808 - nova-compute restart breaks queued live migration objects (artom) | 16:36 |
Uggla | I think this is probably valid. sean-k-mooney do you know about it ? | 16:37 |
sean-k-mooney | so this came form a 16.2 bug report i.e. train | 16:38 |
sean-k-mooney | i know we did some work to improve this since then so before considering it valid we likely need to see if it can happen in master | 16:39 |
Uggla | yes it seems to be the case. Artom mentioned that it is probably applicable on master. | 16:39 |
sean-k-mooney | with that said there probely is a bug in the init_host logic that is ment to heal migrations on start up | 16:39 |
sean-k-mooney | so it may be but the bug does nto really meet or minium stnadard for reporting a bug :) | 16:40 |
Uggla | I know. :) | 16:40 |
sean-k-mooney | https://issues.redhat.com/browse/OSPRH-13122 is public but you should not have to click into downstream tackers to asses the bugs valididity | 16:41 |
sean-k-mooney | anyway the downstream bug suggest the bug is here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1040-L1057 | 16:41 |
Uggla | Is it possible to create a reproducer with fn tests in that case ? | 16:41 |
sean-k-mooney | and the logic that its only catherign for runnign mighrations | 16:42 |
sean-k-mooney | looks correct | 16:42 |
sean-k-mooney | so this proably is a bug | 16:42 |
sean-k-mooney | that affect master but ya we need a proper repoducer fo this to fix it | 16:42 |
sean-k-mooney | and yes our funcitonal tests shoudl be able to repoduce this | 16:43 |
sean-k-mooney | there is one slight complciation with how restarts work in the functional enve but we have some exampels of this already | 16:43 |
sean-k-mooney | lets mark it as triaged and add a permalink to the relevnet code https://github.com/openstack/nova/blob/517415b6cbdd1730b5179b714229cfc0a65b1024/nova/compute/manager.py#L1043-L1060 | 16:44 |
Uggla | ok good. That works for me. I'll do that later. Thanks | 16:44 |
Uggla | Next one: | 16:45 |
Uggla | ttps://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot | 16:45 |
Uggla | https://bugs.launchpad.net/nova/+bug/2112373 - Flavor Extra Specs for SCSI Controller Model Not Honored for Volume Boot | 16:45 |
Uggla | That looks valid too. I may encourage the reporter to submit a patch because he proposed something in the report. | 16:47 |
sean-k-mooney | that is valid, i fixed that for non BFV instnaces years ago | 16:48 |
sean-k-mooney | but i never fixed BFV | 16:48 |
sean-k-mooney | the porblem is with the way we do mappings | 16:48 |
sean-k-mooney | basiclly we mapp sata and scsi to sd* | 16:49 |
sean-k-mooney | and as a result we end up with the the wrong type or something like that | 16:49 |
sean-k-mooney | this is a old know issue | 16:49 |
sean-k-mooney | we have a downstream tracker for this somewhere i think | 16:50 |
sean-k-mooney | oh | 16:50 |
sean-k-mooney | actully this is sligly diffent maybe | 16:50 |
sean-k-mooney | hw:scsi_model i think is only an imave property | 16:50 |
sean-k-mooney | what i was descibing was the bug when you have that image peropery on the volume in the volume_image_propties filed | 16:51 |
sean-k-mooney | i dont recally if we ever added hw:scsi_model and hw:disk_bus | 16:51 |
sean-k-mooney | as flavor extra specs | 16:51 |
sean-k-mooney | the were orginaly only image properties | 16:51 |
sean-k-mooney | that woudl make this a feature request not a bug | 16:52 |
sean-k-mooney | let me check the valdiation code | 16:52 |
sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/api/validation/extra_specs/hw.py | 16:52 |
sean-k-mooney | so ya this is not soemthing we supprot today | 16:53 |
Uggla | What do you think about reporter fallback ? | 16:54 |
sean-k-mooney | we coudl add the rfe tag and make it a wishlest bug | 16:54 |
sean-k-mooney | but it is a new feature and soemthign we intentiaonly did not suprpot in the past | 16:54 |
sean-k-mooney | Uggla: historically flavor extra specs were for capstyisty and imave proprites were used for things liek device model or busses | 16:55 |
sean-k-mooney | the logic is that only the image know what type of virutal device it needs | 16:55 |
sean-k-mooney | and since some clodus do not show flavor extra specs to user allowing it to be set in the flavor coudl break enduses expcations. | 16:56 |
sean-k-mooney | with that said we have relaxed that over the years and now allow many thing to be set in either | 16:56 |
sean-k-mooney | i can write somethign up but really this shoudl be a small specless blueprint or as i said we coudl consider it a wishlist bug with the rfe tag | 16:57 |
Uggla | or we can close the bug and ask reporter to propose something in next PTG ? | 16:58 |
sean-k-mooney | we could | 16:59 |
sean-k-mooney | i added a comment to the bug | 16:59 |
sean-k-mooney | do we want to revisit it next week after they have time to respond | 17:00 |
Uggla | sure. | 17:00 |
Uggla | thanks | 17:00 |
Uggla | We are on the top of the hour. So closing the meeting. | 17:01 |
Uggla | Thanks all, see you next week. | 17:01 |
Uggla | #endmeeting | 17:01 |
opendevmeet | Meeting ended Tue Jun 10 17:01:52 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.html | 17:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.txt | 17:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-06-10-16.01.log.html | 17:01 |
sean-k-mooney | Uggla: that other bug i initally descibe is a real bug with bfv or addtional data volumes by the way i just dont have a link to it | 17:02 |
Uggla | ok | 17:02 |
Uggla | sean-k-mooney, I will look at your comment and update the bugs, probably tomorrow morning. | 17:03 |
Uggla | sean-k-mooney, thanks for your help. | 17:03 |
sean-k-mooney | oh its https://bugs.launchpad.net/nova/+bug/2022093 | 17:03 |
sean-k-mooney | nova is only ment ot read the prories form the root disk so setting hw_disk_bus=sata is a redherring there but | 17:04 |
sean-k-mooney | its still incorrect beahvior | 17:04 |
sean-k-mooney | all disks shoudl bave been added with sata if the disk type was not cdrom | 17:05 |
Uggla | ok I'll read this bug with a fresh mind. | 17:07 |
tkajinam | sean-k-mooney, fyi. full series of tpm feature detection was merged https://review.opendev.org/q/topic:%22bp/libvirt-detect-vtpm-support%22 and now nova detects (1) tpm support (2) tpm models (3) tpm versions via libvirt API | 17:07 |
tkajinam | (1) and (2) requires libvirt 8.0.0 so now I can clean up fallback for these, while (3) requires 8.6.0 so slight logic may be kept | 17:08 |
* tkajinam runs away | 17:08 | |
sean-k-mooney | tkajinam: ack sound good | 17:10 |
opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | 17:33 |
opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | 17:33 |
opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | 17:34 |
opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | 17:34 |
*** ChanServ sets mode: +o dansmith | 17:47 | |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Update to use '/v2.1' prefix, avoid project ID https://review.opendev.org/c/openstack/nova/+/952267 | 17:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Remove unnecessary API version overrides https://review.opendev.org/c/openstack/nova/+/952268 | 17:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use v2.1 API in OSAPIFixture https://review.opendev.org/c/openstack/nova/+/952269 | 17:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | 17:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | 17:48 |
opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | 17:48 |
opendevreview | Merged openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | 22:39 |
opendevreview | Merged openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | 22:47 |
opendevreview | sean mooney proposed openstack/os-vif master: remove linux bridge plugin https://review.opendev.org/c/openstack/os-vif/+/941586 | 23:48 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!