| *** gouthamr_ is now known as gouthamr | 04:07 | |
| *** erlon_ is now known as erlon | 04:07 | |
| opendevreview | Merged openstack/cinder master: RBD: Open RBD images read-only where possible https://review.opendev.org/c/openstack/cinder/+/783643 | 06:37 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: api-ref: Add info regarding default types https://review.opendev.org/c/openstack/cinder/+/816184 | 10:21 |
| *** tobias-urdin5 is now known as tobias-urdin | 11:06 | |
| opendevreview | Zohar Mamedov proposed openstack/cinder-specs master: Os-brick NVMeoF connector multipathing spec https://review.opendev.org/c/openstack/cinder-specs/+/815613 | 11:17 |
| *** lbragstad7 is now known as lbragstad | 11:41 | |
| hemna | happy monday | 13:00 |
| hemna | weee! | 13:00 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/xena: RBD: Open RBD images read-only where possible https://review.opendev.org/c/openstack/cinder/+/816957 | 13:11 |
| rosmaita | geguileo: when you have a minute, i am holding up https://review.opendev.org/c/openstack/os-brick/+/814139 "Use file locks in connectors" with a release note quibble, lmk whether you agree or whether i should go soak my head | 13:17 |
| rosmaita | i am thinking we would want to get that backported to ussuri before the final release this week ? | 13:17 |
| hemna | fwiw, I resolved the conflict on the async backup rework https://review.opendev.org/c/openstack/cinder/+/784477 | 13:39 |
| rosmaita | hemna: thanks, you might want to put something on the meeting agenda to remind people to look it over | 13:41 |
| opendevreview | Maciej Szwed proposed openstack/os-brick master: Actually disconnect NVMe drive on disconnect_volume() https://review.opendev.org/c/openstack/os-brick/+/817005 | 13:50 |
| nschwarz | Hi there. I'm currently writing the spec for the possible change to be able to set multiple status to a volume discussed in the meeting about a week ago with the problem christian rohmann brought up. Due to the fact that the spec should have the same name as the blueprint the spec would be called 'volume-status-for-server-migration'. This is not a great title for the such a change and it does not | 14:21 |
| nschwarz | indicate that change either. What's your opinion on opening a new blueprint with maybe a link to the 'old' blueprint to give the spec a better and appropriate name? Or just leave it like it is? | 14:21 |
| rosmaita | nschwarz: thinking | 14:32 |
| rosmaita | nschwarz: go ahead and make a new blueprint with an appropriate name, and i can mark the old one as 'superseded' by your new BP | 14:37 |
| nschwarz | perfect. Thanks for the fast answer | 14:42 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: filter scheduler https://review.opendev.org/c/openstack/cinder/+/788365 | 14:44 |
| opendevreview | Rajat Dhasmana proposed openstack/python-cinderclient master: Improve help text of volume create command https://review.opendev.org/c/openstack/python-cinderclient/+/816188 | 14:50 |
| abishop | stable cores: can someone please take a look at https://review.opendev.org/c/openstack/cinder/+/813936, it needs one more +2 | 15:23 |
| rosmaita | e0ne eharney geguileo jungleboyj smcginnis whoami-rajat ^^ | 15:23 |
| rosmaita | abishop: thanks for tracking this | 15:23 |
| jungleboyj | abishop: rosmaita Taken care of. | 15:25 |
| whoami-rajat | done | 15:25 |
| rosmaita | jungleboyj: ty | 15:25 |
| abishop | yes, thanks! | 15:25 |
| rosmaita | whoami-rajat: ty | 15:25 |
| whoami-rajat | np | 15:26 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add info about __DEFAULT__ in install docs https://review.opendev.org/c/openstack/cinder/+/816187 | 15:31 |
| jhartkopf | Hi there, in September I have submitted a patch to Cinder. Sofia quickly hinted to add a related blueprint, which I did. Since then, the patch has not received any further review yet. May I ask someone to review the patch? The URL is https://review.opendev.org/c/openstack/cinder/+/810457 | 16:18 |
| simondodsley | cores: I need a final +2 and +W on https://review.opendev.org/c/openstack/cinder/+/815848. | 16:18 |
| jungleboyj | simondodsley: Looking. :-) | 16:40 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Fix _show_msg_check_clone_v2_api https://review.opendev.org/c/openstack/cinder/+/777031 | 18:32 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Update notes about hacking on os-brick https://review.opendev.org/c/openstack/os-brick/+/794658 | 18:39 |
| opendevreview | Merged openstack/cinder master: [Pure Storage] Add check for NVMe-FC capable array https://review.opendev.org/c/openstack/cinder/+/815848 | 19:53 |
| opendevreview | Merged openstack/cinder stable/victoria: Tests: Fix compatibility with ddt 1.2.x https://review.opendev.org/c/openstack/cinder/+/815673 | 19:53 |
| mloza | hello, just wanted to confirm, does cinder backup support multi backend ? | 20:10 |
| opendevreview | Merged openstack/cinder stable/xena: Fix typo in message policy deprecations https://review.opendev.org/c/openstack/cinder/+/813936 | 20:14 |
| opendevreview | Simon Dodsley proposed openstack/cinder stable/xena: [Pure Storage] Add check for NVMe-FC capable array https://review.opendev.org/c/openstack/cinder/+/817090 | 20:24 |
| opendevreview | Merged openstack/cinder master: mypy: filter scheduler https://review.opendev.org/c/openstack/cinder/+/788365 | 20:49 |
| rosmaita | yay! mypy patch of the week has merged! | 21:01 |
| *** dviroel is now known as dviroel|out | 21:11 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Remove all REST version checks https://review.opendev.org/c/openstack/cinder/+/817085 | 21:18 |
| opendevreview | Merged openstack/cinder master: Fixed copy-on-write mode in GPFS NFS driver https://review.opendev.org/c/openstack/cinder/+/813984 | 21:27 |
| rosmaita | mloza: https://docs.openstack.org/cinder/latest/admin/blockstorage-multi-backend.html | 21:37 |
| rosmaita | mloza: forget that, i missed the "backup" in your question | 21:38 |
| rosmaita | mloza: this spec hasn't been implemented yet, and probably needs an update, you might want to look it over to see if it would fit your use cases: https://specs.openstack.org/openstack/cinder-specs/specs/victoria/backup-backends-configuration.html | 21:44 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Prevent CI from running on irrelevant files https://review.opendev.org/c/openstack/os-brick/+/817111 | 21:57 |
| mloza | rosmaita: thanks | 23:39 |
| mloza | the specs looks like what I need | 23:40 |
| rosmaita | mloza: e0ne was working on that, but he's spread pretty thin these days, and i don't know that he's had time to do too much | 23:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!