| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-07 | |||
| 18:16:52 | opendevreview | melanie witt proposed openstack/nova stable/yoga: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887982 | |
| 18:16:53 | opendevreview | melanie witt proposed openstack/nova stable/yoga: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887983 | |
| 18:34:52 | opendevreview | melanie witt proposed openstack/nova stable/xena: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887984 | |
| 18:34:53 | opendevreview | melanie witt proposed openstack/nova stable/xena: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887985 | |
| 18:42:02 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887987 | |
| 18:42:03 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887988 | |
| 20:49:04 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887988 | |
| 21:16:55 | opendevreview | melanie witt proposed openstack/nova stable/victoria: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887997 | |
| 21:16:56 | opendevreview | melanie witt proposed openstack/nova stable/victoria: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887998 | |
| 21:50:36 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/888001 | |
| 21:50:37 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/888002 | |
| #openstack-nova - 2023-07-10 | |||
| 09:49:27 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Add 'COMPUTE_ADDRESS_SPACE_*' traits support https://review.opendev.org/c/openstack/nova/+/873221 | |
| 13:07:17 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Bump MIN_{LIBVIRT,QEMU} for "Bobcat" https://review.opendev.org/c/openstack/nova/+/887255 | |
| 13:11:06 | kashyap | sean-k-mooney: --^ Hope that looks better; I'd also like a closer look at the MIN_LIBVIRT_AARCH64_CPU_COMPARE const removal code (Cc: gibi) | |
| 14:42:13 | dansmith | bauzas: I'd really like to get some review on this while I have time to address any concerns and so there's some soak time before release: https://review.opendev.org/q/topic:bp/compute-object-ids+status:open | |
| 14:42:25 | dansmith | if I should just stop asking for this release let me know :) | |
| 14:51:11 | bauzas | dansmith: no no, sorry, last week I was focusing on spec | |
| 14:51:20 | bauzas | this week, I'll start to look on the series :) | |
| 14:52:15 | dansmith | thanks | |
| 15:30:38 | auniyal71 | hey dansmith, can you please have another look on this - Refactoring CinderFixture - https://review.opendev.org/c/openstack/nova/+/885756/6 | |
| 15:31:11 | dansmith | auniyal71: yep, I will.. I haven't yet processed my review email from being on PTO :) | |
| 15:31:30 | auniyal71 | ack, yes, was waiting for you :) thanks | |
| 16:24:03 | opendevreview | Dan Smith proposed openstack/nova master: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | |
| 16:24:11 | dansmith | auniyal71: fixed ^ | |
| 16:49:01 | auniyal71 | ack, thanks, how did you knew it failed at _test_attach_volume_error | |
| 16:50:57 | auniyal71 | dansmith, did it show in error | |
| 16:51:03 | auniyal71 | with tox | |
| 16:51:45 | dansmith | auniyal71: you saw what I saw, which is what was shown in CI | |
| 16:52:01 | auniyal71 | yes | |
| 16:52:22 | dansmith | the problem was not *your* autospec=False thing, it was that case of the test was attempting to re-mock something you already had mocked and *that* was the failing autospec | |
| 16:52:45 | dansmith | i.e. the mock decorator on that test sub-case that I removed | |
| 16:54:08 | auniyal71 | okay, | |
| 16:56:03 | auniyal71 | yes, I understood , it was re-mocking and thats problem but couldn't find at what place. | |
| 16:56:43 | dansmith | to find that, I added a try..except around the action() call, and then if failed, printed which one of those in the list was causing the problem | |
| 16:56:55 | dansmith | then going to that, it was clear to find that decorator mock | |
| 16:57:36 | auniyal71 | oh okay, got it, thanks dansmith | |
| 17:02:01 | auniyal71 | https://review.opendev.org/c/openstack/nova/+/877446 | |
| 17:02:01 | auniyal71 | https://review.opendev.org/c/openstack/nova/+/873648 | |
| 17:02:01 | auniyal71 | dansmith, can you please review these as well: | |
| 17:07:16 | dansmith | auniyal71: yep I have them on my list but it looks like still some outstanding replies from others are expected | |
| 17:07:46 | auniyal71 | ack, | |
| 18:40:45 | opendevreview | sean mooney proposed openstack/os-vif stable/train: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841779 | |
| 18:40:46 | opendevreview | sean mooney proposed openstack/os-vif stable/train: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841780 | |
| 18:53:24 | opendevreview | sean mooney proposed openstack/os-vif stable/ussuri: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841777 | |
| 18:53:25 | opendevreview | sean mooney proposed openstack/os-vif stable/ussuri: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841778 | |
| 19:08:14 | opendevreview | sean mooney proposed openstack/os-vif stable/train: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841779 | |
| 19:08:15 | opendevreview | sean mooney proposed openstack/os-vif stable/train: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841780 | |
| 22:05:35 | opendevreview | melanie witt proposed openstack/nova master: Improve logging at '_numa_cells_support_network_metadata' https://review.opendev.org/c/openstack/nova/+/860930 | |
| #openstack-nova - 2023-07-11 | |||
| 00:29:59 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888091 | |
| 00:30:00 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888092 | |
| 00:31:27 | opendevreview | melanie witt proposed openstack/nova stable/zed: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888093 | |
| 00:31:28 | opendevreview | melanie witt proposed openstack/nova stable/zed: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888094 | |
| 00:38:08 | opendevreview | melanie witt proposed openstack/nova stable/yoga: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888095 | |
| 00:38:09 | opendevreview | melanie witt proposed openstack/nova stable/yoga: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888096 | |
| 02:18:28 | opendevreview | melanie witt proposed openstack/nova stable/xena: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888099 | |
| 02:18:29 | opendevreview | melanie witt proposed openstack/nova stable/xena: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888100 | |
| 02:23:24 | opendevreview | suzhengwei proposed openstack/nova-specs master: spec: evacuate api return migration https://review.opendev.org/c/openstack/nova-specs/+/888101 | |
| 02:58:53 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888103 | |
| 02:58:54 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888104 | |
| 03:00:41 | opendevreview | melanie witt proposed openstack/nova stable/xena: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888100 | |
| 06:21:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 06:21:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | |
| 06:21:26 | opendevreview | Amit Uniyal proposed openstack/nova master: Delete dangling bdms https://review.opendev.org/c/openstack/nova/+/882284 | |
| 12:55:29 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 12:55:29 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 15:29:23 | bauzas | reminder, nova meeting in 30 mins | |
| 15:29:37 | bauzas | I'd appreciate if we could make it quick, I need to leave earlt | |
| 15:37:13 | elodilles | o7 | |
| 16:00:29 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:29 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:29 | opendevmeet | Meeting started Tue Jul 11 16:00:29 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:29 | bauzas | #startmeeting nova | |
| 16:00:36 | bauzas | howdy | |
| 16:00:46 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:03 | bauzas | hopefully will be short, I need to go off early | |
| 16:01:09 | bauzas | who's around ? | |
| 16:01:12 | gibi | o/ | |
| 16:01:19 | auniyal | o/ | |
| 16:01:20 | sean-k-mooney | o/ ish | |
| 16:01:27 | sean-k-mooney | im here but a littel distracted | |
| 16:01:29 | dansmith | o/ kinda | |
| 16:01:37 | elodilles | o/ | |
| 16:02:06 | bauzas | okay, let's start | |
| 16:02:14 | bauzas | happy to see quorum here | |
| 16:02:20 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:24 | bauzas | #info One Critical bug | |
| 16:02:28 | bauzas | #link https://bugs.launchpad.net/nova/+bug/2025813 | |
| 16:02:30 | bauzas | so | |
| 16:02:56 | bauzas | afaict, https://review.opendev.org/c/openstack/nova/+/887632 is merged | |
| 16:03:19 | bauzas | so we should modify the bug importance to High, fair enough ? | |
| 16:03:33 | bauzas | (at least for the master branch) | |
| 16:03:36 | elodilles | it's only affecting now stable branches | |
| 16:03:51 | bauzas | yup | |
| 16:04:02 | elodilles | and of course nova/virt/libvirt/* patches | |
| 16:04:03 | bauzas | we're tracking the stable branches correctly in the bug report | |
| 16:04:11 | elodilles | ++ | |
| 16:04:31 | bauzas | so, let's keep the bug importance to Critical for all the stable series | |
| 16:04:40 | elodilles | +1 | |
| 16:04:40 | bauzas | but changing to High for the main one | |
| 16:04:45 | bauzas | cool, doing it now | |