opendevreview | Youngjun Yoo proposed openstack/nova master: refectory: simplifying if condition expressions https://review.opendev.org/c/openstack/nova/+/912502 | 02:51 |
---|---|---|
opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/911396 | 06:10 |
opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/912511 | 08:34 |
opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/912512 | 08:34 |
opendevreview | Amit Uniyal proposed openstack/nova stable/2023.1: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/912513 | 08:34 |
*** mklejn_ is now known as mklejn | 08:55 | |
opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | 10:08 |
opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | 10:09 |
opendevreview | ribaudr proposed openstack/nova master: Amend DB model add a unique constraint. https://review.opendev.org/c/openstack/nova/+/912518 | 10:09 |
opendevreview | ribaudr 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/+/877773 | 13:58 |
opendevreview | ribaudr 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/+/903706 | 13:58 |
fwiesel | Hi, 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 |
fwiesel | ComputeManager._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 |
fwiesel | Here is the destroy: https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3695-L3701 | 15:37 |
fwiesel | Then the call to _rebuild_volume_backed_instance (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3714-L3715) | 15:37 |
fwiesel | Which in turn calls _detach_root_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3591) | 15:38 |
fwiesel | And that unsurprisingly calls driver.detach_volume (https://opendev.org/openstack/nova/src/branch/master/nova/compute/manager.py#L3556-L3557) | 15:38 |
fwiesel | I 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 |
bauzas | reminder: nova meeting in 10 mins | 15:51 |
bauzas | fwiesel: I'm currently working hard but we could discuss this in the meeting | 15:51 |
fwiesel | Sure, no hurry from my side. I am also fine doing it via the bug-report. | 15:56 |
bauzas | #startmeeting nova | 16:01 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:01 |
opendevmeet | The meeting name has been set to 'nova' | 16:01 |
elodilles | o/ | 16:01 |
bauzas | hey folks | 16:01 |
dansmith | o/ | 16:01 |
bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
fwiesel | o/ | 16:01 |
auniyal | o/ | 16:02 |
gibi | o/ | 16:02 |
bauzas | okay, let's start | 16:02 |
bauzas | #topic Bugs (stuck/critical) | 16:03 |
bauzas | #info Two Critical bugs | 16:03 |
bauzas | #link https://bugs.launchpad.net/nova/+bug/2052915 | 16:03 |
bauzas | #link https://bugs.launchpad.net/nova/+bug/2056613 | 16:03 |
bauzas | for the first one, AFAIK, we no longer have a problem | 16:03 |
bauzas | so we could close it | 16:03 |
bauzas | elodilles: ^ | 16:04 |
elodilles | well nova-grenade-multinode is not failing | 16:04 |
elodilles | but that's not neutron-ovs-grenade-* | 16:05 |
bauzas | are we running those jobs in Nova ? I don't find so | 16:06 |
elodilles | so if nova was linked due to nova-grenade-multinode then we can close | 16:06 |
elodilles | otherwise we don't even run those jobs | 16:06 |
bauzas | OK, I'll close this for nova | 16:06 |
elodilles | +1 | 16:06 |
bauzas | okay, next one https://bugs.launchpad.net/nova/+bug/2056613 | 16:07 |
bauzas | that one is not really a Critical bug | 16:08 |
bauzas | if it's because we want to make sure we can merge the changes before rc1, we'll discuss this later in the meeting | 16:09 |
bauzas | so I'll move that bug to High and we'll add that bug report to the new rc etherpad I just created | 16:09 |
gibi | If we merge it for rc1 regardless of the priorityh then Im OK to drop the priority | 16:09 |
bauzas | in general, Critical priority for bugs are for either gate failures or if the issue is related to some API that everyone uses, say an user | 16:10 |
bauzas | for example, some bug with the create API | 16:11 |
bauzas | so moving it down to High and adding it to the etherpad | 16: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 |
bauzas | I did a bit of triage for some | 16:13 |
bauzas | gibi: would you be okay to triage some bugs for next week ? | 16:14 |
gibi | I don't feel the power sorryj | 16:14 |
Uggla | o/ | 16:14 |
gibi | but you can ping me with specific questions | 16:14 |
bauzas | okay, then I'll continue to use the baton for next week | 16:15 |
bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:15 |
bauzas | #info bug baton is bauzas | 16: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-minimal | 16:16 |
bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:16 |
bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:16 |
bauzas | all periodics are green | 16:16 |
bauzas | anything people want to discuss about gate failures ? | 16:16 |
sean-k-mooney | cool including the ironic one | 16:16 |
sean-k-mooney | yep https://zuul.openstack.org/build/429f833ce05748d0a840d0c8ac18b726 | 16:17 |
bauzas | yup I was about to reply | 16:17 |
bauzas | anything else to talk about CI ? | 16:18 |
bauzas | looks not, moving on then | 16:19 |
bauzas | #topic Release Planning | 16:19 |
bauzas | some important notes | 16:19 |
bauzas | #link https://releases.openstack.org/caracal/schedule.html#nova | 16:19 |
bauzas | #info RC1 planned this week | 16:19 |
bauzas | and then, the magic etherpad I just created : | 16:20 |
bauzas | #link https://etherpad.opendev.org/p/nova-caracal-rc-potential | 16:20 |
bauzas | as you can see, there is a long list of TODOs against patches we need to merge before RC1 | 16:20 |
bauzas | that's my duty, I'll ensure either we already have them or I'll create them | 16:20 |
dansmith | um, | 16:21 |
bauzas | and then I'll ping some folks to make sure we can merge them before RC1 | 16:21 |
dansmith | not just patches to review, but patches to *write* yeah? | 16:21 |
dansmith | unless I'm looking at the wrong "long list" :) | 16:21 |
bauzas | dansmith: yup, but if they don't exist, I'll create them tonight | 16:21 |
dansmith | okay | 16:21 |
bauzas | TODOs are mostly "I don't know if takashi created them yet" | 16:21 |
bauzas | if we want to be honest :D | 16:22 |
bauzas | those are quite simple to do, we have some pattern for every of them | 16:22 |
bauzas | so, yeah, cores, please understand that if I ping you for RC1 changes, those are a bit important | 16:22 |
bauzas | about one | 16:23 |
bauzas | #link https://review.opendev.org/c/openstack/releases/+/912480 Cycle highlights | 16:23 |
gibi | feel free to ping | 16:23 |
bauzas | cores and others, feel free to review that patch today if you can | 16:23 |
bauzas | that's the cycle highlights, I'll copy them to the prelude as soon as we merge the highlights | 16:24 |
elodilles | bauzas: i can approve it, unless you want to fix the nits | 16:24 |
bauzas | elodilles: I'll do the nits | 16:24 |
elodilles | +1 | 16:24 |
elodilles | then i'll wait :] | 16:24 |
bauzas | it's quite a marketing document | 16:24 |
elodilles | indeed | 16:25 |
bauzas | that's why I'd appreciate if sean-k-mooney, dansmith, auniyal and a few others could review the highlights | 16:25 |
dansmith | yep, but it would help if you'd fix the nits so we can review the actual version proposed | 16:26 |
bauzas | cool, then I'll do it in a couple of seconds | 16:26 |
bauzas | moving | 16:26 |
sean-k-mooney | ack i can try an take a look today | 16:26 |
bauzas | #topic Review priorities | 16:26 |
bauzas | #link https://etherpad.opendev.org/p/nova-caracal-status | 16:27 |
bauzas | I'll add a link to the rc etherpad in the status etherpadf | 16:27 |
bauzas | but I think we're pretty done with that etherpad for now | 16:27 |
bauzas | I'll create another dalmatian status etherpad as soon as we branch RC1 | 16:27 |
bauzas | if people are a bit confused by the difference between etherpads and what to look, as a reminder, the rc etherpad is prioritary until RC1 | 16:28 |
bauzas | after 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 highlightsj | 16:29 | |
bauzas | don't hesitate to bug me on IRC if you are lost in translation | 16:30 |
bauzas | gibi: thanks, will rebase then | 16:30 |
Uggla | fyi 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 |
bauzas | Uggla: I just added it in the rc etherpad https://etherpad.opendev.org/p/nova-caracal-rc-potential#L29* | 16:31 |
Uggla | I understand it is also required to fix another bug with power mgmt. | 16:31 |
bauzas | moving on | 16:31 |
Uggla | bauzas, thx | 16:31 |
bauzas | #topic Stable Branches | 16:31 |
bauzas | elodilles: your time now | 16:31 |
elodilles | ACK | 16:31 |
elodilles | #info stable gates seem to be OK | 16:32 |
elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:32 |
bauzas | \o/ | 16:32 |
elodilles | and 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/victoria | 16: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 them | 16:32 |
elodilles | that's all | 16:33 |
auniyal | elodilles stable/2023.1 nova-ceph-multistore failed few times for me | 16:33 |
bauzas | auniyal: like master | 16:33 |
bauzas | elodilles: thanks | 16:33 |
elodilles | auniyal: oh, good to know. afair it sometimes fails, but not constantly | 16:34 |
auniyal | bauzas stable/2023.1 - antelope branch | 16:34 |
elodilles | but i'll keep an eye on it | 16:34 |
auniyal | ack, just want to inform | 16:35 |
elodilles | +1 | 16:35 |
bauzas | auniyal: I meant I know that nova-ceph-multistore also fails on master | 16:35 |
elodilles | thx | 16:35 |
auniyal | oh okay | 16:35 |
bauzas | anyway, cool thanks guys | 16:35 |
bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:35 |
bauzas | fwiesel: floor is yours | 16:35 |
fwiesel | #info Onboarding two colleagues to take over operations of the CI, as soon as it is "productive" | 16:35 |
bauzas | ++ | 16:35 |
fwiesel | Which 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/+/901942 | 16:36 |
fwiesel | So, 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 |
fwiesel | If that works, we will now in roughly 25minutes. | 16:37 |
fwiesel | That's it from my side. Any questions? | 16:37 |
bauzas | not from me, thanks | 16:37 |
fwiesel | bauzas: Then back to you. | 16:38 |
bauzas | ack | 16:38 |
bauzas | I'll be short | 16:38 |
bauzas | #topic Open discussion | 16:38 |
bauzas | (nothing in the agenda) | 16:38 |
bauzas | anything anyone ? | 16:38 |
bauzas | okay, then I'll give you back 20 mins | 16:39 |
bauzas | thanks all | 16:39 |
bauzas | #endmeeting | 16:39 |
opendevmeet | Meeting ended Tue Mar 12 16:39:25 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:39 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.html | 16:39 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.txt | 16:39 |
opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2024/nova.2024-03-12-16.01.log.html | 16:39 |
elodilles | thanks o/ | 16:39 |
fwiesel | thanks | 16:39 |
bauzas | elodilles: dansmith: sean-k-mooney: updated the cycle highlights | 16:39 |
dansmith | thanks | 16:39 |
* bauzas goes into gerrit queries | 16:40 | |
bauzas | owner:takashi :) | 16:40 |
gibi | bauzas: +1 | 16:41 |
bauzas | cool thanks | 16:44 |
fwiesel | bauzas: Shall I ping you better tomorrow wrt the bug report? | 16:47 |
bauzas | if you want | 16:47 |
fwiesel | Okay, will do. | 16:47 |
bauzas | until RC1, I'm a bit having a lot of things to do | 16:47 |
bauzas | but I'll see how I can help :) | 16:47 |
fwiesel | From 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 |
fwiesel | If not, that's fine. Just by the off chance :) | 16:49 |
opendevreview | Sylvain Bauza proposed openstack/nova master: doc: mark the maximum microversion for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/912616 | 18:06 |
opendevreview | Sylvain Bauza proposed openstack/nova master: Update compute rpc alias for caracal https://review.opendev.org/c/openstack/nova/+/912617 | 18:09 |
opendevreview | OpenStack Release Bot proposed openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912618 | 18:14 |
opendevreview | OpenStack 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/+/912619 | 18:14 |
opendevreview | OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912620 | 18:14 |
opendevreview | OpenStack Release Bot proposed openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912621 | 18:15 |
opendevreview | OpenStack 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/+/912622 | 18:15 |
opendevreview | OpenStack Release Bot proposed openstack/osc-placement master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912623 | 18:15 |
opendevreview | OpenStack Release Bot proposed openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912624 | 18:15 |
opendevreview | Sylvain Bauza proposed openstack/nova master: Add service version for Caracal https://review.opendev.org/c/openstack/nova/+/912625 | 18:15 |
opendevreview | OpenStack 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/+/912626 | 18:15 |
opendevreview | OpenStack Release Bot proposed openstack/python-novaclient master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912627 | 18:15 |
opendevreview | Merged openstack/python-novaclient stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912624 | 19:50 |
opendevreview | Merged openstack/osc-placement stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912621 | 19:53 |
opendevreview | Merged openstack/osc-placement stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/osc-placement/+/912622 | 19:53 |
opendevreview | Merged openstack/python-novaclient stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/python-novaclient/+/912626 | 19:56 |
opendevreview | Merged openstack/os-vif stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912618 | 21:40 |
opendevreview | Merged openstack/os-vif stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/os-vif/+/912619 | 21:56 |
opendevreview | norman shen proposed openstack/nova master: Remove an unncessary LOG https://review.opendev.org/c/openstack/nova/+/912661 | 22:23 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!