Tuesday, 2024-03-12

opendevreviewYoungjun Yoo proposed openstack/nova master: refectory: simplifying if condition expressions  https://review.opendev.org/c/openstack/nova/+/91250202:51
opendevreviewAmit Uniyal proposed openstack/nova stable/2023.2: Disconnecting volume from the compute host  https://review.opendev.org/c/openstack/nova/+/91139606:10
opendevreviewAmit Uniyal proposed openstack/nova stable/2023.1: Added context manager for instance lock  https://review.opendev.org/c/openstack/nova/+/91251108:34
opendevreviewAmit Uniyal proposed openstack/nova stable/2023.1: Separate OSError with ValueError  https://review.opendev.org/c/openstack/nova/+/91251208:34
opendevreviewAmit Uniyal proposed openstack/nova stable/2023.1: Disconnecting volume from the compute host  https://review.opendev.org/c/openstack/nova/+/91251308:34
*** mklejn_ is now known as mklejn08:55
opendevreviewribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call  https://review.opendev.org/c/openstack/nova/+/90886410:08
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction)  https://review.opendev.org/c/openstack/nova/+/83119410:08
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part)  https://review.opendev.org/c/openstack/nova/+/83309010:08
opendevreviewribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process  https://review.opendev.org/c/openstack/nova/+/88007510:08
opendevreviewribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion  https://review.opendev.org/c/openstack/nova/+/88147210:08
opendevreviewribaudr proposed openstack/nova master: Add metadata for shares  https://review.opendev.org/c/openstack/nova/+/85050010:08
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/85482310:08
opendevreviewribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/85482410:08
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/86028410:08
opendevreviewribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028510:08
opendevreviewribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares  https://review.opendev.org/c/openstack/nova/+/86028610:08
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part)  https://review.opendev.org/c/openstack/nova/+/86028710:08
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028810:08
opendevreviewribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol  https://review.opendev.org/c/openstack/nova/+/88386210:08
opendevreviewribaudr proposed openstack/nova master: Check shares support (compute manager)  https://review.opendev.org/c/openstack/nova/+/88575110:08
opendevreviewribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility  https://review.opendev.org/c/openstack/nova/+/89034010:09
opendevreviewribaudr proposed openstack/nova master: Check shares support (only API exception)  https://review.opendev.org/c/openstack/nova/+/88575210:09
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API)  https://review.opendev.org/c/openstack/nova/+/83683010:09
opendevreviewribaudr proposed openstack/nova master: Add helper methods to attach/detach shares  https://review.opendev.org/c/openstack/nova/+/88575310:09
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach notification  https://review.opendev.org/c/openstack/nova/+/85050110:09
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach notification  https://review.opendev.org/c/openstack/nova/+/85102810:09
opendevreviewribaudr proposed openstack/nova master: Add shares to InstancePayload  https://review.opendev.org/c/openstack/nova/+/85102910:09
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach_error notification  https://review.opendev.org/c/openstack/nova/+/86028210:09
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach_error notification  https://review.opendev.org/c/openstack/nova/+/86028310:09
opendevreviewribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working.  https://review.opendev.org/c/openstack/nova/+/85208610:09
opendevreviewribaudr proposed openstack/nova master: Add virt/libvirt error test cases  https://review.opendev.org/c/openstack/nova/+/85208710:09
opendevreviewribaudr proposed openstack/nova master: Docs about Manila shares API usage  https://review.opendev.org/c/openstack/nova/+/87164210:09
opendevreviewribaudr proposed openstack/nova master: Amend DB model add a unique constraint.  https://review.opendev.org/c/openstack/nova/+/91251810:09
opendevreviewribaudr proposed openstack/nova master: Fix [backport]: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly.  https://review.opendev.org/c/openstack/nova/+/87777313:58
opendevreviewribaudr proposed openstack/nova master: Fix: Live migrating to a host with cpu_shared_set configured will now update the VM's configuration accordingly.  https://review.opendev.org/c/openstack/nova/+/90370613:58
fwieselHi, regarding the bug report (https://bugs.launchpad.net/nova/+bug/2055700) (ServerActionsV293TestJSON.test_rebuild_volume_backed_server). I think, I do now why I get the exception. What I don't know is, how that is ever suppose to work with other hypervisors.15:34
fwieselComputeManager._rebuild_default_impl is calling driver.destroy on the instance, and *then* calls a volume detach. That raises an InstanceNotFound in the vmwareapi driver, and I don't know how it should not.15:36
fwieselHere is the destroy: https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3695-L370115:37
fwieselThen the call to _rebuild_volume_backed_instance (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3714-L3715)15:37
fwieselWhich in turn calls _detach_root_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3591)15:38
fwieselAnd that unsurprisingly calls driver.detach_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3556-L3557)15:38
fwieselI am unclear on how that is supposed to work. If you destroy an instance, my expectation would be that the driver does report InstanceNotFound when you do further operations on it. Apparently that is not the case with libvirt. What path will get triggered there?15:49
bauzasreminder: nova meeting in 10 mins15:51
bauzasfwiesel: I'm currently working hard but we could discuss this in the meeting15:51
fwieselSure, no hurry from my side. I am also fine doing it via the bug-report.15:56
bauzas#startmeeting nova16:01
opendevmeetMeeting started Tue Mar 12 16:01:09 2024 UTC and is due to finish in 60 minutes.  The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot.16:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:01
opendevmeetThe meeting name has been set to 'nova'16:01
elodilleso/16:01
bauzashey folks16:01
dansmitho/16:01
bauzas#link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting16:01
fwieselo/16:01
auniyalo/16:02
gibio/16:02
bauzasokay, let's start16:02
bauzas#topic Bugs (stuck/critical) 16:03
bauzas#info Two Critical bugs16:03
bauzas#link https://bugs.launchpad.net/nova/+bug/205291516:03
bauzas#link https://bugs.launchpad.net/nova/+bug/205661316:03
bauzasfor the first one, AFAIK, we no longer have a problem16:03
bauzasso we could close it16:03
bauzaselodilles: ^16:04
elodilleswell nova-grenade-multinode is not failing16:04
elodillesbut that's not neutron-ovs-grenade-*16:05
bauzasare we running those jobs in Nova ? I don't find so16:06
elodillesso if nova was linked due to nova-grenade-multinode then we can close16:06
elodillesotherwise we don't even run those jobs16:06
bauzasOK, I'll close this for nova16:06
elodilles+116:06
bauzasokay, next one https://bugs.launchpad.net/nova/+bug/205661316:07
bauzasthat one is not really a Critical bug 16:08
bauzasif it's because we want to make sure we can merge the changes before rc1, we'll discuss this later in the meeting16:09
bauzasso I'll move that bug to High and we'll add that bug report to the new rc etherpad I just created16:09
gibiIf we merge it for rc1 regardless of the priorityh then Im OK to drop the priority16:09
bauzasin general, Critical priority for bugs are for either gate failures or if the issue is related to some API that everyone uses, say an user16:10
bauzasfor example, some bug with the create API16:11
bauzasso moving it down to High and adding it to the etherpad16:11
bauzas#link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 65 new untriaged bugs (-10 since the last meeting)16:13
bauzasI did a bit of triage for some 16:13
bauzasgibi: would you be okay to triage some bugs for next week ?16:14
gibiI don't feel the power sorryj16:14
Ugglao/16:14
gibibut you can ping me with specific questions16:14
bauzasokay, then I'll continue to use the baton for next week16:15
bauzas#info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster16:15
bauzas#info bug baton is bauzas16:15
bauzas#topic Gate status 16:16
bauzas#link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs 16:16
bauzas#link https://etherpad.opendev.org/p/nova-ci-failures-minimal16:16
bauzas#link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status16:16
bauzas#info Please look at the gate failures and file a bug report with the gate-failure tag.16:16
bauzasall periodics are green16:16
bauzasanything people want to discuss about gate failures ?16:16
sean-k-mooneycool including the ironic one16:16
sean-k-mooney yep https://zuul.openstack.org/build/429f833ce05748d0a840d0c8ac18b72616:17
bauzasyup I was about to reply16:17
bauzasanything else to talk about CI ?16:18
bauzaslooks not, moving on then16:19
bauzas#topic Release Planning 16:19
bauzassome important notes16:19
bauzas#link https://releases.openstack.org/caracal/schedule.html#nova16:19
bauzas#info RC1 planned this week16:19
bauzasand then, the magic etherpad I just created :16:20
bauzas#link https://etherpad.opendev.org/p/nova-caracal-rc-potential16:20
bauzasas you can see, there is a long list of TODOs against patches we need to merge before RC116:20
bauzasthat's my duty, I'll ensure either we already have them or I'll create them16:20
dansmith um,16:21
bauzasand then I'll ping some folks to make sure we can merge them before RC116:21
dansmithnot just patches to review, but patches to *write* yeah?16:21
dansmithunless I'm looking at the wrong "long list" :)16:21
bauzasdansmith: yup, but if they don't exist, I'll create them tonight16:21
dansmithokay16:21
bauzasTODOs are mostly "I don't know if takashi created them yet"16:21
bauzasif we want to be honest :D16:22
bauzasthose are quite simple to do, we have some pattern for every of them16:22
bauzasso, yeah, cores, please understand that if I ping you for RC1 changes, those are a bit important16:22
bauzasabout one16:23
bauzas#link https://review.opendev.org/c/openstack/releases/+/912480 Cycle highlights16:23
gibifeel free to ping16:23
bauzascores and others, feel free to review that patch today if you can16:23
bauzasthat's the cycle highlights, I'll copy them to the prelude as soon as we merge the highlights16:24
elodillesbauzas: i can approve it, unless you want to fix the nits16:24
bauzaselodilles: I'll do the nits16:24
elodilles+116:24
elodillesthen i'll wait :]16:24
bauzasit's quite a marketing document16:24
elodillesindeed16:25
bauzasthat's why I'd appreciate if sean-k-mooney, dansmith, auniyal and a few others could review the highlights16:25
dansmithyep, but it would help if you'd fix the nits so we can review the actual version proposed16:26
bauzascool, then I'll do it in a couple of seconds16:26
bauzasmoving16:26
sean-k-mooneyack i can try an take a look today16:26
bauzas#topic Review priorities 16:26
bauzas#link https://etherpad.opendev.org/p/nova-caracal-status16:27
bauzasI'll add a link to the rc etherpad in the status etherpadf16:27
bauzasbut I think we're pretty done with that etherpad for now16:27
bauzasI'll create another dalmatian status etherpad as soon as we branch RC116:27
bauzasif people are a bit confused by the difference between etherpads and what to look, as a reminder, the rc etherpad is prioritary until RC116:28
bauzasafter RC1, we'll have a dalmatian status etherpad for tracking bugs and blueprints we approve for D, while we'll continue to have a RC etherpad for tracking Caracal RC2 and later (if we need so)16:29
* gibi left two nits in the highlightsj16:29
bauzasdon't hesitate to bug me on IRC if you are lost in translation16:30
bauzasgibi: thanks, will rebase then16:30
Ugglafyi I have updated https://review.opendev.org/c/openstack/nova/+/877773/9, I think sean-k-mooney will review it, it will be great to have another core looking at it.16:30
bauzasUggla: I just added it in the rc etherpad https://etherpad.opendev.org/p/nova-caracal-rc-potential#L29*16:31
UgglaI understand it is also required to fix another bug with power mgmt.16:31
bauzasmoving on16:31
Ugglabauzas, thx16:31
bauzas#topic Stable Branches 16:31
bauzaselodilles: your time now16:31
elodillesACK16:31
elodilles#info stable gates seem to be OK16:32
elodilles#info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci16:32
bauzas\o/16:32
elodillesand maybe these two more sentence:16:32
elodilles#info stable/yoga, stable/xena, stable/wallaby, stable/victoria branches are now replaced by unmaintained/yoga, unmaintained/xena, unmaintained/wallaby, unmaintained/victoria16:32
elodilles#info patches proposed on stable/yoga..victoria branches now abandoned, cherry pick them to unmaintained/yoga..victoria if you want to work on them16:32
elodillesthat's all16:33
auniyalelodilles stable/2023.1 nova-ceph-multistore failed few times for me16:33
bauzasauniyal: like master16:33
bauzaselodilles: thanks16:33
elodillesauniyal: oh, good to know. afair it sometimes fails, but not constantly16:34
auniyalbauzas stable/2023.1 - antelope branch16:34
elodillesbut i'll keep an eye on it16:34
auniyalack, just want to inform16:35
elodilles+116:35
bauzasauniyal: I meant I know that nova-ceph-multistore also fails on master16:35
elodillesthx16:35
auniyaloh okay16:35
bauzasanyway, cool thanks guys16:35
bauzas#topic vmwareapi 3rd-party CI efforts Highlights 16:35
bauzasfwiesel: floor is yours16:35
fwiesel#info Onboarding two colleagues to take over operations of the CI, as soon as it is "productive"16:35
bauzas++16:35
fwieselWhich is hopefully rather sooner than later, still trying to find out why the FIPS are unreliable.16:36
fwiesel#info Tested automated commenting on https://review.opendev.org/c/opendev/ci-sandbox/+/90194216:36
fwieselSo, in principle, the CI should be able to comment on the change requests.16:36
fwiesel#info Enabled commenting also on openstack/nova just before the meeting. With some luck, it works.16:36
fwieselIf that works, we will now in roughly 25minutes. 16:37
fwieselThat's it from my side. Any questions?16:37
bauzasnot from me, thanks16:37
fwieselbauzas: Then back to you.16:38
bauzasack16:38
bauzasI'll be short16:38
bauzas#topic Open discussion 16:38
bauzas(nothing in the agenda)16:38
bauzasanything anyone ?16:38
bauzasokay, then I'll give you back 20 mins16:39
bauzasthanks all16:39
bauzas#endmeeting16:39
opendevmeetMeeting ended Tue Mar 12 16:39:25 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:39
opendevmeetMinutes:        https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.html16:39
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.txt16:39
opendevmeetLog:            https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.log.html16:39
elodillesthanks o/16:39
fwieselthanks16:39
bauzaselodilles: dansmith: sean-k-mooney: updated the cycle highlights16:39
dansmiththanks16:39
* bauzas goes into gerrit queries 16:40
bauzasowner:takashi :)16:40
gibibauzas: +116:41
bauzascool thanks16:44
fwieselbauzas: Shall I ping you better tomorrow wrt the bug report?16:47
bauzasif you want16:47
fwieselOkay, will do.16:47
bauzasuntil RC1, I'm a bit having a lot of things to do16:47
bauzasbut I'll see how I can help :)16:47
fwieselFrom the top of your head, do you happen to know what will happen on the libvirt driver, when you call driver.detach_volume after having called driver.destroy?16:48
fwieselIf not, that's fine. Just by the off chance :)16:49
opendevreviewSylvain Bauza proposed openstack/nova master: doc: mark the maximum microversion for 2024.1 Caracal  https://review.opendev.org/c/openstack/nova/+/91261618:06
opendevreviewSylvain Bauza proposed openstack/nova master: Update compute rpc alias for caracal  https://review.opendev.org/c/openstack/nova/+/91261718:09
opendevreviewOpenStack Release Bot proposed openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/os-vif/+/91261818:14
opendevreviewOpenStack Release Bot proposed openstack/os-vif stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/os-vif/+/91261918:14
opendevreviewOpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2024.1  https://review.opendev.org/c/openstack/os-vif/+/91262018:14
opendevreviewOpenStack Release Bot proposed openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/osc-placement/+/91262118:15
opendevreviewOpenStack Release Bot proposed openstack/osc-placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/osc-placement/+/91262218:15
opendevreviewOpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/2024.1  https://review.opendev.org/c/openstack/osc-placement/+/91262318:15
opendevreviewOpenStack Release Bot proposed openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/python-novaclient/+/91262418:15
opendevreviewSylvain Bauza proposed openstack/nova master: Add service version for Caracal  https://review.opendev.org/c/openstack/nova/+/91262518:15
opendevreviewOpenStack Release Bot proposed openstack/python-novaclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/python-novaclient/+/91262618:15
opendevreviewOpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/2024.1  https://review.opendev.org/c/openstack/python-novaclient/+/91262718:15
opendevreviewMerged openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/python-novaclient/+/91262419:50
opendevreviewMerged openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/osc-placement/+/91262119:53
opendevreviewMerged openstack/osc-placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/osc-placement/+/91262219:53
opendevreviewMerged openstack/python-novaclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/python-novaclient/+/91262619:56
opendevreviewMerged openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1  https://review.opendev.org/c/openstack/os-vif/+/91261821:40
opendevreviewMerged openstack/os-vif stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1  https://review.opendev.org/c/openstack/os-vif/+/91261921:56
opendevreviewnorman shen proposed openstack/nova master: Remove an unncessary LOG  https://review.opendev.org/c/openstack/nova/+/91266122:23

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