Thursday, 2023-07-13

*** geguileo is now known as Guest584207:29
*** Guest5842 is now known as geguileo07:39
*** JohnnyW52 is now known as JohnnyW508:34
opendevreviewtomasz proposed openstack/cinder master: [HPE] Failed to create multiple instances  https://review.opendev.org/c/openstack/cinder/+/88823709:07
opendevreviewcaiqilong proposed openstack/cinder master: Add Cinder driver for TOYOU NetStor TYDS  https://review.opendev.org/c/openstack/cinder/+/88694210:16
opendevreviewRaghavendra Tilay proposed openstack/cinder master: HPE: Fix error during retype of volume without comment  https://review.opendev.org/c/openstack/cinder/+/88544012:16
sfv880Hello 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
opendevreviewToheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples  https://review.opendev.org/c/openstack/cinder/+/88681813:26
opendevreviewRenan Vitor proposed openstack/cinder stable/wallaby: DNM: Wallaby CI Test  https://review.opendev.org/c/openstack/cinder/+/88846813:52
caiqilongwhoami-rajat: hello, we proposed a new patch to resolve your concerns. Could you please take a look when you have time ?14:03
whoami-rajatcaiqilong, added a comment, looks like the create volume from snapshot case isn't handled yet14:18
caiqilongwhoami-rajat: feel sorry that the case handled at the line 660~66314:34
opendevreviewEric Harney proposed openstack/cinder master: Coerce booleans to integer values in paginate_query  https://review.opendev.org/c/openstack/cinder/+/88829614:35
whoami-rajatcaiqilong, sorry i might have missed that since i was in a diff view of PS9 to PS12, will update my comment, thanks14:35
caiqilongthanks.14:38
opendevreviewSofia Enriquez proposed openstack/cinder master: Ceph: Fix restoring old backups to a different backend  https://review.opendev.org/c/openstack/cinder/+/88452614:48
sfv880Hello whoami-rajat and enriquetaso, thank you very much for the review, you made my day!!!14:53
enriquetasosfv880++14:54
opendevreviewToheeb Oyekola proposed openstack/cinder master: Add api-ref functional tests for backup actions  https://review.opendev.org/c/openstack/cinder/+/88660915:16
enriquetasothanks toheeb_oyekola ^ 15:20
*** dviroel_ is now known as dviroel15:22
*** Saravanan-NetApp is now known as msaravan15:31
toheeb_oyekolayou're welcome enriquetaso16:50
opendevreviewToheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples  https://review.opendev.org/c/openstack/cinder/+/88849217:53
opendevreviewMerged openstack/cinder stable/yoga: Fix Infinidat driver to use TLS/SSL communication  https://review.opendev.org/c/openstack/cinder/+/88606718:27
opendevreviewToheeb Oyekola proposed openstack/cinder master: Automate generation of backups api-ref samples  https://review.opendev.org/c/openstack/cinder/+/88849219:08
opendevreviewEric Harney proposed openstack/os-brick master: mypy: improve executor coverage  https://review.opendev.org/c/openstack/os-brick/+/85378919:31
sfernandhey guys, is it too late to give a few -1s to the drivers patches? 😅19:37
sfernandI've dropped some comments to the TOYOU TYDS patch already. I think I'm half the way now19:40
sfernand Looks like they don't have a bp19:40

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