| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-07-07 | |||
| 12:53:40 | sean-k-mooney | if its set to compelte or error i think it will allow the ocmpute service to be deleted | |
| 12:54:25 | ozzzo_work | ok. I was hoping to avoid spelunking in the DB but fortunately this is a dev cluster | |
| 12:55:07 | sean-k-mooney | well the other approch is to understadn why the compute service is not coming up | |
| 12:55:09 | sean-k-mooney | and fix that | |
| 12:55:14 | sean-k-mooney | and avoid redpeloying it | |
| 12:55:25 | bauzas | elodilles: you probably have seen it but I updated https://review.opendev.org/c/openstack/releases/+/885365 | |
| 12:57:58 | elodilles | bauzas: ACK, thanks! \o/ | |
| 13:31:18 | opendevreview | sean mooney proposed openstack/nova stable/2023.1: enable validations in nova-lvm https://review.opendev.org/c/openstack/nova/+/887674 | |
| 15:13:47 | opendevreview | Merged openstack/nova-specs master: Add support for Napatech LinkVirt SmartNICs https://review.opendev.org/c/openstack/nova-specs/+/859290 | |
| 16:20:49 | gouthamr | o/ sean-k-mooney Uggla_ bauzas : we've one last manila api spec relating to virtiods to close out on, you'd looked at it in the past, could you please look again? https://review.opendev.org/c/openstack/manila-specs/+/881934 | |
| 16:24:51 | sean-k-mooney | i tried to look at this yesteredsay but i was too tired. to dig into it. there are still refecne to thte nova service user that make me uncomfortable | |
| 16:43:24 | sean-k-mooney | i think Uggla_ and bauzas are best placed to review form the context of the nova mainal share feature | |
| 16:43:58 | sean-k-mooney | gouthamr: if ye have a spec freeze i would proceed with it and we can try and loop back to it | |
| 16:44:03 | sean-k-mooney | when its used | |
| 17:00:58 | gouthamr | sean-k-mooney++ ty; ack, we can surely work some details over the code reviews | |
| 17:02:07 | sean-k-mooney | what im expecting is when an access rules visablity restriciotn is create by nova | |
| 17:03:55 | sean-k-mooney | the rule use shoudl be defied using the proejct id and user id form the users token not nova's servife token | |
| 17:04:52 | sean-k-mooney | but we would record that the visabity restriction was create by a service not a user | |
| 17:09:22 | sean-k-mooney | nova may be talkign to manilla with an applciation credital or its user account could change over time so we shoudl not be recording hte userid or proejct id form the service token in manilla | |
| 17:18:01 | gouthamr | yeah we’d not be storing any information regarding nova in the Manila db | |
| 17:18:55 | gouthamr | nova user I mean :) | |
| 17:19:48 | gouthamr | the service user is validated, and we store a string to remember that a service user was used to create a lock… the api would only allow service users to remove the lock | |
| 18:07:44 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887978 | |
| 18:07:45 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887979 | |
| 18:08:58 | opendevreview | melanie witt proposed openstack/nova stable/zed: testing: Fix and robustify archive_deleted_rows test https://review.opendev.org/c/openstack/nova/+/887980 | |
| 18:08:59 | opendevreview | melanie witt proposed openstack/nova stable/zed: database: Archive parent and child rows "trees" one at a time https://review.opendev.org/c/openstack/nova/+/887981 | |
| 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 | dansmith, can you please review these as well: | |
| 17:02:01 | auniyal71 | https://review.opendev.org/c/openstack/nova/+/873648 | |
| 17:02:01 | auniyal71 | https://review.opendev.org/c/openstack/nova/+/877446 | |
| 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: Refactor CinderFixture https://review.opendev.org/c/openstack/nova/+/885756 | |
| 06:21:25 | opendevreview | Amit Uniyal proposed openstack/nova master: Reproducer for dangling bdms https://review.opendev.org/c/openstack/nova/+/881457 | |
| 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: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 12:55:29 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 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 | bauzas | #startmeeting nova | |
| 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 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:29 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:36 | bauzas | howdy | |
| 16:00:46 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |