Tuesday, 2024-01-02

opendevreviewXuQi proposed openstack/cinder master: Fujitsu Driver: Support for update migrated volume  https://review.opendev.org/c/openstack/cinder/+/90114603:36
opendevreviewRajat Dhasmana proposed openstack/cinder master: DNM: Pin boto3 to 1.28.63 (released on 13th oct)  https://review.opendev.org/c/openstack/cinder/+/90446906:37
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: DNM: test s3 job with boto3<=1.28.63  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/90447206:43
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: DNM: test s3 job with boto3<=1.28.63  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/90447206:52
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: DNM: test s3 job with boto3<=1.28.63  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/90447206:58
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Make cinder-tempest-plugin-cbak-s3 job non-voting  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/90447508:38
opendevreviewRajat Dhasmana proposed openstack/cinder-tempest-plugin master: Make cinder-tempest-plugin-cbak-s3 job non-voting  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/90447508:41
opendevreviewDr. Jens Harbott proposed openstack/cinder master: Allow quota sync to actually fix quota issues  https://review.opendev.org/c/openstack/cinder/+/90448710:55
fricklerstephenfin: while you may relax about the previous bugs, ^^ actually seems to have been caused by you ;) would be cool if you could double check my findings and possible even help with the testing10:57
stephenfinfrickler: ugh, yeah, looks correct. I must have missed the 'do_fix' parameter10:59
stephenfinAs an aside, that function is also poorly named IMO. It's not obvious that it would "fix" stuff based on variable names10:59
stephenfinfrickler: Would it make sense to rename the functions to e.g. '_sync_quotas' and '_sync_project_quotas' and negate and rename the 'do_fix' variable to e.g. 'dry_run'?11:01
fricklerstephenfin: sounds plausible to me, yes11:02
stephenfinCool. If you're already working on that I'm happy to review a change, otherwise I can propose one later11:02
stephenfin+1 on the existing one btw11:03
fricklerstephenfin: feel free to take over, I've been more than deep enough in this rabbit hole for my taste11:14
fricklerintegrated CI job seems to be pretty broken, certainly all recurring errors there are unrelated on my recent patches13:45
opendevreviewAbhinav Shrivastava proposed openstack/cinder master: Handling NoneType for ssh related commands  https://review.opendev.org/c/openstack/cinder/+/90449615:03
opendevreviewBryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots  https://review.opendev.org/c/openstack/cinder/+/89905118:00
opendevreviewBryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots  https://review.opendev.org/c/openstack/cinder/+/89905118:13
opendevreviewRahman Muhammad proposed openstack/cinder master: Create multiple snapshots from same volume simultaneously, Increasing retry timeout for http status-code 400, 500, and 503  https://review.opendev.org/c/openstack/cinder/+/90452720:25
opendevreviewRahman Muhammad proposed openstack/cinder master: Create multiple snapshots from same volume simultaneously, Increasing retry timeout for http status-code 400, 500, and 503  https://review.opendev.org/c/openstack/cinder/+/90453020:58
opendevreviewRahman Muhammad proposed openstack/cinder master: Create multiple snapshots from same volume simultaneously, Increasing retry timeout for http status-code 400, 500, and 503  https://review.opendev.org/c/openstack/cinder/+/90453121:53

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