Wednesday, 2024-12-04

opendevreviewMerged openstack/cinder stable/2024.1: Fix "signature_verified" metadata propagation to images  https://review.opendev.org/c/openstack/cinder/+/93539000:05
*** mhen_ is now known as mhen02:11
ebgo49[m]Hello, Could you please review and provide feedback?... (full message at <https://matrix.org/oftc/media/v1/media/download/Aexl13nfng17SLoISNBJ848hk_zvDvDmSQPnbKofvxJF2pbfPeYUtlHETq7rWpejfPY--di-5qU957XeHx2R1BZCeT18RbXAAG1hdHJpeC5vcmcva3JXU3BLSFJ3WWxiU1VZY0h6R25UZGlM>)03:16
misook[m]Hello, could someone please review?... (full message at <https://matrix.org/oftc/media/v1/media/download/AV3TfBnvO6_Xds7bMMaMURzp0wdo5mssMEBIjchOW8tL7wuoJgcgfI33bm9-A3vQAwk5vHsxisCOIeYw3CiisMNCeT2GqjJQAG1hdHJpeC5vcmcvQktyeGNJZHF2SGtTYWVaUFVhaEltR1lK>)06:18
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax Driver: Added Support for NVMe/TCP  https://review.opendev.org/c/openstack/cinder/+/92490208:00
*** geguileo is now known as Guest203408:08
opendevreviewPierre Riteau proposed openstack/cinder stable/2023.2: Fix "signature_verified" metadata propagation to images  https://review.opendev.org/c/openstack/cinder/+/93699308:14
*** kgube1 is now known as kgube08:51
opendevreviewNilesh Thathagar proposed openstack/cinder master: Dell PowerMax Driver: Added Support for NVMe/TCP  https://review.opendev.org/c/openstack/cinder/+/92490209:00
kevkomisook[m]: don't use matrix09:59
kevko\o/10:28
opendevreviewYosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group snapshots  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/89697911:14
opendevreviewYosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing groups  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/89697711:19
opendevreviewYosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group types  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/89660511:21
gochist[m]As far as I know, it was decided to create a Matrix usage guide during the last vPTG 'Bridging the Gap' session. However, Matrix users should avoid entering multi-line messages. In this case, it appears as a Matrix link in IRC, causing inconvenience to IRC users11:30
raghavendrathi Rajat, are you around ?13:19
*** kgube1 is now known as kgube13:45
jbernardjungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi: meeting in #openstack-meeting-alt \o/14:00
whoami-rajatthanks jbernard 14:01
kevkoHi folks , can i ask something ? Or do you still have a meeting ? 14:16
jbernardkevko: meeting at the moment, but it's IRC, so ask away and someone will read and respond if/when they can14:19
kevkookay, I have  a cinder volume backed instance with root volume, backend is only ceph, from nova api version 2.93 there is an option to rebuild instance also with volume backed root volume. What I was trying is to make a server snapshot ..which produced 0 bytes image in glance (proper snapshot of volume in cinder) ...with metadata from which14:31
kevkosnapshot it was created ...during the rebuild (openstack server rebuild --image ....)..it's going through cinder code and trying to download image ..with 0 bytes ...  in reimage function ... my question is ..how cinder wanted to deal with this ? is it supported ? is it bug ? When I changed function of cinder reimage to this ...14:31
kevkohttps://paste.openstack.org/show/bAmGLHpZ2qMRSbpuixn9/ 14:31
kevkoI mostly want to know if is it supported, if there is some plan to remove glance image connection and implement somehow new ? or idea will be the same .... because I don't know if i am going to fix the current code ..or if it is know issue or what ...14:32
kevkoThank you very much 14:32
whoami-rajatkevko, hey, that's an actual bug which i fixed here14:41
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/91640814:41
whoami-rajati need to update that patch to make it review-able14:42
whoami-rajatkevko, thanks for reminding about it and please remind me if i don't update it by next week14:42
opendevreviewJan Horstmann proposed openstack/cinder-specs master: Add spec for dm-clone based local storage driver  https://review.opendev.org/c/openstack/cinder-specs/+/93534714:45
kevkowhoami-rajat: ooh, thank you ...but can I patch it myself for now for customer ? 14:54
kevkoin downstream ? 14:54
opendevreviewRodrigo Barbieri proposed openstack/cinder stable/2023.2: Fix "signature_verified" metadata propagation to images  https://review.opendev.org/c/openstack/cinder/+/93699314:59
raghavendrathi whoami-rajat: are you around ?15:31
*** tosky_ is now known as tosky15:40
raghavendratIf you get some time, could you please re-visit below patch.16:11
raghavendrati think, i have addressed review comments.16:11
raghavendratHPE 3par - Fix retype volume16:12
raghavendrathttps://review.opendev.org/c/openstack/cinder/+/92859916:12
*** geguileo is now known as Guest206916:20
kevkowhoami-rajat: can i amend your review ? 16:45
gmannrosmaita: whoami-rajat jbernard can you check this https://review.opendev.org/c/openstack/cinder/+/93293217:33
gmannand this https://review.opendev.org/c/openstack/os-brick/+/93558717:34
whoami-rajatkevko, sure you can, and I would be glad if that testing helps improve the patch I've proposed, thanks!17:46
whoami-rajatraghey17:46
whoami-rajatkevko, you mean post update on my patch? generally it's not considered a good practice but you can add a review comment and i can work on the improvements if you have any in mind17:47
whoami-rajatgmann, done, thanks for proposing those!17:50
gmannthanks18:09
opendevreviewmelanie witt proposed openstack/cinder master: NFS update volume attachment format during volume snapshot  https://review.opendev.org/c/openstack/cinder/+/85752818:10
opendevreviewRajat Dhasmana proposed openstack/cinder master: Add 'enforce_multipath' in connection_properties  https://review.opendev.org/c/openstack/cinder/+/93401118:22
opendevreviewFrancesco Pantano proposed openstack/devstack-plugin-ceph master: DNM - Bump to squid  https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/92780618:46
opendevreviewMerged openstack/cinder master: Huawei hypermetro: RECOVERYPOLICY typo  https://review.opendev.org/c/openstack/cinder/+/92550119:30
opendevreviewGhanshyam proposed openstack/python-brick-cinderclient-ext master: DNM: testing gate  https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/93706119:33
*** elodilles is now known as elodilles_pto19:54
opendevreviewMerged openstack/os-brick master: Updating testing as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/os-brick/+/93558719:55
opendevreviewMerged openstack/cinder master: Update gate jobs as per the 2025.1 cycle testing runtime  https://review.opendev.org/c/openstack/cinder/+/93293220:03
*** geguileo is now known as Guest209022:13

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