| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-13 | |||
| 17:58:10 | sean-k-mooney | dansmith: by the way if you have time to start revieing melwitt serise i think most of the patches are in good shape | |
| 17:58:30 | sean-k-mooney | https://review.opendev.org/q/topic:%22specs/yoga/approved/ephemeral-encryption-libvirt%22+status:open+project:openstack/nova | |
| 17:58:56 | dansmith | yeah it's kinda half on my list, but I'm not sure I'll really be able to dive into it in detail | |
| 18:13:05 | melwitt | sean-k-mooney: ack, thanks for testing that | |
| 18:44:02 | bauzas | sean-k-mooney, dansmith : ok, I'm eventually done with testing my series, I'll provide the asciinema gif tomorrow | |
| 18:44:23 | dansmith | bauzas: updates needed to the set or good as is? | |
| 18:52:04 | sean-k-mooney | i started looking at the serise a littel to day but likely wont have time to review it till tomorrow | |
| 18:52:24 | sean-k-mooney | going though the vgpu serise is on my todo list for tomorrow before i loop back to melainies series | |
| 19:05:52 | bauzas | dansmith: nope, no need to change, we just need to set a large number for downtime but you will see it tomorrow ;-) | |
| 19:06:02 | dansmith | cool | |
| 19:06:57 | bauzas | the reason that qemu doesn't know about the right GPU memory, it just thinks that it needs to migrate 4GB | |
| 19:07:28 | dansmith | ack | |
| 19:27:49 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Add todo to remove HyperVLiveMigrateData object https://review.opendev.org/c/openstack/nova/+/906636 | |
| 19:28:09 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console connection information API https://review.opendev.org/c/openstack/nova/+/906991 | |
| 19:28:16 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console API https://review.opendev.org/c/openstack/nova/+/906809 | |
| 19:28:22 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove extra specs of HyperV driver https://review.opendev.org/c/openstack/nova/+/906992 | |
| 19:28:56 | gmann | dansmith: sean-k-mooney ^^ keeping hyperv object and rebased API change, please check | |
| 19:29:06 | dansmith | gmann: already done | |
| 19:29:18 | gmann | thanks, so fast :) | |
| 19:29:36 | sean-k-mooney | im just leaveing but ill do that then drop for today | |
| 19:30:27 | gmann | sean-k-mooney: thanks, and sorry for making it late for you | |
| 19:30:49 | sean-k-mooney | its normally not laste but i got up very early today | |
| 19:30:58 | sean-k-mooney | so no worries | |
| 19:37:41 | sean-k-mooney | gmann: ok im +2 on all of those they look ok assuming they also pass ci | |
| 19:47:13 | opendevreview | Merged openstack/nova stable/2023.1: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/908806 | |
| 20:13:55 | spatel | I am getting strange nova-novncproxy error and not sure what is wrong. Everything was working but after restart nova-novncproxy it started throwing error - https://paste.opendev.org/show/baZjIpTcibpksgajegav/ | |
| 20:25:15 | opendevreview | Ghanshyam proposed openstack/nova master: HyperV: Remove RDP console API https://review.opendev.org/c/openstack/nova/+/906809 | |
| 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: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 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: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: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:43 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 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: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: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 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:47 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 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: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: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:51 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 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:53 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 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:55 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 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:57 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 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:59 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 07:37:59 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 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:01 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 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: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: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: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 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: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:36 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 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:37 | opendevreview | melanie witt proposed openstack/nova master: libvirt: make |
|
| 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 rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 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: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: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: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: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: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: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: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: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/908825 | |
| 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:39 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 10:26:39 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 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 | |