| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-14 | |||
| 19:26:34 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838032 | |
| 19:31:05 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838033 | |
| 19:40:18 | opendevreview | Merged openstack/placement master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/placement/+/828119 | |
| 19:41:24 | opendevreview | Elod Illes proposed openstack/nova stable/train: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838037 | |
| 19:47:38 | opendevreview | Elod Illes proposed openstack/nova stable/stein: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838038 | |
| 20:02:08 | melwitt | elodilles: hm, looks like this is saying on master too https://lists.openstack.org/pipermail/openstack-discuss/2022-April/028199.html so I guess it's not supposed to be stable-only? | |
| 20:05:18 | opendevreview | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838041 | |
| 20:06:50 | elodilles | melwitt: yes, though i wanted to wait for the resolution to drop it on master | |
| 20:07:08 | melwitt | ok | |
| 20:25:17 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838042 | |
| 20:26:03 | opendevreview | Merged openstack/placement master: db: Use Row, not LegacyRow https://review.opendev.org/c/openstack/placement/+/828305 | |
| 20:26:06 | opendevreview | Merged openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 23:08:20 | opendevreview | melanie witt proposed openstack/nova stable/train: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838050 | |
| 23:47:15 | gmann | elodilles: melwitt +1, for master we can wait for resolution to merge but it is all good to drop from stable as they are broken | |
| 23:47:40 | melwitt | gmann: understood, thanks | |
| 23:50:49 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838042 | |
| 23:55:22 | opendevreview | melanie witt proposed openstack/nova stable/train: Lookup nic feature by PCI address https://review.opendev.org/c/openstack/nova/+/838050 | |
| #openstack-nova - 2022-04-15 | |||
| 07:34:52 | bauzas | good Friday Nova | |
| 12:28:38 | chateaulav | happy friday! | |
| 14:06:08 | bauzas | gmann: I know you're not around till' Tuesday, but this bug seems valid to me https://bugs.launchpad.net/nova/+bug/1969054 | |
| 15:01:55 | kashyap | bauzas: Is upstream Gerrit down? | |
| 15:01:56 | kashyap | Duuuh | |
| 15:02:00 | kashyap | Yeah, it is | |
| 15:02:10 | kashyap | "a few minutes" :D | |
| 15:03:45 | bauzas | kashyap: yup, see notice ^ | |
| 15:07:08 | kashyap | Yep, saw it | |
| 15:07:14 | kashyap | Have a good weekend! | |
| 15:07:23 | kashyap | It's actually back up | |
| 15:28:25 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/762330 | |
| 15:28:26 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: fixtures: Add libvirt domCapabilities data for s390x and PPC* https://review.opendev.org/c/openstack/nova/+/838192 | |
| 15:28:26 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt/config: Parse the 'cpu' element in domainCapabilities https://review.opendev.org/c/openstack/nova/+/838191 | |
| 15:28:27 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace baselineCPU() with baselineHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/838194 | |
| 15:28:27 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt/host: Introduce _get_hypervisor_info() https://review.opendev.org/c/openstack/nova/+/838193 | |
| 15:28:28 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: test_report_cpu_traits: Update stale comment https://review.opendev.org/c/openstack/nova/+/838195 | |
| 16:38:20 | gmann | bauzas: sure, let me check after my lunch. | |
| 17:30:19 | gmann | bauzas: added details in that bug: https://bugs.launchpad.net/nova/+bug/1969054/comments/3 | |
| 17:31:59 | gmann | bauzas: in summary: this is exact issue we solved in Yoga with new RBAC direction (decoupled the scope from new defaults). reported issue is in Victoria. to solve it in victoria, I will say to override the external event policy but there might be some other isuse which we are not sure. | |
| 17:33:13 | gmann | bauzas: as we cannot backport the new policy changes done in yoga to victoria, we can say "new defaults from ussuri to xena does not work as expected so please do not try those, start trying those from Yoga release". | |
| 17:33:42 | gmann | or any other better way to notify this ^^ to users. | |
| 17:57:21 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove unnecessary packages in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/838199 | |
| #openstack-nova - 2022-04-16 | |||
| 08:35:33 | opendevreview | ZhouYanbing proposed openstack/nova master: correct the wrong word in the notes https://review.opendev.org/c/openstack/nova/+/838205 | |
| #openstack-nova - 2022-04-18 | |||
| 15:59:48 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 15:59:49 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838323 | |
| 17:58:34 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838323 | |
| 17:58:35 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 18:02:45 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836016 | |
| 18:02:46 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838332 | |
| 18:04:25 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836017 | |
| 18:04:26 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838334 | |
| 19:09:14 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838332 | |
| 19:09:15 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/wallaby: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836016 | |
| 19:33:11 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838323 | |
| 19:33:12 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836015 | |
| 19:43:27 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838334 | |
| 19:43:28 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/victoria: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/836017 | |
| #openstack-nova - 2022-04-19 | |||
| 06:23:13 | gibi | good morning | |
| 06:32:17 | opendevreview | Wenping Song proposed openstack/os-traits master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-traits/+/838436 | |
| 07:38:29 | bauzas | good morning | |
| 07:38:53 | bauzas | this one seems weird to me : https://bugs.launchpad.net/nova/+bug/1968605 | |
| 07:39:13 | bauzas | probably a dependency issue, I'm about to ask the oslo.policy release version they use | |
| 07:44:57 | gibi | bauzas: o/ | |
| 07:47:42 | gibi | yeah the second half of that stack trace really look like an infinite recursion | |
| 07:48:44 | gibi | ahh the first half even states it that 2022-04-11 19:49:41.126 3391739 ERROR nova.api.openstack.wsgi RecursionError: maximum recursion depth exceeded | |
| 07:53:22 | gibi | yeah we should figure out a reproducer for this... | |
| 07:57:33 | bauzas | gibi: marked as Incomplete and asked for more details about the policies they use | |
| 07:58:40 | gibi | ack | |
| 07:58:49 | bauzas | another bug which seems legit but I can't test it : https://bugs.launchpad.net/nova/+bug/1968645 | |
| 07:59:00 | gibi | if the policy language allows the definition of recursive rules then that would be interesting | |
| 07:59:22 | bauzas | gibi: yeah, they could have fucked their DSL | |
| 07:59:43 | gibi | bauzas: the devstack lvm cinder backend allows multiattach so in theory we could test this | |
| 08:00:14 | bauzas | gibi: I know, that's just me who lacks having an env ready for this :) | |
| 08:01:06 | gibi | yeah, I also left my multinode env at my previous employer, but I will set up something new in the coming days... | |
| 08:03:09 | opendevreview | Wenping Song proposed openstack/os-resource-classes master: Remove unnecessary unicode prefixes https://review.opendev.org/c/openstack/os-resource-classes/+/838448 | |
| 08:03:11 | bauzas | gibi: I'll pass you the triage baton tonight :p | |
| 08:03:53 | gibi | bauzas: that will be nice, anther reason to build some devstack env :D | |
| 08:04:13 | bauzas | gibi: I have two VMs running somewhere :) | |
| 08:04:16 | gibi | bauzas: btw I think I see similar multiattach state handling issues in cinder. See my comment https://bugs.launchpad.net/cinder/+bug/1968645/comments/3 | |
| 08:04:31 | bauzas | gibi: this sounds legit as we race | |
| 08:04:40 | bauzas | we race for the status change | |
| 08:04:51 | gibi | yepp | |
| 08:04:56 | bauzas | and given we look at it before... | |
| 08:04:57 | gibi | It seems cinder suggest to retry | |
| 08:05:06 | bauzas | simple and classic concurrency issue | |
| 08:09:49 | opendevreview | Wenping Song proposed openstack/os-resource-classes master: Update python testing classifier https://review.opendev.org/c/openstack/os-resource-classes/+/822473 | |
| 08:18:08 | songwenping | bauzas, gibi: please review the spec of 'Usage of new trait of OWNER_NOVA' https://review.opendev.org/c/openstack/nova-specs/+/819510 | |
| 08:18:32 | bauzas | songwenping: OK, I can try later in the day | |
| 08:21:05 | songwenping | thanks | |
| 08:21:28 | opendevreview | ZhouYanbing proposed openstack/nova master: correct the wrong content in the notes https://review.opendev.org/c/openstack/nova/+/838205 | |
| 08:55:45 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Ignore LibvirtConfigObject kwargs https://review.opendev.org/c/openstack/nova/+/830644 | |
| 08:55:46 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Add vIOMMU device to guest https://review.opendev.org/c/openstack/nova/+/830646 | |
| 08:55:46 | opendevreview | Rico Lin proposed openstack/nova master: libvirt: Remove unnecessary TODO https://review.opendev.org/c/openstack/nova/+/830645 | |
| 08:56:32 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: test_report_cpu_traits: Update stale comment https://review.opendev.org/c/openstack/nova/+/838195 | |
| 08:56:32 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/762330 | |
| 08:57:33 | kashyap | gibi: When you get time this week, can you please have a look at these? I've spent a lot of time splitting up the patches: https://review.opendev.org/q/topic:bp%252Fcpu-selection-with-hypervisor-consideration | |
| 08:57:45 | gibi | kashyap: added to my review list | |
| 08:58:28 | kashyap | gibi: Thank you. (I'd really like to be done with this. So all help appreciated :)) | |
| 08:58:44 | gibi | I also would like to get this done :D | |
| 09:01:15 | kashyap | Yeah, sorry for pausing and dragging this | |
| 09:01:44 | gibi | no worries | |
| 13:09:09 | opendevreview | Erlon R. Cruz proposed openstack/nova stable/xena: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/838323 | |