| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-26 | |||
| 19:47:56 | opendevreview | Dan Smith proposed openstack/nova master: Detect host renames and abort startup https://review.opendev.org/c/openstack/nova/+/863920 | |
| 19:47:56 | opendevreview | Dan Smith proposed openstack/nova master: Fixup patch for stable-compute-uuid series https://review.opendev.org/c/openstack/nova/+/871848 | |
| 21:50:38 | sean-k-mooney[m] | dansmith: i did a first pass i have one or two question later in the series the first 3 should be in the gate | |
| 21:51:02 | dansmith | sean-k-mooney: ack, thanks, I saw the first three, I'll look at the others in a bit or tomorrow | |
| 22:30:21 | gmann | gibi: updated the placement RBAC change https://review.opendev.org/c/openstack/placement/+/865618 | |
| 22:48:45 | opendevreview | Merged openstack/nova master: Add virt/node module for stable uuids https://review.opendev.org/c/openstack/nova/+/863915 | |
| 22:48:53 | opendevreview | Merged openstack/nova master: Pass service ref to init_host(), if exists https://review.opendev.org/c/openstack/nova/+/863916 | |
| #openstack-nova - 2023-01-27 | |||
| 04:41:25 | opendevreview | Ghanshyam proposed openstack/nova stable/xena: DNM: testing tempest pin for stable/wallaby https://review.opendev.org/c/openstack/nova/+/871800 | |
| 04:51:43 | gmann | gibi: bauzas: updates on stable/wallaby gate: I have updated the devstack patch (depends-on) and it fixes the gate - https://review.opendev.org/c/openstack/nova/+/871798 | |
| 04:52:38 | gmann | gibi: bauzas: but it unhide another bug in devstack/grenade side due to which stable/xena grenade job start failing (with devstack stable/wallaby tempest pin) - https://bugs.launchpad.net/grenade/+bug/2003993 | |
| 04:53:18 | gmann | I have proposed the fix https://review.opendev.org/q/I5e938139b47f443a4c358415d0d4dcf6549cd085 and testing it in https://review.opendev.org/c/openstack/nova/+/871800 | |
| 05:58:02 | opendevreview | Ghanshyam proposed openstack/nova stable/xena: DNM: testing tempest pin for stable/wallaby https://review.opendev.org/c/openstack/nova/+/871800 | |
| 07:10:56 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 07:10:57 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 07:11:09 | sahid | ^ fixed merge conflict | |
| 07:11:10 | sahid | o/ | |
| 09:01:38 | bauzas | sahid: I have a busy morning but I'll try to take a look | |
| 09:01:57 | bauzas | gmann: ack, thanks for the heads-up | |
| 09:02:26 | priteau | Would the nova team consider disabling the failing grenade job on stable/wallaby temporarily to be able to merge the VMDK patch? | |
| 09:02:45 | bauzas | priteau: that's one of the options in the table | |
| 09:03:45 | bauzas | but before doing it, I need to correctly understand the problem | |
| 09:40:54 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/zed: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/871968 | |
| 09:45:14 | gibi | gmann: those fixes looks good to me. thanks for proposing them | |
| 09:46:11 | kashyap | Hm, /me lost track of these upstream backports from last July still :-( -- https://review.opendev.org/q/topic:bug%252F1982853 | |
| 09:54:35 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/yoga: Add a workaround to skip hypervisor version check on LM https://review.opendev.org/c/openstack/nova/+/851202 | |
| 09:56:15 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/yoga: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/871969 | |
| 09:57:57 | kashyap | elodilles: bauzas: This backport has been waiting for a while, can this be put through? - https://review.opendev.org/c/openstack/nova/+/851205 | |
| 10:02:42 | bauzas | kashyap: done | |
| 10:02:49 | kashyap | Thx! | |
| 10:09:01 | elodilles | it won't merge, as the yoga patch has not been merged yet: https://review.opendev.org/c/openstack/nova/+/851202 | |
| 10:09:12 | elodilles | bauzas: ^^^ | |
| 10:09:48 | bauzas | voila why I didn't +W before | |
| 10:09:53 | sahid | bauzas: no worries thank you for your time! | |
| 10:09:57 | bauzas | -ETOOMANYREVIEWSONFLY | |
| 10:11:50 | elodilles | (as I see you +W'd it once, but the patch wasn't cherry picked from the latest, merged PS) | |
| 10:12:56 | kashyap | elodilles: Oh, yeah; the Yoga one is still waiting. And are you saying the Xena cherry-pick is not correct? | |
| 10:13:15 | kashyap | Ah, you were talking about the _past_ ("wasn't"). Now it should be fine | |
| 10:21:13 | elodilles | kashyap: the cherry-pick needs to be done again from yoga patch to stable/xena | |
| 10:21:30 | kashyap | Duh, I thought I just did it ... /me face-palms and looks | |
| 10:21:44 | elodilles | (and the current xena patch won't merge as it is not cherry picked from the latest yoga PS) | |
| 10:22:11 | elodilles | kashyap: thx for fixing it | |
| 10:23:51 | kashyap | elodilles: Gonna cherry-pick from this Yoga commit to Xena: c07495d9d64dd0635d72fc7ff67d73a656a40d13 | |
| 10:24:55 | elodilles | kashyap: yepp, that is the hash of the latest PS | |
| 10:29:04 | kashyap | elodilles: Hmm, I also need to backport another one before that (https://review.opendev.org/c/openstack/nova/+/845045) | |
| 10:29:14 | kashyap | For Xena, i.e.; /me goes to do it | |
| 10:31:09 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/xena: Add a workaround to skip hypervisor version check on LM https://review.opendev.org/c/openstack/nova/+/851205 | |
| 10:31:10 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/xena: libvirt: Add a workaround to skip compareCPU() on destination https://review.opendev.org/c/openstack/nova/+/871975 | |
| 10:31:14 | kashyap | elodilles: Hope that looks better --^ | |
| 10:33:07 | elodilles | kashyap: yepp, looks good, that should be accepted by the backport validator job as well | |
| 10:45:01 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Dividing global privsep profile https://review.opendev.org/c/openstack/nova/+/871729 | |
| 10:47:11 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Dividing global privsep profile https://review.opendev.org/c/openstack/nova/+/871729 | |
| 11:22:15 | opendevreview | Sofia Enriquez proposed openstack/nova master: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | |
| 11:51:35 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Dividing global privsep profile https://review.opendev.org/c/openstack/nova/+/871729 | |
| 11:51:35 | opendevreview | Jorge San Emeterio proposed openstack/nova master: WIP: Moving privsep profiles to nova/__init__.py https://review.opendev.org/c/openstack/nova/+/872010 | |
| 12:13:12 | opendevreview | Kashyap Chamarthy proposed openstack/nova stable/xena: libvirt: At start-up rework compareCPU() usage with a workaround https://review.opendev.org/c/openstack/nova/+/872011 | |
| 12:37:37 | elodilles | bauzas: speaking about the tons of things you are to review, i don't know whether you saw the 2023.2 Bobcat schedule plan: https://review.opendev.org/c/openstack/releases/+/869976 | |
| 12:37:41 | elodilles | o:) | |
| 12:38:21 | auniyal | Hello o/ | |
| 12:39:06 | auniyal | please review these Improving logging at '_allocate_mdevs'. | |
| 12:39:06 | auniyal | - https://review.opendev.org/q/topic:bug%252F1992451 | |
| 13:18:13 | bauzas | elodilles: I've seen it and I forgot to ask on Tuesday for folks to look at this change | |
| 13:18:21 | bauzas | elodilles: during the nova meeting | |
| 13:18:52 | bauzas | elodilles: I'm not bad about it fwiw, as we would have the Summit *before* Specfreeze which is nice | |
| 13:44:38 | opendevreview | Jorge San Emeterio proposed openstack/nova master: DNM: Testing check pipeline on master branch https://review.opendev.org/c/openstack/nova/+/872018 | |
| 13:52:24 | elodilles | bauzas: yepp, it's a fairly long cycle (28 weeks) so milestones have longer times | |
| 14:12:07 | darkhorse | Hi team, Is it possible to disallow a user from all compute resources using keystone policy or any other means? The use case is I want to have multiple roles to a project. Some users have compute permissions, some have network permissions, some have volume permissions etc. | |
| 14:12:36 | darkhorse | I am not sure if this question is more relevant to keystone channel. | |
| 15:54:47 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833436 | |
| 16:01:03 | opendevreview | Merged openstack/nova stable/yoga: Add a workaround to skip hypervisor version check on LM https://review.opendev.org/c/openstack/nova/+/851202 | |
| 16:02:42 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | |
| 16:02:58 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | |
| 16:03:45 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833437 | |
| 16:04:03 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | |
| 16:04:21 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/train: reenable greendns in nova. https://review.opendev.org/c/openstack/nova/+/833438 | |
| 16:14:33 | sean-k-mooney | dansmith: can you take a look at https://review.opendev.org/c/openstack/nova/+/863919/12/nova/tests/unit/compute/test_resource_tracker.py#1553 when you have time | |
| 16:15:26 | dansmith | sean-k-mooney: yeah, it's on my list.. ya'll drug me into a call at 6:30am and I have another starting in a few ;P | |
| 16:15:48 | sean-k-mooney | sorry about that | |
| 16:15:49 | dansmith | thanks for hitting the ones below, that's some progress for sure | |
| 16:16:25 | sean-k-mooney | so ill try and test teh rest when i get devstack deploy so ill loop back to them again later today if that works out | |
| 16:16:39 | dansmith | cool | |
| 17:10:51 | sean-k-mooney | cool devstack stacked first time | |
| 17:11:24 | sean-k-mooney | now ot actully use your patches | |
| 17:12:26 | sean-k-mooney | so currently i have all in one so when i checkout your branch and restart the nova serices im expecting the compute to start up and writeh the uuid to a file and update the service version right | |
| 17:16:39 | sean-k-mooney | yep that worked https://paste.opendev.org/show/bLMvRfuUugp9iMcn4R60/ | |
| 17:17:47 | sean-k-mooney | im going to break this in a few differnt ways and document it in an ether pad and ill let you know if i find anything odd or broken | |
| 17:25:44 | dansmith | sean-k-mooney: yeah, cool | |
| 17:29:59 | sean-k-mooney | https://etherpad.opendev.org/p/Stable-compute-uuid-manual-testing ill be usign that to keep my notes | |
| 17:30:37 | sean-k-mooney | there is not really much there yet but ill kep adding to it as i go | |
| 17:59:27 | sean-k-mooney | dansmith: found a bug | |
| 17:59:36 | sean-k-mooney | getting logs now | |
| 18:00:02 | dansmith | in "test 3 restart with deleted file" ? | |
| 18:00:08 | sean-k-mooney | yep | |
| 18:00:38 | dansmith | does that mean deleted node id file? | |
| 18:00:53 | sean-k-mooney | i moved it to compute_id_old | |
| 18:01:11 | sean-k-mooney | the agent tried to add a duplicate row in the db which resulted in a key error | |
| 18:01:15 | sean-k-mooney | and it wrote a new file | |
| 18:01:22 | sean-k-mooney | with a diffent uuid | |
| 18:01:50 | dansmith | well, right, | |
| 18:02:08 | dansmith | because it knows it's not an upgrade (now) and it didn't find a local compute node uuid | |
| 18:02:14 | dansmith | what do you think it should do there? | |