opendevreview | Merged openstack/cinder stable/2024.1: Fix "signature_verified" metadata propagation to images https://review.opendev.org/c/openstack/cinder/+/935390 | 00:05 |
---|---|---|
*** mhen_ is now known as mhen | 02: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 |
opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax Driver: Added Support for NVMe/TCP https://review.opendev.org/c/openstack/cinder/+/924902 | 08:00 |
*** geguileo is now known as Guest2034 | 08:08 | |
opendevreview | Pierre Riteau proposed openstack/cinder stable/2023.2: Fix "signature_verified" metadata propagation to images https://review.opendev.org/c/openstack/cinder/+/936993 | 08:14 |
*** kgube1 is now known as kgube | 08:51 | |
opendevreview | Nilesh Thathagar proposed openstack/cinder master: Dell PowerMax Driver: Added Support for NVMe/TCP https://review.opendev.org/c/openstack/cinder/+/924902 | 09:00 |
kevko | misook[m]: don't use matrix | 09:59 |
kevko | \o/ | 10:28 |
opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group snapshots https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896979 | 11:14 |
opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing groups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896977 | 11:19 |
opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group types https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896605 | 11: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 users | 11:30 |
raghavendrat | hi Rajat, are you around ? | 13:19 |
*** kgube1 is now known as kgube | 13:45 | |
jbernard | jungleboyj 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-rajat | thanks jbernard | 14:01 |
kevko | Hi folks , can i ask something ? Or do you still have a meeting ? | 14:16 |
jbernard | kevko: meeting at the moment, but it's IRC, so ask away and someone will read and respond if/when they can | 14:19 |
kevko | okay, 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 which | 14:31 |
kevko | snapshot 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 |
kevko | https://paste.openstack.org/show/bAmGLHpZ2qMRSbpuixn9/ | 14:31 |
kevko | I 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 |
kevko | Thank you very much | 14:32 |
whoami-rajat | kevko, hey, that's an actual bug which i fixed here | 14:41 |
whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/916408 | 14:41 |
whoami-rajat | i need to update that patch to make it review-able | 14:42 |
whoami-rajat | kevko, thanks for reminding about it and please remind me if i don't update it by next week | 14:42 |
opendevreview | Jan Horstmann proposed openstack/cinder-specs master: Add spec for dm-clone based local storage driver https://review.opendev.org/c/openstack/cinder-specs/+/935347 | 14:45 |
kevko | whoami-rajat: ooh, thank you ...but can I patch it myself for now for customer ? | 14:54 |
kevko | in downstream ? | 14:54 |
opendevreview | Rodrigo Barbieri proposed openstack/cinder stable/2023.2: Fix "signature_verified" metadata propagation to images https://review.opendev.org/c/openstack/cinder/+/936993 | 14:59 |
raghavendrat | hi whoami-rajat: are you around ? | 15:31 |
*** tosky_ is now known as tosky | 15:40 | |
raghavendrat | If you get some time, could you please re-visit below patch. | 16:11 |
raghavendrat | i think, i have addressed review comments. | 16:11 |
raghavendrat | HPE 3par - Fix retype volume | 16:12 |
raghavendrat | https://review.opendev.org/c/openstack/cinder/+/928599 | 16:12 |
*** geguileo is now known as Guest2069 | 16:20 | |
kevko | whoami-rajat: can i amend your review ? | 16:45 |
gmann | rosmaita: whoami-rajat jbernard can you check this https://review.opendev.org/c/openstack/cinder/+/932932 | 17:33 |
gmann | and this https://review.opendev.org/c/openstack/os-brick/+/935587 | 17:34 |
whoami-rajat | kevko, sure you can, and I would be glad if that testing helps improve the patch I've proposed, thanks! | 17:46 |
whoami-rajat | raghey | 17:46 |
whoami-rajat | kevko, 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 mind | 17:47 |
whoami-rajat | gmann, done, thanks for proposing those! | 17:50 |
gmann | thanks | 18:09 |
opendevreview | melanie witt proposed openstack/cinder master: NFS update volume attachment format during volume snapshot https://review.opendev.org/c/openstack/cinder/+/857528 | 18:10 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add 'enforce_multipath' in connection_properties https://review.opendev.org/c/openstack/cinder/+/934011 | 18:22 |
opendevreview | Francesco Pantano proposed openstack/devstack-plugin-ceph master: DNM - Bump to squid https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/927806 | 18:46 |
opendevreview | Merged openstack/cinder master: Huawei hypermetro: RECOVERYPOLICY typo https://review.opendev.org/c/openstack/cinder/+/925501 | 19:30 |
opendevreview | Ghanshyam proposed openstack/python-brick-cinderclient-ext master: DNM: testing gate https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/937061 | 19:33 |
*** elodilles is now known as elodilles_pto | 19:54 | |
opendevreview | Merged openstack/os-brick master: Updating testing as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/os-brick/+/935587 | 19:55 |
opendevreview | Merged openstack/cinder master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/cinder/+/932932 | 20:03 |
*** geguileo is now known as Guest2090 | 22:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!