Friday, 2025-12-12

*** mhen_ is now known as mhen02:10
opendevreviewMerged openstack/cinder-specs master: Remove unnecessary explicit unicode literal strings  https://review.opendev.org/c/openstack/cinder-specs/+/85118104:21
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats  https://review.opendev.org/c/openstack/cinder/+/97065307:37
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079608:35
zigoWhat's the team opinion about this concept of heartbeat enhancement?!?08:46
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079609:12
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079609:16
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats  https://review.opendev.org/c/openstack/cinder/+/97065310:33
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats  https://review.opendev.org/c/openstack/cinder/+/97065310:58
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079611:03
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079612:45
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079612:53
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079612:54
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079613:45
mhenI just stumbled upon `image_volume_cache_enabled` in Cinder on a DevStack while doing things unrelated to image encryption but now I wonder how does that relate to image encryption?13:48
mhenon second thought, it should not make a difference as copying the same LUKS blocks from image or a local volume copy with a cloned secret resembling the same key would end up being the same and encryption is not touched in any case13:53
mhennevermind then, just thinking out loud here13:53
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079613:55
jbernardrosmaita: we need to approve the openapi spec (https://review.opendev.org/c/openstack/cinder-specs/+/963934) as the patchset is nearing completion14:17
rosmaitaack14:17
jbernardrosmaita: we'll need https://review.opendev.org/c/openstack/cinder-specs/+/962325 as well (easy one)14:25
rosmaitaack again14:25
jbernardhave you had a chance to look at either of these?14:26
jbernardhttps://review.opendev.org/c/openstack/cinder-specs/+/95883814:26
jbernardhttps://review.opendev.org/c/openstack/cinder-specs/+/96230614:26
rosmaitaprobably not? i was setting aside this morning for spec reviews, so these reminders are helpful14:27
jbernardalso, any idea if rajat has interest in retargetting https://review.opendev.org/c/openstack/cinder-specs/+/93158114:27
opendevreviewThomas Goirand proposed openstack/cinder master: WIP, do not merge: enhance heartbeats take 2  https://review.opendev.org/c/openstack/cinder/+/97079614:28
jbernardalso also, is this still doable https://review.opendev.org/c/openstack/cinder-specs/+/926307 that might be an easy one that i can work on14:29
jbernardrosmaita: ^14:29
rosmaitaabout 931581, i'd say probably; i thought we had already merged code, but i don't remember those new options, so i guess not14:31
rosmaitai think you should go ahead and ninja in https://review.opendev.org/c/openstack/cinder-specs/+/926307 , it's going into 'untargeted'14:34
jbernardshould i move it to 2026.1 when it's complete?14:35
jbernardi dont think im familiar with the untargeted spec workflow14:36
opendevreviewMerged openstack/cinder-specs master: Add spec folder structure for 2026.1 release  https://review.opendev.org/c/openstack/cinder-specs/+/96232514:38
rosmaitajbernard: there is a note on the specs page explaing: https://specs.openstack.org/openstack/cinder-specs/#untargeted-specs14:44
jbernardrosmaita: perfect, thanks15:00
jbernardrosmaita: and since you're at the keyboard :) we're dangerously close to completing the openapi set: https://review.opendev.org/q/topic:%22openapi%22+project:openstack/cinder15:02
opendevreviewMerged openstack/cinder master: api: Simplify enable/disable APIs (services)  https://review.opendev.org/c/openstack/cinder/+/96583815:03
opendevreviewMerged openstack/cinder-specs master: Add rbd snapshot mirroring spec  https://review.opendev.org/c/openstack/cinder-specs/+/92630715:08
opendevreviewFlorian proposed openstack/cinder master: Add different backup scheduling logic option  https://review.opendev.org/c/openstack/cinder/+/93551115:54
raghavendrathi jungleboyj: are you around ?15:58
raghavendrathi core team,16:26
raghavendratIt would be great to receive feedback on below Dalmatian backport. It has one +2.16:26
raghavendratHPE 3par - skip license check for new wsapi16:27
raghavendrathttps://review.opendev.org/c/openstack/cinder/+/96455816:27
raghavendratThanks16:27
jbernardzigo: just read your mail, did you not see my earlier response?16:40
raghavendratthank you Jon for +2 and +W. Appreciate your quick help16:42
opendevreviewMerged openstack/cinder-specs master: Re-propose openapi spec  https://review.opendev.org/c/openstack/cinder-specs/+/96393416:42
opendevreviewMerged openstack/cinder stable/2024.2: HPE 3par - skip license check for new wsapi  https://review.opendev.org/c/openstack/cinder/+/96455818:08
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily at 20:00 UTC (approximately 55 minutes from now) for a project rename maintenance: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/ZPIV7CTPXZUYKUCBYMQ3OKQZDXOSOQGF/19:08
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a project rename maintenance: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/ZPIV7CTPXZUYKUCBYMQ3OKQZDXOSOQGF/20:01
opendevreviewMerged openstack/cinder master: api: Merge os-services extension  https://review.opendev.org/c/openstack/cinder/+/96583920:34
opendevreviewMerged openstack/cinder master: api: Rename controllers  https://review.opendev.org/c/openstack/cinder/+/96584020:34
opendevreviewMerged openstack/os-brick master: reno: Update master for unmaintained/2024.1  https://review.opendev.org/c/openstack/os-brick/+/96547821:44
opendevreviewIvan Anfimov proposed openstack/cinder master: Remove installation guide for openSUSE/SLES  https://review.opendev.org/c/openstack/cinder/+/94876621:44
opendevreviewMerged openstack/cinder master: api: Add remaining missing request body schemas  https://review.opendev.org/c/openstack/cinder/+/96584123:47
opendevreviewMerged openstack/cinder master: api: Remove unnecessary validation helpers  https://review.opendev.org/c/openstack/cinder/+/96584223:59
opendevreviewMerged openstack/cinder master: tests: Ensure all APIs have a request body schema  https://review.opendev.org/c/openstack/cinder/+/96584323:59

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!