| agalica | I might be coming in hot on no sleep for the festival tomorrow, haha | 00:31 |
|---|---|---|
| *** mhen_ is now known as mhen | 02:17 | |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM - do not merge. this is test patch https://review.opendev.org/c/openstack/cinder/+/969373 | 04:26 |
| opendevreview | Markus Hentsch proposed openstack/cinder-specs master: Re-propose Spec for the Cinder part of the Image Encryption https://review.opendev.org/c/openstack/cinder-specs/+/964777 | 09:50 |
| mhen | ^ minor update to the image encryption spec for Cinder to make sure it's 100% in sync with all the most recent changes and also polished some of the phrasing and terminology in preparation for today's review festival | 09:52 |
| * mhen was focusing on the implementation side recently and neglected the spec a bit | 09:54 | |
| opendevreview | Max proposed openstack/cinder master: feat: use image multipart download https://review.opendev.org/c/openstack/cinder/+/969493 | 10:49 |
| opendevreview | Mykhailo Ivanchenko proposed openstack/cinder-tempest-plugin master: add missing deletion for the second volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/969908 | 12:24 |
| opendevreview | Lucas Lafontaine proposed openstack/cinder master: Fujitsu driver: Improve multi-attach volume detach robustness https://review.opendev.org/c/openstack/cinder/+/969910 | 12:41 |
| opendevreview | Mykhailo Ivanchenko proposed openstack/cinder-tempest-plugin master: Add missing deletion for the second volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/969908 | 12:42 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM - do not merge. this is test patch https://review.opendev.org/c/openstack/cinder/+/969373 | 12:57 |
| jbernard | rosmaita, all: we have snow and school is closed, ill be on the call but im expecting some interruptions | 13:20 |
| rosmaita | jbernard: ack, just got in from shoveling | 13:59 |
| agalica | we don't have snow in these parts | 14:03 |
| opendevreview | Markus Hentsch proposed openstack/cinder master: Adopt image encryption standardization https://review.opendev.org/c/openstack/cinder/+/926298 | 14:04 |
| erlon | jbernard: rosmaita: are we having the review festval today? | 14:12 |
| erlon | or my calendar is wrong once again | 14:12 |
| rosmaita | must be your calendar! | 14:12 |
| rosmaita | erlon: https://meet.google.com/ijz-ivqg-bam | 14:13 |
| rosmaita | reminder: Cinder Festival of Reviews today, 1400-1600 UTC | 14:14 |
| rosmaita | join us in Google meet: https://meet.google.com/ijz-ivqg-bam | 14:14 |
| rosmaita | or go directly to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:14 |
| rosmaita | jungleboyj rosmaita tosky whoami-rajat m5z e0ne geguileo eharney jbernard sfernand hemna fabiooliveira happystacker sfv880 bryanneumann agalica erlon lutimura ^^ | 14:14 |
| rosmaita | (apologies for the late reminder) | 14:14 |
| *** max is now known as Guest32968 | 14:23 | |
| opendevreview | Dr. Jens Harbott proposed openstack/cinder master: Update cinder for Gazpacho Python Test Interface https://review.opendev.org/c/openstack/cinder/+/968480 | 14:25 |
| rosmaita | mhen: left comments for you on https://review.opendev.org/c/openstack/cinder/+/968480/3 | 15:35 |
| rosmaita | (i forgot to refresh my browser, so the comments are on patch set 3 ... mostly still relevant though) | 15:35 |
| mhen | rosmaita: thank you! It seems three of your comments reference an older version of the spec. | 15:35 |
| rosmaita | yeah, sorry about that | 15:36 |
| rosmaita | i thought we weren't going to do the qcow2+luks thing? | 15:36 |
| mhen | rosmaita: what in particular were we not going to do? | 15:37 |
| mhen | the implementation patchset already includes conversion from qcow2+luks to raw luks, so Cinder can consume those | 15:37 |
| mhen | it won't be able to produce that format though | 15:37 |
| rosmaita | ok | 15:37 |
| rosmaita | how are these being represented in image properties? | 15:38 |
| rosmaita | ok, you explain that in the glance spec | 15:39 |
| mhen | qcow2+luks is: container_format=bare + disk_format=qcow2 + os_encrypt_* properties (+ qcow2 header includes metadata about qcow2's native LUKS encryption feature) | 15:39 |
| mhen | luks is: container_format=luks + disk_format=(raw or gpt) + os_encrypt_* properties | 15:40 |
| rosmaita | 0k, thanks | 15:40 |
| mhen | rosmaita: I responded to most of your comments, I'll do an update to spec on Monday | 16:07 |
| rosmaita | mhen: sounds good! | 16:08 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!