caiqilong | rosmaita: 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 |
---|---|---|
opendevreview | Supriya Kotwal proposed openstack/cinder master: DSM is unable to fetch lowercase WWNs in SC. https://review.opendev.org/c/openstack/cinder/+/886612 | 05:25 |
caiqilong | rajatdhasmana: 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_Kotwal | Hi 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 |
caiqilong | Supriya_Kotwal: what their time region. i am at utc +8 | 08:11 |
*** shoffmann is now known as Guest4853 | 08:43 | |
*** shoffmann1 is now known as shoffmann | 08:43 | |
opendevreview | Senthil Vasudevan proposed openstack/cinder master: DNM: Create for temporary test https://review.opendev.org/c/openstack/cinder/+/885435 | 09: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 geguileo | 09:26 | |
Supriya_Kotwal | @caiqilong - I am at GMT+5:30 | 09:31 |
*** denji is now known as list | 10:29 | |
*** list is now known as amir | 10:30 | |
amir | hi 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 |
amir | I know that we need to keep snapshots in source image in order to be able to have incremental backups | 10:58 |
amir | but I was wondering if we can have the ability to always keep full backups | 11:01 |
amir | for 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 |
amir | any comments? | 11:04 |
opendevreview | caiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS https://review.opendev.org/c/openstack/cinder/+/886942 | 11:52 |
opendevreview | Merged openstack/python-cinderclient stable/wallaby: [stable-only] Pin tox <4 https://review.opendev.org/c/openstack/python-cinderclient/+/869267 | 15:25 |
opendevreview | Merged openstack/os-brick master: LVM: Remove outdated comment https://review.opendev.org/c/openstack/os-brick/+/879840 | 16:29 |
opendevreview | Merged openstack/cinder master: Deadlock prevention support in synchronize https://review.opendev.org/c/openstack/cinder/+/848899 | 19:09 |
opendevreview | Merged openstack/cinder stable/ussuri: Add warning message about slow volume backend https://review.opendev.org/c/openstack/cinder/+/861057 | 19:09 |
opendevreview | Merged openstack/cinder stable/zed: Update url of "Unity Replication White Paper" https://review.opendev.org/c/openstack/cinder/+/878602 | 19:09 |
opendevreview | Merged openstack/cinder master: db: Remove the legacy 'migration_version' table https://review.opendev.org/c/openstack/cinder/+/874242 | 19:31 |
akawai | Hi, 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/!