Monday, 2023-07-03

caiqilongrosmaita: hello, my third-party-ci meet some failed test cases, i am working to resolve it, but now it's approaching the new driver merge deadline. I would like to ask if there can be a few more days if there is not enough time.01:43
opendevreviewSupriya Kotwal proposed openstack/cinder master: DSM is unable to fetch lowercase WWNs in SC.  https://review.opendev.org/c/openstack/cinder/+/88661205:25
caiqilongrajatdhasmana: hello, my third-party-ci meet some failed test cases, i am working to resolve it, but now it's approaching the new driver merge deadline. I would like to ask if there can be a few more days if there is not enough time.05:46
Supriya_KotwalHi Sofia Enriquez, Vladislav Belogrudov, Sean McGinnis  I have update commit message for https://review.opendev.org/c/openstack/cinder/+/886612/. It will be very great and helpful for us if you all review it again.08:03
caiqilongSupriya_Kotwal: what their time region. i am at utc +808:11
*** shoffmann is now known as Guest485308:43
*** shoffmann1 is now known as shoffmann08:43
opendevreviewSenthil Vasudevan proposed openstack/cinder master: DNM: Create for temporary test  https://review.opendev.org/c/openstack/cinder/+/88543509:07
sfv880_Hello, could you please review backports https://review.opendev.org/c/openstack/cinder/+/886067 and https://review.opendev.org/c/openstack/cinder/+/886141 ? Thank you very much!09:11
*** akrog is now known as geguileo09:26
Supriya_Kotwal@caiqilong - I am at GMT+5:3009:31
*** denji is now known as list10:29
*** list is now known as amir10:30
amirhi folks. I was playing with cinder's ceph backup driver and realized that if u wanna back up between multiple clusters, it creates an snapshot on source image and then transfer it to dest cluster. which means that we'll have snapshots on source and dest.10:57
amirI know that we need to keep snapshots in source image in order to be able to have incremental backups10:58
amirbut I was wondering if we can have the ability to always keep full backups11:01
amirfor full backups we dont need to keep all the snapshots. which means that we can remove source image's snapshot in this case.11:03
amirany comments?11:04
opendevreviewcaiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS  https://review.opendev.org/c/openstack/cinder/+/88694211:52
opendevreviewMerged openstack/python-cinderclient stable/wallaby: [stable-only] Pin tox <4  https://review.opendev.org/c/openstack/python-cinderclient/+/86926715:25
opendevreviewMerged openstack/os-brick master: LVM: Remove outdated comment  https://review.opendev.org/c/openstack/os-brick/+/87984016:29
opendevreviewMerged openstack/cinder master: Deadlock prevention support in synchronize  https://review.opendev.org/c/openstack/cinder/+/84889919:09
opendevreviewMerged openstack/cinder stable/ussuri: Add warning message about slow volume backend  https://review.opendev.org/c/openstack/cinder/+/86105719:09
opendevreviewMerged openstack/cinder stable/zed: Update url of "Unity Replication White Paper"  https://review.opendev.org/c/openstack/cinder/+/87860219:09
opendevreviewMerged openstack/cinder master: db: Remove the legacy 'migration_version' table  https://review.opendev.org/c/openstack/cinder/+/87424219:31
akawaiHi, cores. I would like to ask about 'Zuul', as reviewer. My patch https://review.opendev.org/c/openstack/cinder/+/885986 is not added the reviewer 'Zuul'. Although I know the reviewer will be added automatically, run Zuul test and give +1, my patch could not run the test and get +1 by Zuul. I've tried to add 'Zuul', but it doesn't work. So, could you add the reviewer 'Zuul' to my patch? Or, Could my patch be merged without +1 by Zuul test? 23:32

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