| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Remove v2 support from the shell https://review.opendev.org/c/openstack/python-cinderclient/+/791834 | 03:06 |
|---|---|---|
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Remove v2 classes https://review.opendev.org/c/openstack/python-cinderclient/+/792959 | 03:08 |
| opendevreview | Merged openstack/python-brick-cinderclient-ext master: Remove skip_missing_interpreters https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/796835 | 03:13 |
| opendevreview | Merged openstack/python-cinderclient stable/wallaby: Remove skip_missing_interpreters https://review.opendev.org/c/openstack/python-cinderclient/+/799882 | 04:23 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 05:25 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add user message for migrate multiattach volume https://review.opendev.org/c/openstack/cinder/+/800430 | 08:01 |
| *** akekane_ is now known as abhishekk | 09:38 | |
| opendevreview | Luigi Toscano proposed openstack/cinder stable/ussuri: DNM dummy test change https://review.opendev.org/c/openstack/cinder/+/800618 | 10:45 |
| *** dviroel|out is now known as dviroel | 11:20 | |
| hberaud | rosmaita: o/ Quick question: rbd-isci-client was added in January 2021 and never released yet, do we should include it for Xena release? | 12:14 |
| rosmaita | hberaud: it's on an independent release model, we have a CI issue to work out, but we intend to release sometime during xena | 12:23 |
| hberaud | rosmaita: ack, no rush, especially if this is an independent project. Thanks for your reply | 12:24 |
| rosmaita | hberaud: cool, thanks for being on top of this | 12:25 |
| hberaud | rosmaita: We want to avoid to you to miss the train :) | 12:26 |
| rosmaita | :) | 12:28 |
| hemna | freakin CI | 12:40 |
| *** geguileo is now known as Guest659 | 12:42 | |
| *** ricolin_ is now known as ricolin | 12:48 | |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient stable/victoria: Remove skip_missing_interpreters https://review.opendev.org/c/openstack/python-cinderclient/+/800635 | 12:49 |
| rosmaita | hemna: if you have some time to revisit the cinderclient v2 shell removal patch, that would be very helpful: https://review.opendev.org/c/openstack/python-cinderclient/+/791834 | 12:55 |
| hemna | ok | 13:13 |
| hemna | so by stripping v2 support from the client, then the newer client can't talk to the older deployments (that for whatever reason is still using only v2) | 13:14 |
| hemna | v3 was enabled by default years ago | 13:15 |
| rosmaita | right | 13:15 |
| rosmaita | hemna: also, we asked for feedback on the ML: http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018876.html | 13:17 |
| rosmaita | (when I thought we'd be removing it in wallaby) | 13:17 |
| hemna | do you remember when v3 was enabled by default? | 13:17 |
| hemna | I think I was still at HPE then | 13:17 |
| rosmaita | it was a while back, i have a note somewhere ... looking | 13:18 |
| hemna | we are using v3 | 13:19 |
| *** Guest659 is now known as geguileo | 13:23 | |
| rosmaita | looks like v3 was introduced as the default same time v2 was deprecated, by this commit: https://opendev.org/openstack/cinder/commit/6b11d276d13cc866aea9b3cfa321b3b19ee13d55 | 13:24 |
| rosmaita | maybe in mitaka? | 13:24 |
| hemna | going to test the client against my qa deployment | 13:24 |
| rosmaita | thanks! | 13:24 |
| hemna | looks like it's workin :P | 13:26 |
| hemna | lgtm | 13:30 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: continued manager, scheduler, rpcapi https://review.opendev.org/c/openstack/cinder/+/784453 | 13:49 |
| caiqilong | rosmaita: there is a volume feature named "manage volume", but it seems not display on the page "https://docs.openstack.org/cinder/latest/reference/support-matrix.html", if my driver added this feature, how to let it display on the support matrix. | 13:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: coordination.py https://review.opendev.org/c/openstack/cinder/+/788364 | 13:53 |
| caiqilong | And the driver name on page "https://docs.openstack.org/cinder/latest/reference/support-matrix.html" can be changed ? Because my company's storage product name has been updated. | 13:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: filter scheduler https://review.opendev.org/c/openstack/cinder/+/788365 | 13:54 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate volume/api.py https://review.opendev.org/c/openstack/cinder/+/733622 | 14:03 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: policy.py https://review.opendev.org/c/openstack/cinder/+/775008 | 14:04 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: backup https://review.opendev.org/c/openstack/cinder/+/787195 | 14:05 |
| rosmaita | caiqilong: yes, manage volume is not currently in the support matrix, but it would probably be useful to add it ... for now, you can mention that you can handle manage/unmanage/list-manageable in your driver docs | 14:15 |
| opendevreview | Merged openstack/os-brick master: Remove skip_missing_interpreters https://review.opendev.org/c/openstack/os-brick/+/798956 | 14:18 |
| rosmaita | caiqilong: about the name change ... if it's just the display name in the matrix, i think you can patch the support-matrix.ini file, but if you want to change the python class name, that's a much bigger deal | 14:20 |
| caiqilong | rosmaita: ok, thanks a lot. | 14:23 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: quota.py https://review.opendev.org/c/openstack/cinder/+/788366 | 14:26 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Use 'import sqlalchemy as sa' pattern https://review.opendev.org/c/openstack/cinder/+/786927 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Reorder initial migration https://review.opendev.org/c/openstack/cinder/+/786928 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Move sqlalchemy-migrate migrations https://review.opendev.org/c/openstack/cinder/+/786929 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Vendor 'oslo_db.sqlalchemy.migration' https://review.opendev.org/c/openstack/cinder/+/787264 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add alembic boilerplate https://review.opendev.org/c/openstack/cinder/+/786930 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add initial alembic migration https://review.opendev.org/c/openstack/cinder/+/786931 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add all migrations to initial alembic migration https://review.opendev.org/c/openstack/cinder/+/786932 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Integrate alembic https://review.opendev.org/c/openstack/cinder/+/786933 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove 'db_driver' option https://review.opendev.org/c/openstack/cinder/+/800695 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: db: Remove 'db' argument from various managers https://review.opendev.org/c/openstack/cinder/+/800696 | 17:32 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: WIP: db: Remove 'self.db' from most code paths https://review.opendev.org/c/openstack/cinder/+/800697 | 17:32 |
| -opendevstatus- NOTICE: Depends-On using https://review.opendev.org URLs are currently not working. This was due to a config change in Zuul that we are reverting and will be restarting Zuul to pick up. | 17:40 | |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove various unused code https://review.opendev.org/c/openstack/cinder/+/800703 | 18:02 |
| opendevreview | Eric Harney proposed openstack/cinder master: Drivers: remove unused code https://review.opendev.org/c/openstack/cinder/+/800704 | 18:02 |
| hemna | rosmaita found one small issue with the devstack-ceph-plugin | 18:03 |
| hemna | https://zuul.opendev.org/t/openstack/build/838170f537aa425e899fe6bb36a39115/log/controller/logs/devstacklog.txt#3392-3404 | 18:03 |
| hemna | the syntax for that has changed looks like | 18:03 |
| rosmaita | hemna: was wondering if it might be something like that | 18:03 |
| hemna | trying to fix it and get devstack up | 18:03 |
| hemna | looks like focal is being used now | 18:04 |
| rosmaita | i think our only options are focal or centos-8-stream | 18:06 |
| hemna | ok, I'm trying to get through standing devstaack up with all the CI settings | 18:15 |
| *** dviroel is now known as dviroel|out | 21:41 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!