| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-01-18 | |||
| 18:20:24 | _colby | its compaining about no available instances on that device but there is: cat /sys/class/mdev_bus/0000\:17\:02.0/mdev_supported_types/nvidia-563/available_instances | |
| 18:22:22 | _colby | also that device is not listed in the config for that type: device_addresses=0000:17:00.4,0000:17:00.5,0000:17:00.6,0000:17:00.7 | |
| 21:51:51 | opendevreview | Merged openstack/nova master: Allow config to support virtiofs (driver) https://review.opendev.org/c/openstack/nova/+/886522 | |
| 21:51:59 | opendevreview | Merged openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| #openstack-nova - 2024-01-19 | |||
| 00:36:34 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for 2024.2 Dalmatian https://review.opendev.org/c/openstack/nova-specs/+/906073 | |
| 05:23:47 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Updates glance fixture for create image https://review.opendev.org/c/openstack/nova/+/906088 | |
| 05:23:48 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Fixes: bfv vm reboot ends up in an error state. https://review.opendev.org/c/openstack/nova/+/906089 | |
| 05:44:49 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 05:44:49 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 05:44:50 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 08:32:35 | bauzas | _colby: I'm here now but let's discuss this when you're back, I have some questions (saw your pastebin) | |
| 09:19:05 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: Fixes: bfv vm reboot ends up in an error state. https://review.opendev.org/c/openstack/nova/+/906089 | |
| 09:55:03 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 09:55:04 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 09:55:04 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 09:55:05 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 09:55:05 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 09:55:06 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 09:55:06 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 09:55:07 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild and unshelve with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870939 | |
| 09:55:07 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 09:55:09 | opendevreview | melanie witt proposed openstack/nova master: libvirt: make |
|
| 09:55:09 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/873675 | |
| 09:55:11 | opendevreview | melanie witt proposed openstack/nova master: Reject (resize|rebuild) API requests with conflicting encryption https://review.opendev.org/c/openstack/nova/+/904240 | |
| 09:55:11 | opendevreview | melanie witt proposed openstack/nova master: WIP Support migration with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/905512 | |
| 09:55:13 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 09:55:13 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 09:55:15 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 10:46:02 | opendevreview | Sylvain Bauza proposed openstack/nova master: vgpu: Allow device_addresses to not be set https://review.opendev.org/c/openstack/nova/+/902084 | |
| 10:46:02 | opendevreview | Sylvain Bauza proposed openstack/nova master: libvirt: Cap with max_instances GPU types https://review.opendev.org/c/openstack/nova/+/899625 | |
| 10:46:03 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: add coverage for registered dyn opts https://review.opendev.org/c/openstack/nova/+/902771 | |
| 10:54:54 | opendevreview | Sylvain Bauza proposed openstack/nova master: FUP: add coverage for registered dyn opts https://review.opendev.org/c/openstack/nova/+/902771 | |
| 13:18:46 | opendevreview | Tobias Urdin proposed openstack/nova master: Fail nova-compute startup when RP cannot be created https://review.opendev.org/c/openstack/nova/+/904381 | |
| 13:25:04 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: set remaining to 0 when no disk to migrate https://review.opendev.org/c/openstack/nova/+/873846 | |
| 13:25:09 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: set remaining to 0 when no disk to migrate https://review.opendev.org/c/openstack/nova/+/873846 | |
| 13:29:28 | opendevreview | Tobias Urdin proposed openstack/nova master: Fix wrong nova-manage command in upgrade check https://review.opendev.org/c/openstack/nova/+/880819 | |
| 13:33:50 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove libvirt tunnelled migration https://review.opendev.org/c/openstack/nova/+/879021 | |
| 13:36:20 | opendevreview | Tobias Urdin proposed openstack/nova master: Remove libvirt tunnelled migration https://review.opendev.org/c/openstack/nova/+/879021 | |
| 14:28:45 | opendevreview | Tobias Urdin proposed openstack/nova master: Fail nova-compute startup when RP cannot be created https://review.opendev.org/c/openstack/nova/+/904381 | |
| 14:38:30 | opendevreview | Tobias Urdin proposed openstack/nova master: Fail nova-compute startup when RP cannot be created https://review.opendev.org/c/openstack/nova/+/904381 | |
| 14:48:43 | opendevreview | Merged openstack/nova master: [ironic] Partition & use cache for list_instance* https://review.opendev.org/c/openstack/nova/+/900831 | |
| 14:52:32 | bauzas | gibi: thanks a lot for the mdev series on both bugfix series | |
| 14:52:47 | bauzas | gibi: are you available for discussing about https://review.opendev.org/c/openstack/nova/+/845757/3/nova/virt/libvirt/driver.py#8563 ? | |
| 14:58:55 | bauzas | (just replied) | |
| 14:59:17 | gibi | bauzas: o/. | |
| 14:59:34 | bauzas | tl;dr: we continue to support VGPU allocations reshape in our code | |
| 14:59:40 | bauzas | and yeah, that's crazy. | |
| 14:59:45 | gibi | so the reshape test needs a from state and to create a from state we need to support some old behavior? | |
| 15:00:07 | bauzas | gibi: well, the better would be to cut from tree all the reshape code | |
| 15:00:15 | bauzas | that's what I replied | |
| 15:00:30 | bauzas | I'm just about to file a blueprint about it | |
| 15:01:37 | bauzas | papework-wise, I feel brave enough to have a 6th effort on VGPU (and a 6th potential conflict) that would drop the whole reshape code (and tests) | |
| 15:01:50 | bauzas | and then amend my bugfix to not care about non-nested RPs | |
| 15:02:08 | bauzas | but see, that's chicken and eggs here | |
| 15:02:09 | gibi | can we decouple the test from the reshape support? E.g. can we prepare the from state purely in placement / db and then let the reshape code run on that state? | |
| 15:02:53 | gibi | (I'm not suggesting to do the reshape removal in the current cycle) | |
| 15:03:26 | bauzas | gibi: not sure I get your point, are you asking me to modify my patch to assert that that the allocated RP is always a child RP ? | |
| 15:03:51 | bauzas | and then, just fix the test to not fail | |
| 15:03:54 | gibi | 1) on your patch I only asked for a tracker for the cleanup so you don't need to change anything there | |
| 15:04:32 | gibi | 2) when thinking about the removal of the `allocated_rp.parent_uuid is None:` conditional | |
| 15:05:13 | gibi | I assume that you need that conditional becuase the functional reshape test sets up the from state (with VGPU on root RP) while nova is running and therefore you need the conditional | |
| 15:05:28 | gibi | is it so? | |
| 15:06:51 | gibi | OR, does _allocate_mdevs called during reshape itself? | |
| 15:07:16 | gibi | and during reshape you need the conditional | |
| 15:07:38 | gibi | anyhow this is not super important now. We have a tracker, we will get back to this when we have time to clean up | |
| 15:09:45 | bauzas | sorry, was filing the blueprint | |
| 15:09:48 | bauzas | https://blueprints.launchpad.net/nova/+spec/drop-vgpu-reshape-support | |
| 15:10:19 | gibi | thanks! | |
| 15:10:35 | bauzas | yeah, that's correct, the reshape test assumes to create a mdev *before* the reshape | |
| 15:10:45 | bauzas | so we can reshape the allocation | |
| 15:10:53 | bauzas | hence the crazy conditional | |
| 15:11:39 | gibi | OK. so hypotetically we could try to set up the from state purely in DB / placement and only start up the nova-compute service after it. So we won't try to create an mdev with nova in the from state | |
| 15:12:08 | bauzas | I would need to read the functional test in question but I get your point | |
| 15:12:08 | gibi | but I guess it is better use of our time to remove the whole reshape support | |
| 15:12:34 | bauzas | that's actually maybe something to test quickly | |
| 15:12:55 | bauzas | I can write a POC on top of that patch | |
| 15:13:16 | gibi | if you have time :) | |
| 15:13:21 | bauzas | as a FUP, that'd remove the logic and then we could see what exactly to do in the functest | |
| 15:13:46 | bauzas | gibi: I can try to spend a couple of hours on that, I'm pretty done with implementing stuff now | |
| 15:13:58 | bauzas | my left tasks are mostly rebases and reviews | |
| 15:14:07 | bauzas | + the mtty patch | |
| 15:14:52 | bauzas | gibi: thanks for your idea, I'll try it ;) | |
| 15:15:14 | gibi | cool :) | |
| 16:10:54 | bauzas | sean-k-mooney: tbc, you're -1.9999 because of the documentation for https://review.opendev.org/c/openstack/nova/+/845757 ? | |
| 16:12:03 | bauzas | I'm asking for that due to the fact you're OK with the functest https://review.opendev.org/c/openstack/nova/+/845747/3 | |
| 16:12:24 | bauzas | what I could do is try to test with two different custom rcs | |
| 16:12:37 | bauzas | so we could document that this way | |
| 16:24:10 | kashyap | tobias-urdin: Thanks for the respin here: https://review.opendev.org/c/openstack/nova/+/879021. Will look on Mon. (I'm buried in another context now) | |
| 16:40:13 | opendevreview | Sylvain Bauza proposed openstack/nova master: Support multiple allocations for vGPUs https://review.opendev.org/c/openstack/nova/+/845757 | |
| 16:44:34 | bauzas | sean-k-mooney: updated based on your concerns of the portability of group_policy ^ | |
| 16:45:06 | bauzas | I still think we want to document this as this is simpler for operators if they don't give a shit about other nested resource classes | |
| 16:45:34 | bauzas | if you look at the generated docs, they will see a big red warning about using group_policy | |
| 17:08:48 | sean-k-mooney | bauzas: i thikn we shoudl be removing all supprot for group_policy in the flavor | |
| 17:09:33 | sean-k-mooney | and in placement in a new placment microverison | |
| 17:09:49 | sean-k-mooney | isolsate beween sepicifc groups is fine | |
| 17:09:54 | sean-k-mooney | but not how it currently works | |
| 17:14:31 | sean-k-mooney | bauzas: regarding the funtional test i woudl personally prefer if you provided fucntioal tests usign custom resouce classes and/or traits too | |
| 17:14:47 | bauzas | have you seen my new revision ? | |
| 17:15:30 | sean-k-mooney | of the followup patch yes im still -1.999 | |
| 17:15:41 | bauzas | I tend to disagree with you | |