| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-04-14 | |||
| 13:30:43 | sean-k-mooney | i think it depend on your enviornment | |
| 13:31:03 | sean-k-mooney | ill spin up a c9s vm in a sec and see if i can repoduce | |
| 13:31:45 | sean-k-mooney | 17 would be stable wallaby i guess. i wonder if its stable branch specific or if it happens on master | |
| 13:43:37 | bauzas | stephenfin: https://review.opendev.org/c/openstack/placement/+/801104/9/placement/objects/allocation.py#166 hmpf, I'm a bit sad we need to use a private field for getting the mapping | |
| 13:43:52 | bauzas | sounds weird and anti-pattern | |
| 13:44:19 | bauzas | I tried to read the SQLA doc about the deprecation warning you mentioned, but this didn't help | |
| 13:45:31 | stephenfin | bauzas: agreed but I fixed it later https://review.opendev.org/c/openstack/placement/+/801107/9 | |
| 13:45:53 | bauzas | ah ok, I stopped on reading the series on that one | |
| 13:47:15 | sean-k-mooney | bauzas: stephen kept refactor to not use it seperate form the deprecation fix but i had the same feeling when reviewing it initally | |
| 13:47:30 | bauzas | sean-k-mooney: I just commented | |
| 13:47:45 | bauzas | oh by the way, I hate my bias | |
| 13:48:01 | bauzas | I said loudly "STOP DOING BLIND RECHECKS" and I fuckingly made one | |
| 13:48:18 | bauzas | bad manners are so easy to keep | |
| 13:48:43 | sean-k-mooney | hehe i noticed stephenfin actully copyt the fact that they rechecked beause the base patch failed to merge which made me happy :) | |
| 15:56:25 | opendevreview | Merged openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 15:56:27 | opendevreview | Merged openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 15:56:30 | opendevreview | Merged openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 15:56:32 | opendevreview | Merged openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 16:06:25 | opendevreview | Merged openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 18:41:32 | opendevreview | Elod Illes proposed openstack/nova stable/yoga: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838000 | |
| 18:41:36 | elodilles | melwitt: if this passes then i'll cherry pick them to older branches ^^^ | |
| 18:42:31 | melwitt | elodilles: thanks! I will watch it | |
| 19:16:38 | opendevreview | Elod Illes proposed openstack/nova stable/xena: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838007 | |
| 19:16:55 | elodilles | melwitt: though the gate is not finished still it seems the job settings (and e.g. requirements-check job) are OK, so I started cherry picking ^^^ | |
| 19:17:30 | melwitt | elodilles: ack, thanks! | |
| 19:24:40 | opendevreview | Elod Illes proposed openstack/nova stable/wallaby: [stable-only] Drop lower-constraints job https://review.opendev.org/c/openstack/nova/+/838030 | |
| 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: libvirt/config: Parse the 'cpu' element in domainCapabilities https://review.opendev.org/c/openstack/nova/+/838191 | |
| 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:27 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt/host: Introduce _get_hypervisor_info() https://review.opendev.org/c/openstack/nova/+/838193 | |
| 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: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 | |