Tuesday, 2024-07-16

opendevreviewMerged openstack/nova master: hardware: Fix image_meta.id within get_mem_encryption_constraint  https://review.opendev.org/c/openstack/nova/+/90419704:39
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack"04:51
whoami-rajathey #openstack-nova , can i have review on this small patch? it's sitting for quite a while https://review.opendev.org/c/openstack/nova/+/88925705:19
amorinmelwitt, sean-k-mooney so maybe I misconfigured something because my volume endup in a different AZ when doing my tests, I will double check, thanks for your answers07:34
bauzas_morning folks09:11
*** bauzas_ is now known as bauzas09:14
sean-k-mooneyamorin: check if you have the cross az attach config option configured to prevent cross az attach11:25
sean-k-mooneybauzas: this is a very short spec form melwitt which we have discussed at previous ptgs https://review.opendev.org/c/openstack/nova-specs/+/923807 will you have time to review that today12:47
bauzasyes, I'm slowing starting back12:47
sean-k-mooneyack melwitt also has a poc implemation up for it12:48
fricklerbauzas: Uggla: auniyal: could you please check https://review.opendev.org/c/openstack/releases/+/923118 once again? (osc-placement release)12:52
weanewHello! I've submitted a bug fix: https://review.opendev.org/c/openstack/nova/+/923395 Could someone review it?13:16
opendevreviewTakashi Natsume proposed openstack/nova-specs master: Create specs directory for 2025.1 Epoxy  https://review.opendev.org/c/openstack/nova-specs/+/92422713:21
jlejeunehello, can I have some review for my fix in unit tests please: https://review.opendev.org/c/openstack/nova/+/924148 ?13:36
bauzasfolks, can we try to not use this channel for asking for some fixes ?13:39
bauzasjlejeune: weanew: if you want, you can add your changes in the Dalmatian etherpad : https://etherpad.opendev.org/p/nova-dalmatian-status13:40
jlejeunebauzas: ok, I didn't create a bug in launchpad for that because it just affects unit tests, is that ok ?13:49
bauzasjlejeune: yup, for internal changes like that, indeed you don't need to add a new Launchpad bug report13:57
jlejeunebauzas: perfect, thanks13:58
bauzasreminder (it was long due) : next nova meeting here in 1.5 hours (1600UTC)14:25
opendevreviewBalazs Gibizer proposed openstack/nova master: DNM: test back and forth migration  https://review.opendev.org/c/openstack/nova/+/92423414:55
elodillesdansmith sean-k-mooney : i know it's unmaintained, but could you please have a quick look that my thinking was right when resolving conflict in this backport? https://review.opendev.org/c/openstack/nova/+/92408315:00
elodillesconflicts were due to patch to consolidate image creation functions (I111cfc8a5eae27b15c6312957255fcf973038ddf) was added only in zed15:01
elodillesand as far as i understood, now i only had to touch the create_cow_image function, and no need for to change anything in create_image function (nova/virt/libvirt/utils.py)15:02
elodillesas create_cow_image functions handles only the case when there is a backing_file15:03
sean-k-mooneyelodilles: a very light reivew, it looks correct but havented dnoe anything more then compare the 2023.1 patch to the yoga version15:08
elodillessean-k-mooney: ACK, thanks, i know that this is unmaintained, so i appreciate it! o:)15:09
sean-k-mooneythe ci results are relivily clean too15:10
opendevreviewMerged openstack/nova-specs master: Propose config option to control behavior of unset unified limits  https://review.opendev.org/c/openstack/nova-specs/+/92380715:10
elodillesyepp, hence i dared to ask o:)15:10
sean-k-mooneythis obviously still has the iso regression15:11
sean-k-mooneyand im not sure if you have alredy fixed the ami format15:12
sean-k-mooneyok the ami regression is adressed in the backport of a previous file as it was in master15:13
sean-k-mooneyhttps://review.opendev.org/c/openstack/nova/+/924082/1/nova/virt/images.py#16315:13
sean-k-mooneyso the main regression is the breakgae of iso supprot form rescue, volumes and boot form iso15:14
elodillesso you say something is still missing?15:19
sean-k-mooneyelodilles: not for the cve mitigation but the cve broke seveeral features15:22
sean-k-mooneythe one you are missing is the fixes for ISO files15:22
sean-k-mooneyelodilles: this series https://review.opendev.org/q/topic:%22format-inspector%2215:23
elodillessean-k-mooney: oh, that, those are still on my TODO to backport o:) ACK15:26
elodillesthx o/15:26
sean-k-mooneywe backported them downstream to our trian equivelent branch with very little issues15:27
sean-k-mooneyall of thhis code is basically new so there is almost no opertunity for there to be a conflict15:27
elodillesoh, good to hear that :]15:29
bauzas#startmeeting nova16:00
opendevmeetMeeting started Tue Jul 16 16:00:02 2024 UTC and is due to finish in 60 minutes.  The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'nova'16:00
fwieselo/16:00
tkajinamo/16:00
bauzashey everyone, welcome back, it was long overdue16:00
elodilleso/16:00
bauzas#link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting16:01
auniyal o/16:01
bauzasI guess we can softly start16:02
bauzas#topic Bugs (stuck/critical) 16:02
bauzas#info No Critical bug16:02
bauzas#info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster16:02
bauzasdo people want to discuss about some specific bugs, like some now public ones :)16:03
sean-k-mooneyi think you can take that as a no16:04
bauzasthanks btw. dansmith and sean-k-mooney for https://bugs.launchpad.net/nova/+bug/205980916:04
bauzasmoving on16:05
gibio./16:05
sean-k-mooney*for fixing https://bugs.launchpad.net/nova/+bug/2059809 :) elodilles did a lot fo the hevay lifting on the backports as did gibi 16:05
bauzas:)16:05
elodilleso:)16:05
bauzassorry, I was on PTO so I wasn't able to help :)16:06
bauzasanyway, moving on16:06
bauzas#topic Gate status 16:06
sean-k-mooneyits fine it got done in the end16:06
bauzas#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:06
bauzas#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:06
bauzas#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status16:06
bauzasall greens16:06
bauzas#info Please look at the gate failures and file a bug report with the gate-failure tag.16:06
bauzas#info Please try to provide meaningful comment when you recheck16:06
bauzasso, how is the current CI ?16:07
sean-k-mooneybetter then it was. we merged the zswap changes and that seams to help stablise things16:07
sean-k-mooneyi have a patch up for review to slightly increase the ram of the default falvor16:07
sean-k-mooneyim hoping that will reduce or remove the kernel panics16:08
bauzasok16:08
sean-k-mooneyso we will want to monitor those once that merges16:08
bauzasI could look at opensearch 16:09
sean-k-mooneywe could do that yes alhtough im hopign it will become obvious or not if its enouch16:10
sean-k-mooneyhttps://review.opendev.org/c/openstack/devstack/+/924094 is the patch in question16:10
sean-k-mooneydownstream we obseved a crash that we have not seen since we changed form 128 to 256 mb upstream i only went to 19216:10
sean-k-mooneyso lets see how thing go for the next week16:11
tkajinamthat makes very clear sense16:12
bauzasokay, I see that it's close to be merged16:12
sean-k-mooneyyep i also did a DNM to nova https://review.opendev.org/c/openstack/nova/+/92414216:13
*** ykarel is now known as ykarel|away16:13
sean-k-mooneyjust to run our full set of jobs and it was green16:13
bauzasfor the guest panics, I'll ask opensearch about something like " ---[ end Kernel panic "16:14
bauzasanyway, we'll see16:15
bauzasmoving on ?16:15
bauzaslooks so16:15
bauzas#topic Release Planning 16:15
bauzas#link https://releases.openstack.org/dalmatian/schedule.html16:15
bauzas#info Dalmatian-3 in 6 weeks16:16
bauzas#info Spec approval freeze planned on Thursday16:16
bauzasso that will be my review priority until Thursday16:16
bauzasafter Thursday EOD, we won't accept any other specs for Dalmatian, but you can then ask them for Epoxy once we merge https://review.opendev.org/c/openstack/nova-specs/+/92422716:17
bauzasanything about the spec freeze ?16:17
tkajinamI'm willing to move SEV-ES support forward this cycle so am looking for further feedback to https://review.opendev.org/c/openstack/nova-specs/+/907702 :-)16:18
fwieselI'd like raise attention then to my lazy-metadata-loading spec: https://review.opendev.org/c/openstack/nova-specs/+/92220116:19
sean-k-mooneyi can try and review that before thrusday16:19
bauzastkajinam: ack, I'll review 16:19
bauzasditto for fwiesel16:19
bauzasmoving on so16:20
bauzas#topic Review priorities 16:20
bauzas#link https://etherpad.opendev.org/p/nova-dalmatian-status16:20
bauzas#action bauzas to update the above etherpad and Launchpad by tomorrow morning16:20
bauzasI need to exactly look at what we already accepted16:20
bauzas#topic Stable Branches 16:21
bauzaselodilles: heya16:21
elodilleso716:21
elodilles#info stable gates should be in OK state16:21
elodilles#info nova stable versions were released a week ago: 29.1.0 (2024.1 Caracal), 28.2.0 (2023.2 Bobcat), 27.4.0 (Antelope)16:21
elodillesbut i guess we will need new releases after some time as new fixes landed after the release16:21
elodilleseither feel free to propose the release patch or ping me16:22
elodillesand that's all from me for stable for now16:22
bauzasokay, we'll see 16:22
bauzasand indeed we will need to create new releases due to the cve fixes16:22
sean-k-mooneydoes already shipped btu the regression fixes are pending16:23
bauzaswe'll just see when it's needed16:23
bauzasyeah that16:23
tkajinamonce a new release with follow-up fixes is created then we can start the discussion to oslo'nize the implementation16:23
sean-k-mooneyrelease are fortunetly relitively cheap16:24
elodillessean-k-mooney: yepp16:24
sean-k-mooneyalso yes that 16:24
bauzasgiven all the distros already fixed their cves, this is not that urgent, but I'd still want to get those stable versions quickly16:24
sean-k-mooneybauzas: i mean it semi urgent16:24
sean-k-mooneywe broke all useage of isos in nova16:24
bauzassemi-urgent indeeed16:24
bauzasI'm not telling about the backports16:25
bauzasbut about the stable releases16:25
sean-k-mooneyyep we broke all useage of iso on the sable branches too16:25
sean-k-mooneythey are now fixed but not released16:25
gibi:)16:25
sean-k-mooneybut we can likely od that in the next weeek or so16:26
sean-k-mooneyi think the last patch to stable/2023.1 landed friday16:26
bauzas#topic vmwareapi 3rd-party CI efforts Highlights 16:26
fwiesel#info No updates16:26
bauzasfwiesel: want to say anything ?16:26
bauzascool with me16:26
bauzas#topic Open discussion 16:26
bauzasanything from me, anything anyone ?16:27
tkajinamI have one thing16:27
tkajinamI have a few items pending on os-traits change. I wonder if I can ask for an early release (probably right after spec freeze) so that I can start pushing patches early16:27
tkajinamSee https://review.opendev.org/q/project:openstack/os-traits+status:open16:28
elodilles+1 for early release o:)16:28
bauzasthat's always a good idea (to release os-traits release way earlier than milestone-3)16:28
bauzasditto with os-vif if required16:29
tkajinamok, that's nice. :-)16:29
bauzasonce the spec is approved, I don't see any reason to hold a os-traits patch16:30
bauzastkajinam: accordingly, please ping me if you want once your both specs are validated16:30
sean-k-mooneyi dont see why not16:30
bauzasI just can't remember the state of both specs :)16:31
bauzasso I'll first look whether they're accepted or not16:31
tkajinamthe SEV-ES work still needs spec approval but the other two have specs approved16:31
tkajinamI'll update the status after the meeting16:32
bauzasthanks16:32
bauzas(yeah I remembered about the firmware one)16:32
sean-k-mooneyim not sure if we need the es trait16:33
opendevreviewTakashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support  https://review.opendev.org/c/openstack/os-traits/+/90888516:33
sean-k-mooneydidnt we say we would use a seperate resouce class for that16:33
bauzasfor SEV-ES ?16:33
sean-k-mooneyyep16:33
sean-k-mooneyoh no we didnt16:34
sean-k-mooneyits the same resouce class16:34
tkajinamsean-k-mooney, we agreed using trait, not resource class to distinguish SEV and SEV-ES, IIUC16:34
sean-k-mooneyjust with a triat16:34
sean-k-mooneyit been a while since i looked16:34
tkajinamyes16:34
sean-k-mooneyya ok 16:34
bauzasyeah https://etherpad.opendev.org/p/nova-dalmatian-ptg#L49816:34
bauzas(I was looking at the etherpad)16:35
bauzaswe didn't wanted to differentiate all the SEV supports16:35
bauzass/supports/types16:35
bauzasanything else to discuss ?16:35
sean-k-mooneynot on this topic at least not form me16:36
tkajinamnothing else from me16:36
bauzassorry, I looked at other stuff16:41
bauzasthanks folks16:41
bauzas#endmeeting16:41
opendevmeetMeeting ended Tue Jul 16 16:41:23 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:41
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.html16:41
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.txt16:41
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2024/nova.2024-07-16-16.00.log.html16:41
fwieselthank you too, bye16:41
elodillesthanks o/16:41
tkajinamthanks !16:42
tkajinambauzas, if you have a moment, could you approve https://blueprints.launchpad.net/nova/+spec/libvirt-stateless-firmware ? its spec was already merged https://review.opendev.org/c/openstack/nova-specs/+/908297 16:43
tkajinamI remember we discussed the missing bp link which was fixed by https://review.opendev.org/c/openstack/nova-specs/+/920612 16:43
bauzastkajinam: as I said, I'll update Launchpad for all approved specs16:43
bauzasI was on PTO last 10 days, so I'll both check the open specs but also the new merged ones16:43
bauzas(and then update the status etherpad for adding series of the accepted specs)16:44
tkajinamah, ok. I'll recheck it after that "batch update"16:44
opendevreviewMerged openstack/os-traits master: Add traits for TPM models  https://review.opendev.org/c/openstack/os-traits/+/90910716:48
melwittthanks sean-k-mooney and bauzas for the spec reviews 🙏 19:07
*** bauzas_ is now known as bauzas19:36
*** bauzas_ is now known as bauzas20:55
*** bauzas_ is now known as bauzas22:21
*** bauzas_ is now known as bauzas23:56

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!