| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-13 | |||
| 20:25:24 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove extra specs of HyperV driver https://review.opendev.org/c/openstack/nova/+/906992 | |
| 23:04:51 | gmann | dansmith: ^^ updated it to keep RPC method | |
| 23:33:42 | dansmith | gmann: ack thanks | |
| #openstack-nova - 2024-02-14 | |||
| 02:58:35 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for AMD SEV-ES https://review.opendev.org/c/openstack/os-traits/+/908966 | |
| 02:59:55 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for AMD SEV-ES https://review.opendev.org/c/openstack/os-traits/+/908966 | |
| 03:00:10 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for AMD SEV-ES https://review.opendev.org/c/openstack/os-traits/+/908966 | |
| 03:02:34 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: Add MEM_ENCRYPTED_CONTEXT_V2 resource class https://review.opendev.org/c/openstack/os-resource-classes/+/908967 | |
| 03:49:54 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: Add MEM_ENCRYPTED_CONTEXT_V2 resource class https://review.opendev.org/c/openstack/os-resource-classes/+/908967 | |
| 05:36:21 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 07:23:03 | opendevreview | Amit Uniyal proposed openstack/nova master: enforce remote console shutdown https://review.opendev.org/c/openstack/nova/+/901824 | |
| 07:37:41 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 07:37:41 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 07:37:42 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 07:37:42 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 07:37:43 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 07:37:43 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 07:37:44 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 07:37:45 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 07:37:45 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 07:37:47 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 07:37:47 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 07:37:49 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 07:37:49 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 07:37:51 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 07:37:51 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 07:37:53 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 07:37:53 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 07:37:55 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 07:37:55 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 07:37:57 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 07:37:57 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 07:37:59 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 07:37:59 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 07:38:01 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 07:38:01 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 07:38:03 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 07:38:03 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 07:38:05 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 08:04:34 | opendevreview | melanie witt proposed openstack/nova master: block_device: Add encryption attributes to swap disks https://review.opendev.org/c/openstack/nova/+/884312 | |
| 08:04:35 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 08:04:35 | 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 | |
| 08:04:36 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 08:04:36 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 08:04:37 | opendevreview | melanie witt proposed openstack/nova master: libvirt: make |
|
| 08:04:37 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 08:04:38 | opendevreview | melanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 08:04:39 | opendevreview | melanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/905512 | |
| 08:04:39 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 08:04:41 | opendevreview | melanie witt proposed openstack/nova master: Support rescue with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/873675 | |
| 08:04:41 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 08:04:43 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 08:04:43 | opendevreview | melanie witt proposed openstack/nova master: Add backing_encryption_secret_uuid to BlockDeviceMapping https://review.opendev.org/c/openstack/nova/+/907960 | |
| 08:04:45 | opendevreview | melanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| 08:04:45 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 08:04:47 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 09:02:18 | opendevreview | Sylvain Bauza proposed openstack/nova master: Modify the mdevs in the migrate XML https://review.opendev.org/c/openstack/nova/+/904258 | |
| 10:08:44 | Uggla | gibi, I pushed an updated serie this morning. Hopefully I fixed most of your comments. | |
| 10:12:39 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: Support maxphysaddr. https://review.opendev.org/c/openstack/nova/+/907516 | |
| 10:18:44 | bauzas | Uggla: I'm starting to look at your series :) | |
| 10:18:56 | Uggla | bauzas, thx | |
| 10:26:38 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 10:26:38 | opendevreview | Amit Uniyal proposed openstack/nova master: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/908825 | |
| 10:26:39 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 10:26:39 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 12:50:10 | stephenfin | Uggla: Are you happy with https://review.opendev.org/c/openstack/openstacksdk/+/893505 now? Sounds like you figured out the issues you were having using SDK for Manila so that doc is probably correct? | |
| 12:59:01 | Uggla | stephenfin, tbh not really I did not manage to use the service token. So for the moment I left it behind. | |
| 12:59:15 | stephenfin | ah, okay | |
| 13:11:24 | ganso | gibi, dansmith, sean-k-mooney: hi! could you please review this small patch when you have a few minutes? thanks in advance! https://review.opendev.org/c/openstack/nova/+/906053 | |
| 13:12:29 | sean-k-mooney | ganso: its currenlty in merge conflict so it needs a rebase | |
| 13:13:29 | ganso | sean-k-mooney: it is? I thought it was predicting a merge conflict with other unmerged patches. Let me rebase it and fix the conflits then, just a sec | |
| 13:14:01 | sean-k-mooney | ganso: no when it says merge conflcit on the left like that its not mergable with master | |
| 13:14:50 | sean-k-mooney | this is really a feature by the way but ill try and load some context | |
| 13:15:48 | ganso | sean-k-mooney: oh I was looking at a stale page. I had to ctrl+F5 for it to show up | |
| 13:15:59 | ganso | sean-k-mooney: a feature? you mean it wouldn't be backportable? | |
| 13:19:02 | opendevreview | Rodrigo Barbieri proposed openstack/nova master: Use dedicated live migration network during pre-migration https://review.opendev.org/c/openstack/nova/+/906053 | |
| 13:25:57 | sean-k-mooney | ganso: without new config option i dont think it is backportable no | |
| 13:26:13 | sean-k-mooney | ganso: any change in what network are used could break an existing deployment | |
| 13:26:35 | sean-k-mooney | ganso: im on a call by the way so i have only skimed the patch so perhaps that not a concern | |
| 13:26:54 | sean-k-mooney | we ahve a new related feature | |
| 13:27:10 | sean-k-mooney | which allows configuing what network we use for cold migration | |
| 13:27:26 | sean-k-mooney | which is i belive the same code that is deictating what network we are usign itn pre-migration | |
| 13:28:17 | ganso | sean-k-mooney: I understand the concern. I have a customer that is running into this issue. They have 2 networks, one for controller and one for migration. The bug causes the controller network to be used and the patch actually addresses the use of the correct migration network. It doesn't change their setup. | |
| 13:28:37 | sean-k-mooney | we added https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.migration_inbound_addr | |
| 13:29:15 | ganso | sean-k-mooney: yep, that is the setting my customer is using to determine the migration network. The issue is that the setting is being ignored by the pre-live-migration step | |
| 13:29:39 | sean-k-mooney | ganso: migration_inbound_addr is somethign we are adding this cycle | |
| 13:29:57 | sean-k-mooney | so unless they are using master they are suign live_migration_inbound_addr | |
| 13:30:01 | ganso | sean-k-mooney: hmmmmmm let me confirm the naming, maybe it is similar to the other one I'm referring to | |
| 13:30:02 | sean-k-mooney | which is currently only for the libvirt part | |
| 13:31:00 | ganso | sean-k-mooney: yes! live_migration_inbound_addr | |
| 13:31:00 | ganso | sean-k-mooney: yes they are using libvirt | |
| 13:31:00 | sean-k-mooney | right so for cold migration there was no way to use a diffent netwrok before now | |
| 13:31:00 | ganso | sean-k-mooney: so the pre-live-migration code is using the wrong address, derived from the same way cold migration determine its address | |
| 13:31:00 | sean-k-mooney | yep | |
| 13:31:20 | sean-k-mooney | so on master you could "fix this" by settin gboth option to use the migration network | |
| 13:31:48 | sean-k-mooney | for older release it may be valid to change but it not valid to make cold migratoin use the live migration network | |
| 13:32:07 | sean-k-mooney | based on only live_migration_inbound_addr | |
| 13:32:14 | ganso | sean-k-mooney: I am not familiar with the patch code, I just found the fix proposal that I need, but I am a bit surprised that the fix needs an object versioning bump to make it use the correct network config instead of the wrong one | |
| 13:32:45 | ganso | sean-k-mooney: certainly, wouldn't make sense the change the cold-migration behavior, that indeed would be a new feature | |
| 13:32:48 | sean-k-mooney | so its not clear its using the wrong netwrok | |
| 13:33:02 | sean-k-mooney | techncially the exisitng option is only for the libvirt traffic | |