Tuesday, 2023-11-14

opendevreviewmelanie witt proposed openstack/nova stable/zed: Fix exception catch when volume mount fails  https://review.opendev.org/c/openstack/nova/+/90083501:13
opendevreviewmelanie witt proposed openstack/nova stable/yoga: Fix exception catch when volume mount fails  https://review.opendev.org/c/openstack/nova/+/90083601:14
opendevreviewmelanie witt proposed openstack/nova stable/xena: Fix exception catch when volume mount fails  https://review.opendev.org/c/openstack/nova/+/90083701:14
opendevreviewmelanie witt proposed openstack/nova stable/wallaby: Fix exception catch when volume mount fails  https://review.opendev.org/c/openstack/nova/+/90083801:15
opendevreviewmelanie witt proposed openstack/nova stable/2023.2: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084001:37
opendevreviewmelanie witt proposed openstack/nova stable/2023.1: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084101:38
opendevreviewmelanie witt proposed openstack/nova stable/zed: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084201:39
opendevreviewmelanie witt proposed openstack/nova stable/yoga: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084301:40
opendevreviewmelanie witt proposed openstack/nova stable/yoga: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084301:40
opendevreviewmelanie witt proposed openstack/nova stable/xena: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084401:40
opendevreviewmelanie witt proposed openstack/nova stable/wallaby: Improve logging at '_numa_cells_support_network_metadata'  https://review.opendev.org/c/openstack/nova/+/90084501:41
hamidlotfi_Good morning.05:41
hamidlotfi_I want to remove a controller node from the OpenStack cluster, how do that?05:43
hamidlotfi_What are the steps to do it? I didn't see anything in the documents.05:44
hamidlotfi_I am using the zed/26.1 version.05:47
hamidlotfi_@noonedeadpunk @jrosser  06:25
artGood morning06:39
hamidlotfi_Sorry, I posted this message here by mistake.06:43
opendevreviewAmit Uniyal proposed openstack/nova-specs master: Enforce console session timeout  https://review.opendev.org/c/openstack/nova-specs/+/89855309:27
bauzasreminder: nova meeting in 47 mins here15:13
opendevreviewDan Smith proposed openstack/nova master: Support setting alias on libvirt disks  https://review.opendev.org/c/openstack/nova/+/89280015:43
opendevreviewDan Smith proposed openstack/nova master: Set libvirt device alias for volumes  https://review.opendev.org/c/openstack/nova/+/89280115:43
opendevreviewDan Smith proposed openstack/nova master: Detach disks using alias when possible  https://review.opendev.org/c/openstack/nova/+/89306815:43
bauzas#startmeeting nova16:00
opendevmeetMeeting started Tue Nov 14 16:00:26 2023 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
elodilleso/16:00
bauzasI was about to write 'startmeeting nvidia'16:00
elodilles:S16:00
bauzasETOOMANYTHINGS16:00
dvo-plvo/16:00
elodillessomeone spend too much time with nvidia things? :-o16:01
bauzas#link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting16:01
bauzaselodilles: probably me, but who guesses ?16:01
elodilles:)16:01
gibio/16:02
dansmitho/16:02
bauzaslet's start16:02
bauzasand let's be quick16:02
bauzas#topic Bugs (stuck/critical) 16:02
bauzas#info No Critical bug16:02
bauzas#link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 32 new untriaged bugs (+0 since the last meeting)16: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
bauzaselodilles: anything you want to tell about bugs ?16:02
elodilles(damn, it was 31 one hr ago..)16:03
elodillesnothing to tell16:03
bauzascool16:03
elodilleshad little time to deal with bugs :/16:03
bauzasauniyal6: want to be the next for triaging ?16:03
bauzasokay, given his TZ, he's probably off16:04
bauzasno worries at all, will discuss with him later tomorrow16:04
bauzas#info bug baton may be auniyal16:05
bauzas#topic Gate status 16:05
bauzas#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:05
bauzas#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status16:05
bauzasperiodics are green16:06
bauzas#info Please look at the gate failures and file a bug report with the gate-failure tag.16:06
bauzasdansmith: what's up with the grenade vnc failure ?16:06
dansmithlong since fixed?16:06
dansmithbecause melwitt 16:06
bauzas\o/16:06
bauzasokay, so, no other gate issues we just discovered this week ?16:06
tkajinamo/16:07
tkajinamthis is not new... but as you may now the earlier issue with sqlalchemy 2.0 job has been fixed and it should be green now16:07
tkajinamso we can merge https://review.opendev.org/c/openstack/nova/+/900538 to make the job voting again16:07
bauzastkajinam: ahah, good time for telling this, we're on a meeting and we discuss about CI failures :)16:08
tkajinam(which is waiting for 2nd +216:08
tkajinamyeah I saw it and that's why I jumped in here ;-)16:08
bauzastkajinam: sent to the gate16:08
tkajinamthanks !16:08
bauzastkajinam: perfect timing ! ;-)16:09
bauzasanyway, anything else to say about failures ?16:09
bauzaslooks not16:09
bauzas#topic Release Planning 16:09
bauzas#link https://releases.openstack.org/caracal/schedule.html16:09
bauzas#info Nova deadlines will be proposed in the schedule above16:09
bauzas... once I have time :)16:09
bauzas#info Caracal-1 milestone this week16:09
bauzasI also need to look at my gerrit emails :)16:10
bauzaselodilles: have we already a release change for it ?16:10
* bauzas goes looking at Gmail16:10
elodilleschanges were generated yes16:10
elodilleslet me look16:10
bauzasos-vif is proposed https://review.opendev.org/c/openstack/releases/+/900762?usp=email16:11
elodillesand as i see there's no nova related caracal-1 release16:11
bauzasbut I don't see nova in my mailbox16:11
bauzaselodilles: cool, will check emails later16:11
bauzasso, something else16:12
bauzasI wrote 2 weeks before that today is an implementation review day16:12
bauzasof course, I forgot about it16:12
elodillesbauzas: btw, you told some time ago that we could release beta/milestone releases off nova. do you want me to propose a patch for that?16:12
bauzasso, would people agree with us having an implementation review day tomorrow ?16:12
dansmithbauzas: ++16:13
bauzaselodilles: what have I said ?16:13
elodilles:)16:13
bauzasI mean, I can surely propose a m-1 release change if nobody does this16:14
elodillesfor cycle-with-rc (like nova) release team does not generate release patches16:14
bauzasthat said, I don't know how many people use that tag tbh16:14
bauzasdoh my bad16:14
bauzasthen I'll do my duty16:14
elodillesbauzas: i'm just asking :) maybe i misheard something :)16:15
bauzasI honestly can't remember anything, apart the fact I think noone is using milestone-1 tag16:15
bauzasmilestone-3 for sure16:15
bauzasbut the first two ones are just there for nothing :)16:16
elodillesOK, then i'll wait for Caracal-3 :)16:16
elodillesACK :)16:16
elodillessorry for the noise :X16:16
bauzasnah, I'll create caracal-1 and caracal-2 because creating a tag named "caracal-3" alone would be super bizarre, right ? :D16:16
bauzaswe're not Microsot16:17
bauzasMicrosoft*16:17
bauzasanyway16:17
bauzas#action bauzas to propose caracal-1 tag16:17
bauzas#undo16:17
opendevmeetRemoving item from minutes: #action bauzas to propose caracal-1 tag16:17
bauzas#action bauzas to propose caracal-1 tag for nova and placement16:17
elodilles(it won't be caracal-3, it'll be 29.0.0.0b1 ;))16:17
bauzasI'm lost and don't have a lot of time, but I trust you16:18
bauzasanyway, thanks for asking16:18
elodillesnp16:18
bauzasso, back on track16:18
bauzasI heard dansmith saying yay for tomorrow being implementation review day16:19
bauzaswho else nays ?16:19
gibiyay from me16:19
gibiat least I can add implementation to review :)16:20
bauzascool16:20
dansmithheh16:20
gibihttps://review.opendev.org/q/topic:bp%252Flibvirt-migrate-with-hostname-instead-of-ip16:20
dansmithI admit I'm biased since I just pushed an implementation _to_ review :P16:20
bauzas#agreed Tomorrow (Nov 15) will be implementation review day16:20
gibisame here :)16:20
bauzas#action bauzas to create an etherpad for tracking features and bugs like we agreed at PTG, so it would be used tomorrow16:21
sean-k-mooneywell any review that are rety can be reviewd including they ones you just pushed so16:21
bauzasI too much punted that action item16:21
bauzas#action bauzas to notify by email about both the implementation review day and the etherpad to use16:22
bauzasthe email will be late today for sure16:22
bauzas#topic Review priorities 16:23
bauzas#action bauzas need to create the etherpad he just said above16:23
bauzas#topic Stable Branches 16:23
bauzaselodilles: I'm all ears16:23
elodilleso716:23
elodilles#info stable gates don't seem blocked16:24
elodillesi've seen some failing jobs but nothing constantly failing...16:24
elodilles(or I'm not aware of it)16:24
elodilles#info stable release patches still open for review: https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova16:24
elodilles+1: yoga is going to be unmaintained, so final stable/yoga release should happen ASAP - https://etherpad.opendev.org/p/nova-stable-yoga-eom16:24
elodillesif we want to do a final release at all16:25
bauzasthat's a need16:26
elodillesotherwise we can just live with the last release from summer(?)16:26
bauzaselodilles: nah, I'd prefer we do some release16:26
bauzaselodilles: bug me this week or next week16:26
elodillesnow we have 1 merged patch as can be seen in the etherpad16:26
bauzasI guess you know that I'm busy, so I'll forget if nobody harasses me16:26
elodillesbauzas: i'll ping you then some time16:27
bauzasthanks16:27
elodillesthe open patches are also listed on the etherpad16:27
bauzasI'm unfortunately not very good at doing multiple things at the same time16:27
elodillesfeel free to use the pad, edit it if you want16:27
bauzaslike, my mind is not good for concurrency16:27
elodillesi'll try to add details to the pad so that will be easier to reviewers to look at 'things-to-be-reviewed'16:28
bauzasack16:28
elodillesanyway, last but not least:16:29
elodilles#info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci16:29
elodillesadd stable gate issues here if you find one ^^^16:29
elodillesand that was all from me16:29
bauzascool16:30
bauzaslast topic16:30
bauzas#topic open discussion16:30
bauzasanything anyone ?16:30
dvo-plvo/ In the open discussion topic I would like to remind about ability to reviwe this cool spec file:) https://review.opendev.org/c/openstack/nova-specs/+/895924/816:30
dvo-plvSorry, I forgot to fill agenda with this 16:30
sean-k-mooneyi have two related things we we can talk about after ^16:30
sean-k-mooneydvo-plv: if dansmith and bauzas feel stronly that the tait is not required and want to use a min compute service check instead i can relent on that16:31
sean-k-mooneybut you will need to modify the spec to say that the feature will only work when its fully upgraded16:32
sean-k-mooneybauzas: dansmith ^ is that still your prefernce16:32
dansmithI have never been super opinionated about it.. I'm pretty meh all around16:32
dvo-plv i beleive that we alreadt agreed with trait usage16:32
dansmithI still feel like a trait is just too much work for such a thing16:32
dvo-plvI added upgrade impact https://review.opendev.org/c/openstack/nova-specs/+/895924/8/specs/2024.1/approved/virtio_packedring_configuration_support.rst#12616:33
sean-k-mooneydvo-plv: is this what you wanted to discuss by the way16:33
sean-k-mooneyi want to make sure we have a path to unblocking you16:33
dvo-plvyes, I also want a path to unblock this feature and i thought that nova core already agreed with that16:34
dvo-plvso I've updated spec and nova code16:34
sean-k-mooneyim fine with the trait i assuemd that was the only open16:34
sean-k-mooneyi can re review the new version of the spec16:34
bauzasI think I said loudly a couple of times that I eventually give up16:35
bauzasso, I let other cores accept using traits16:35
bauzasmy only concern is that I just want to make sure nobody would use that trait in a flavor so we could eventually remove it if we want, but given I won't review the spec, I leave others to say what they want16:36
dvo-plvI suggest to add thi item somewhere regarding trait usage16:36
dvo-plvbut it does not required as far as it will assigned automatically16:37
dvo-plvhttps://review.opendev.org/c/openstack/nova/+/876075/27/nova/scheduler/request_filter.py#275 here for instance16:37
dvo-plvhttps://review.opendev.org/c/openstack/nova/+/876075/27/nova/virt/libvirt/driver.py#12429 here for node16:38
sean-k-mooneyso can i suggest we take a look at this (the spec and the implematnion) tomorrow n the implemation review day16:40
sean-k-mooneyfor those that are interested16:40
bauzasI am not but that's a good idea16:40
bauzasanyway, can we end the meeting now ?16:41
sean-k-mooneyi would like to add 2 thigns quickly16:41
sean-k-mooneyhttps://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver and https://blueprints.launchpad.net/nova/+spec/remove-vmware-support16:41
sean-k-mooneyare we ok to appove those or would you like use to expeand the description with more details16:42
bauzasI think gmann wrote specs16:42
sean-k-mooneyyes which they abandoned 16:42
bauzasokay16:42
sean-k-mooneybecause multiple of us said it was not requried16:42
bauzasso we need to review both blueprints in a meeting16:42
bauzasyeah, including me16:43
dvo-plvThank you16:43
sean-k-mooneyso i just want to decied on a path forward for the paper work16:43
bauzasbut I don't wanna force someone to not write a spec if he/she really wants to :)16:43
sean-k-mooneyif we want to review them next week we can16:43
sean-k-mooneybut i would like to merge the first patches for the removal sooner rather then later16:43
bauzassure16:43
bauzasand sure16:43
sean-k-mooneybauzas: i was orgianlly hoping ot merge the vmware removal patch tomorrow in the implemation day16:43
sean-k-mooneybut it can wait16:44
bauzasthen, we can approve those today16:44
bauzasfolks,16:44
bauzasanyone having concerns with https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver as a specless BP for one ?16:44
sean-k-mooneyim fine with that. for the vmware one i do have more patches ill need to write to complete the removal but the inital patch https://review.opendev.org/c/openstack/nova/+/897017 is the entirty of the driver removal. i need to remove the console and objects in follow up patches16:46
bauzasokay, so16:46
bauzas#agreed https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver to be accepted as a specless bp16:46
bauzasnext one16:46
bauzasanyone having concerns with https://blueprints.launchpad.net/nova/+spec/remove-vmware-support as a specless BP ?16:47
bauzassean-k-mooney: the fact that you have left things to implement is just some implementation nit16:47
bauzaswe can still approve the bp16:48
sean-k-mooneyyep 16:48
bauzaslike we did with nova-net16:48
sean-k-mooneyjust wanted to clarify that since we expanded the scope at the ptg16:48
bauzasanyone having concerns again ?16:48
bauzasokay so16:48
bauzas#agreed https://blueprints.launchpad.net/nova/+spec/remove-vmware-support to be accepted as a specless BP16:48
bauzasthere we go16:48
bauzassean-k-mooney: I'll just set the LP fields right after the meeting16:49
sean-k-mooneythanks16:49
sean-k-mooneythats all from me16:49
bauzasand we can start to review your patches tomorrow16:49
bauzascool16:49
bauzasso, let me call it a wrap16:49
bauzasthanks all16:49
bauzas#endmeeting16:49
opendevmeetMeeting ended Tue Nov 14 16:49:45 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:49
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2023/nova.2023-11-14-16.00.html16:49
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-11-14-16.00.txt16:49
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2023/nova.2023-11-14-16.00.log.html16:49
dvo-plvsean-k-mooney, How we can attend tomorrow review. Does it meeting or what ?16:50
sean-k-mooneyno its just that tommrrow we are goign to make time to reveiw via gerrit16:51
sean-k-mooneywe will be disucssing things here 16:51
sean-k-mooneyon irc too as needed16:51
bauzassean-k-mooney: paperwork stamps dully noted.16:52
dvo-plvI see ,okay. Feel free to ping me, thank you16:52
bauzas(on both BPs)16:52
elodillesbauzas: minor clarification: pyhton-novaclient and osc-placement had no relevant changes i think, that's why we only have os-vif Caracal-1 proposed. nova and placement are services, so they don't need Caracal-1 release, hence release patches are not generated for them. (though we could release .0b1 releases, but that is unnecessary as we discussed)16:52
bauzasack16:52
gmannbauzas: yeah, abandon the hyperV removal spec. i proposed it before our PTG discussion. 17:03
bauzasdansmith: can't find the procedural blueprint for https://blueprints.launchpad.net/nova/+spec/libvirt-dev-alias 17:39
bauzasfancy creating it ?17:39
bauzashttps://specs.openstack.org/openstack/nova-specs/specs/2024.1/approved/libvirt-dev-alias.html links to it17:39
dansmithoh I probably never created it yeah17:40
dansmithbauzas: done17:41
bauzascool17:41
bauzasokay, https://blueprints.launchpad.net/nova/2024.1 reflects what we have17:43
bauzas(except john's BP which isn't approved yet)17:43
bauzasfolks, etherpad just created https://etherpad.opendev.org/p/nova-caracal-status17:44
bauzasI'll curate the list of blueprints to find the relevant gerrit series17:44
bauzasbut you should be able to look at it tomorrow17:44
sean-k-mooneycool thanks bauzas 17:46
opendevreviewSylvain Bauza proposed openstack/nova master: WIP: libvirt: Cap with max_instances GPU types  https://review.opendev.org/c/openstack/nova/+/89962518:01
bauzasdansmith: sean-k-mooney: squashed my SRIOV patches into a single one18:01
bauzaslike dansmith said to me ;)18:02
bauzasoperators would then provide the max instances and then restarting nova-compute so the virt driver would delete the ResourceProviders18:02
sean-k-mooneybauzas: i looked at those previosuly but i dont think i found where dansmith said that18:03
sean-k-mooneyso i was waiting for an update before looking again18:04
bauzassean-k-mooney: we discussed this in a gmeet18:09
bauzashence why no comments18:09
opendevreviewDan Smith proposed openstack/nova master: Detach disks using alias when possible  https://review.opendev.org/c/openstack/nova/+/89306818:18
opendevreviewDan Smith proposed openstack/nova master: DNM: Debug some volumes not setting alias  https://review.opendev.org/c/openstack/nova/+/90093518:18
opendevreviewMerged openstack/nova master: Make sqlalchemy-2x job voting again  https://review.opendev.org/c/openstack/nova/+/90053818:24
opendevreviewDan Smith proposed openstack/nova master: DNM: Debug some volumes not setting alias  https://review.opendev.org/c/openstack/nova/+/90093520:52
opendevreviewmelanie witt proposed openstack/nova master: Add encryption support to convert_image  https://review.opendev.org/c/openstack/nova/+/87093422:34

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