opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Support for update migrated volume https://review.opendev.org/c/openstack/cinder/+/901146 | 03:36 |
---|---|---|
opendevreview | Rajat Dhasmana proposed openstack/cinder master: DNM: Pin boto3 to 1.28.63 (released on 13th oct) https://review.opendev.org/c/openstack/cinder/+/904469 | 06:37 |
opendevreview | Rajat 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/+/904472 | 06:43 |
opendevreview | Rajat 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/+/904472 | 06:52 |
opendevreview | Rajat 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/+/904472 | 06:58 |
opendevreview | Rajat 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/+/904475 | 08:38 |
opendevreview | Rajat 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/+/904475 | 08:41 |
opendevreview | Dr. Jens Harbott proposed openstack/cinder master: Allow quota sync to actually fix quota issues https://review.opendev.org/c/openstack/cinder/+/904487 | 10:55 |
frickler | stephenfin: 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 testing | 10:57 |
stephenfin | frickler: ugh, yeah, looks correct. I must have missed the 'do_fix' parameter | 10:59 |
stephenfin | As an aside, that function is also poorly named IMO. It's not obvious that it would "fix" stuff based on variable names | 10:59 |
stephenfin | frickler: 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 |
frickler | stephenfin: sounds plausible to me, yes | 11:02 |
stephenfin | Cool. If you're already working on that I'm happy to review a change, otherwise I can propose one later | 11:02 |
stephenfin | +1 on the existing one btw | 11:03 |
frickler | stephenfin: feel free to take over, I've been more than deep enough in this rabbit hole for my taste | 11:14 |
frickler | integrated CI job seems to be pretty broken, certainly all recurring errors there are unrelated on my recent patches | 13:45 |
opendevreview | Abhinav Shrivastava proposed openstack/cinder master: Handling NoneType for ssh related commands https://review.opendev.org/c/openstack/cinder/+/904496 | 15:03 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots https://review.opendev.org/c/openstack/cinder/+/899051 | 18:00 |
opendevreview | Bryan Neumann proposed openstack/cinder master: Dell EMC: PowerMax - Configurable SRDF snapshots https://review.opendev.org/c/openstack/cinder/+/899051 | 18:13 |
opendevreview | Rahman 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/+/904527 | 20:25 |
opendevreview | Rahman 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/+/904530 | 20:58 |
opendevreview | Rahman 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/+/904531 | 21:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!