| *** mhen_ is now known as mhen | 02:00 | |
| *** trident- is now known as trident | 09:49 | |
| tkajinam | o/ I wonder if update of metadata can be accepted as an exception during FF period ? The nova feature was merged at the last-minute before its FF data and I'm hoping to get this merged to glance for consistency. https://review.opendev.org/c/openstack/glance/+/959984 | 10:25 |
|---|---|---|
| tkajinam | update of *metadef* I mean | 10:25 |
| dansmith | I defer to the PTO but metadefs are not code so it seems (IMHO) fine to lump them in with bug fixes and test changes, up to rc1 | 13:49 |
| dansmith | PTO? PTL..maybe I need some PTO :D | 13:58 |
| frickler | iiuc the PTL is on PTO, that might explain the confusion ;) | 14:02 |
| dansmith | abhi is, but I didn't think croelandt was (the former is the PTL) | 14:04 |
| croelandt | tkajinam: we're not doing rc1 iirc | 15:26 |
| croelandt | oh no actually we are | 15:29 |
| croelandt | yeah OK i can probably merge this and release rc1 then | 15:29 |
| tkajinam | yeah rc1 is supposed to be released in a few days | 15:31 |
| croelandt | dansmith: does the patch look good to our resident Nova expert? | 15:32 |
| croelandt | tkajinam: not sure why the reference to KVM disappeared | 15:32 |
| croelandt | does AMD SEV replace KVM in Nova or smoething? | 15:32 |
| tkajinam | no. AMD SEV is used via qemu+KVM | 15:33 |
| dansmith | croelandt: tkajinam is your resident nova expert for this feature :D | 15:34 |
| tkajinam | X-D | 15:34 |
| tkajinam | croelandt, https://review.opendev.org/c/openstack/nova/+/781437 | 15:34 |
| tkajinam | update of the link follows that doc restructure merged a few years back | 15:34 |
| croelandt | tkajinam: but can you use KVM without AMD SEV? | 15:35 |
| tkajinam | yes | 15:35 |
| croelandt | I wonder if it makes sense to have KVM mentioned for hw_mem_encryption then | 15:35 |
| tkajinam | AMD SEV is a kind of additional feature | 15:35 |
| croelandt | see what I mean? | 15:35 |
| tkajinam | ah, ok | 15:35 |
| croelandt | like, let's say I care about KVM but don't know about AMD SEV | 15:36 |
| croelandt | I'm going to be a bit confused here | 15:36 |
| croelandt | so maybe we mention the docs for AMD SEV in the hw_mem_encryption_model section | 15:36 |
| croelandt | but we keep a link to KVM-specific doc for hw_mem_encryption? | 15:36 |
| croelandt | does that make sense? | 15:37 |
| opendevreview | Takashi Kajinami proposed openstack/glance master: Add new metadata definition of memory encryption model https://review.opendev.org/c/openstack/glance/+/959984 | 15:38 |
| tkajinam | croelandt, does this match your expectation ? https://review.opendev.org/c/openstack/glance/+/959984/3..4/doc/source/admin/useful-image-properties.rst | 15:38 |
| croelandt | nice, when I lazily grep for KVM I'll find what I'm looking for ;) | 15:40 |
| croelandt | \o/ | 15:40 |
| tkajinam | croelandt, nova no longer maintains SEV related descriptions in KVM section so we can no longer use that kvm specific link | 15:40 |
| tkajinam | so I mentioned kvm in the description, instead | 15:40 |
| croelandt | ok, I see | 15:40 |
| croelandt | tkajinam: but what about https://docs.openstack.org/nova/latest/admin/configuration/hypervisor-kvm.html ? Couldn't this be part of the section about hw_mem_encryption? | 15:42 |
| croelandt | oh no ok hw_mem_encryption is not even in KVM | 15:42 |
| tkajinam | yeah it was migrated to the top-level dedicated doc for SEV | 15:43 |
| croelandt | yeah but also technically, hw_mem_encrpytion only makes sense if you use SEV | 15:43 |
| tkajinam | yup | 15:45 |
| tkajinam | ... until we add support for Intel TDX :-P | 15:45 |
| tkajinam | (TDX is similar to SEV, but is available in Intel CPUs. Ignore it now. | 15:46 |
| croelandt | Apparently consumers have been ignoring Intel CPUs for a little while :D | 15:47 |
| croelandt | ok so I've pushed it, and I'll release rc1 once it's in | 15:49 |
| opendevreview | Markus Hentsch proposed openstack/glance master: Standardization of encrypted images https://review.opendev.org/c/openstack/glance/+/926295 | 15:51 |
| tkajinam | croelandt, thanks ! | 16:28 |
| opendevreview | Merged openstack/glance master: Add new metadata definition of memory encryption model https://review.opendev.org/c/openstack/glance/+/959984 | 20:50 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!