| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-09 | |||
| 08:46:36 | bauzas | kashyap: thanks for your comments, good explanation why we don't want it for production use | |
| 08:46:56 | kashyap | No prob | |
| 08:47:04 | opendevreview | Sylvain Bauza proposed openstack/nova master: Add the Yoga prelude section https://review.opendev.org/c/openstack/nova/+/832292 | |
| 08:47:05 | bauzas | there it goes... | |
| 08:47:22 | bauzas | dansmith: gibi: amended the prelude based on dan's comment | |
| 08:49:52 | bauzas | elodilles: today and tomorrow will be busy days for merging things | |
| 08:50:13 | bauzas | elodilles: I guess we will only be able to amend the nova rc1 yaml by the end of our day | |
| 08:50:32 | bauzas | (tomorrow I mean) | |
| 08:53:33 | bauzas | hah, all water issues fixed, let me visit my shower :) | |
| 09:26:54 | opendevreview | ribaudr proposed openstack/os-traits master: Attach Manila shares via virtiofs(traits) https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 09:38:37 | elodilles | bauzas: ack, if the patch is -1'd then the release managers won't force-approve the patch without pinging you ;) | |
| 09:38:53 | bauzas | yah I know | |
| 09:39:08 | bauzas | elodilles: but I wanted to explain we need to merge a few changes before removing the -1 | |
| 09:39:18 | bauzas | so it would be a bit late tomorrow | |
| 09:39:25 | elodilles | bauzas: cool, roger :) | |
| 09:39:34 | bauzas | (as we need to wait for the merge) | |
| 09:39:38 | opendevreview | ribaudr proposed openstack/os-traits master: Add 'COMPUTE_STORAGE_VIRTIO_FS', 'COMPUTE_MEM_BACKING_FILE' https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 09:39:51 | elodilles | i'll be around, so no problem :) | |
| 09:40:25 | elodilles | and thanks for the heads-up | |
| 09:41:07 | bauzas | elodilles: haha, https://www.youtube.com/watch?v=IC7pZSd8iRg | |
| 09:56:24 | elodilles | yepp, that's classic ;) | |
| 10:22:16 | rribaud__ | hello, question, I have modified os-trait locally. What is the best way to use it with tox (nova) ? | |
| 10:33:16 | opendevreview | Merged openstack/nova master: Follow up for unified limits https://review.opendev.org/c/openstack/nova/+/830918 | |
| 11:27:11 | ygk_12345 | Folks, in which database is the nova flavor details stored ? | |
| 11:27:34 | ygk_12345 | for all the created flavors I mean | |
| 11:36:54 | ygk_12345 | any idea ? | |
| 11:40:56 | rribaud__ | ygk_12345, I think it is in nova_api --> flavors table | |
| 11:51:15 | ygk_12345 | rribaud__: thanks for the info | |
| 12:35:00 | opendevreview | ribaudr proposed openstack/os-traits master: Add 'COMPUTE_STORAGE_VIRTIO_FS', 'COMPUTE_MEM_BACKING_FILE' https://review.opendev.org/c/openstack/os-traits/+/832769 | |
| 12:55:33 | sean-k-mooney1 | rribaud__: you just do .tox/py3/bin/pip install ../os-traits | |
| 12:56:04 | sean-k-mooney1 | so after you have created the tox env if you need a local os-traits just install it and run tox again | |
| 12:56:26 | sean-k-mooney1 | or run the command tox would run manuatlly | |
| 12:56:43 | sean-k-mooney1 | .tox/py3/bin/stestr ... | |
| 13:45:00 | opendevreview | Merged openstack/nova master: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/821840 | |
| 13:53:21 | rribaud__ | sean-k-mooney1, thx. | |
| 13:53:56 | rribaud__ | sean-k-mooney1++ | |
| 14:08:33 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/830010 | |
| 14:17:11 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/828570 | |
| 14:20:59 | erlon | bauzas: hey, in the pre_live_migration fix, what you mean with 'Can you please provide a FUP for this', to implement seans' suggestion? | |
| 14:27:13 | bauzas | erlon: sorry, FUP means "a follow-up patch" | |
| 14:27:37 | bauzas | erlon: that means I can +W this patch if you agree with providing a separate patch for modifying the reno file | |
| 14:27:49 | erlon | bauzas: oh, ok, sure, Ill do and address the other comments too | |
| 14:27:58 | bauzas | erlon: thanks | |
| 14:28:08 | bauzas | erlon: we can merge this FUP before tomorrow | |
| 14:28:15 | bauzas | (before RC1) | |
| 14:28:27 | erlon | right, that was my only concern | |
| 15:23:34 | bauzas | erlon: I don't yet see your FUP ? | |
| 15:24:45 | erlon | bauzas: was in a meeting | |
| 15:27:18 | bauzas | erlon: ok, just ping me when you send it | |
| 15:27:28 | bauzas | and then I'll +W the original patch | |
| 15:33:52 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | |
| 15:34:02 | erlon | bauzas: ^ | |
| 15:34:09 | bauzas | thanks | |
| 15:35:08 | bauzas | erlon: oh you made a new revision | |
| 15:35:22 | bauzas | anyway, will +2/+W it then | |
| 15:35:26 | bauzas | sean-k-mooney: ^ | |
| 15:36:03 | erlon | bauzas: I was wondering about that, that a new ps would trigger all CI runs | |
| 15:36:13 | bauzas | erlon: it will | |
| 15:36:21 | erlon | bauzas: how did I suppose to do to avoid that? | |
| 15:36:21 | bauzas | erlon: but it should be OK | |
| 15:36:31 | bauzas | erlon: by providing a new change | |
| 15:36:38 | bauzas | or a patch | |
| 15:36:44 | erlon | ow, depended on that one | |
| 15:36:47 | erlon | got it | |
| 15:36:48 | bauzas | yup | |
| 15:36:53 | bauzas | anyway, +Wd | |
| 15:37:01 | bauzas | so the CI will verify it twice | |
| 15:37:05 | erlon | ok, it should be good for tomorrow | |
| 15:37:06 | bauzas | should be OK | |
| 15:37:08 | bauzas | yes | |
| 15:37:17 | erlon | thanks for helping with that | |
| 15:45:45 | sean-k-mooney | rribaud__: by the way do you prefer that nic or uggla | |
| 15:46:15 | sean-k-mooney | erlon: bauzas i quickly reviewd it and +2'd it also | |
| 15:46:30 | sean-k-mooney | erlon: thanks for following up with us :) | |
| 15:46:42 | rribaud__ | sean-k-mooney, uggla one, it changes following a disconnect. | |
| 15:47:24 | sean-k-mooney | cool | |
| 15:47:56 | sean-k-mooney | i havent fully memorised your name yet so was not sure which one you wanted to go by | |
| 15:49:15 | Uggla | I try to use rribaud on internal and Uggla external | |
| 15:49:45 | Uggla | to make a clear diff and avoid to disclose internal stuff. :) | |
| 15:49:46 | sean-k-mooney | ya some do that | |
| 15:49:56 | sean-k-mooney | i prefer to use the same for both | |
| 15:50:41 | Uggla | but for strange reason after disconnect, reconnect pick my internal one. | |
| 17:08:18 | bauzas | gibi: I'm doing a bit of rounding about the RC1 bugs https://bugs.launchpad.net/nova/+bugs?field.tag=yoga-rc-potential | |
| 17:08:22 | bauzas | sean-k-mooney: ^ | |
| 17:08:46 | bauzas | I'm marking the according patches as review-prio | |
| 17:09:09 | bauzas | for the open bugs where no core hasn't reviewed yet, I'll descope them from the RC1 tag tomorrow morning | |
| 17:09:51 | Uggla | yuzu + lemon rules. | |
| 17:13:51 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 17:15:35 | melwitt | bauzas: not related to RC1 but on bugs, I have a proposed change for a gate failure bug here if you could lmk if you see any issues with it https://review.opendev.org/c/openstack/nova/+/831084 | |
| 17:15:47 | bauzas | melwitt: sure | |
| 17:16:04 | melwitt | ty | |
| 17:44:27 | bauzas | melwitt: looked at your patch, needs my full brain for it | |
| 17:44:53 | melwitt | bauzas: np, it's not urgent | |
| 17:45:01 | bauzas | melwitt: will look at it once I'm rejuverated tomorrow morning :) | |
| 17:45:18 | bauzas | rejuvenated* :) | |
| 17:45:32 | melwitt | thanks :) | |
| 17:47:51 | whoami-rajat | dansmith, hey, the rebuild test passed \o/ https://review.opendev.org/c/openstack/tempest/+/831018 | |
| 17:50:58 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reproduce bug 1953359 https://review.opendev.org/c/openstack/nova/+/822047 | |
| 17:50:59 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Extend the reproducer for 1953359 and 1952915 https://review.opendev.org/c/openstack/nova/+/822048 | |
| 17:51:00 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: [rt] Apply migration context for incoming migrations https://review.opendev.org/c/openstack/nova/+/822050 | |