*** geguileo is now known as Guest5842 | 07:29 | |
*** Guest5842 is now known as geguileo | 07:39 | |
*** JohnnyW52 is now known as JohnnyW5 | 08:34 | |
opendevreview | tomasz proposed openstack/cinder master: [HPE] Failed to create multiple instances https://review.opendev.org/c/openstack/cinder/+/888237 | 09:07 |
---|---|---|
opendevreview | caiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS https://review.opendev.org/c/openstack/cinder/+/886942 | 10:16 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: HPE: Fix error during retype of volume without comment https://review.opendev.org/c/openstack/cinder/+/885440 | 12:16 |
sfv880 | Hello enriquetaso, if you have a chance, 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! | 12:59 |
opendevreview | Toheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples https://review.opendev.org/c/openstack/cinder/+/886818 | 13:26 |
opendevreview | Renan Vitor proposed openstack/cinder stable/wallaby: DNM: Wallaby CI Test https://review.opendev.org/c/openstack/cinder/+/888468 | 13:52 |
caiqilong | whoami-rajat: hello, we proposed a new patch to resolve your concerns. Could you please take a look when you have time ? | 14:03 |
whoami-rajat | caiqilong, added a comment, looks like the create volume from snapshot case isn't handled yet | 14:18 |
caiqilong | whoami-rajat: feel sorry that the case handled at the line 660~663 | 14:34 |
opendevreview | Eric Harney proposed openstack/cinder master: Coerce booleans to integer values in paginate_query https://review.opendev.org/c/openstack/cinder/+/888296 | 14:35 |
whoami-rajat | caiqilong, sorry i might have missed that since i was in a diff view of PS9 to PS12, will update my comment, thanks | 14:35 |
caiqilong | thanks. | 14:38 |
opendevreview | Sofia Enriquez proposed openstack/cinder master: Ceph: Fix restoring old backups to a different backend https://review.opendev.org/c/openstack/cinder/+/884526 | 14:48 |
sfv880 | Hello whoami-rajat and enriquetaso, thank you very much for the review, you made my day!!! | 14:53 |
enriquetaso | sfv880++ | 14:54 |
opendevreview | Toheeb Oyekola proposed openstack/cinder master: Add api-ref functional tests for backup actions https://review.opendev.org/c/openstack/cinder/+/886609 | 15:16 |
enriquetaso | thanks toheeb_oyekola ^ | 15:20 |
*** dviroel_ is now known as dviroel | 15:22 | |
*** Saravanan-NetApp is now known as msaravan | 15:31 | |
toheeb_oyekola | you're welcome enriquetaso | 16:50 |
opendevreview | Toheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples https://review.opendev.org/c/openstack/cinder/+/888492 | 17:53 |
opendevreview | Merged openstack/cinder stable/yoga: Fix Infinidat driver to use TLS/SSL communication https://review.opendev.org/c/openstack/cinder/+/886067 | 18:27 |
opendevreview | Toheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples https://review.opendev.org/c/openstack/cinder/+/888492 | 19:08 |
opendevreview | Eric Harney proposed openstack/os-brick master: mypy: improve executor coverage https://review.opendev.org/c/openstack/os-brick/+/853789 | 19:31 |
sfernand | hey guys, is it too late to give a few -1s to the drivers patches? 😅 | 19:37 |
sfernand | I've dropped some comments to the TOYOU TYDS patch already. I think I'm half the way now | 19:40 |
sfernand | Looks like they don't have a bp | 19:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!