| *** JohnnyW559 is now known as JohnnyW55 | 00:25 | |
| opendevreview | ohjiwoo proposed openstack/cinder master: Add logging for snapshot creation failures https://review.opendev.org/c/openstack/cinder/+/972066 | 07:36 |
|---|---|---|
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/982617 | 10:33 |
| *** kevko7 is now known as kevko | 12:25 | |
| opendevreview | Alejandro Santoyo Gonzalez proposed openstack/cinder master: Fix properties={} being sent to Glance https://review.opendev.org/c/openstack/cinder/+/980692 | 13:01 |
| opendevreview | Abhishek Gupta proposed openstack/cinder master: Dell PowerStore: Add manage/unmanage snapshot support https://review.opendev.org/c/openstack/cinder/+/987177 | 14:43 |
| opendevreview | Abhishek Gupta proposed openstack/cinder master: Dell PowerStore: Add manage/unmanage snapshot support https://review.opendev.org/c/openstack/cinder/+/987177 | 14:44 |
| opendevreview | Anoop Kumar Shukla proposed openstack/cinder master: Added additional capabilities to ASAr2 driver https://review.opendev.org/c/openstack/cinder/+/977911 | 17:37 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Properly set shared_targets for all NVMe-oF protocols https://review.opendev.org/c/openstack/cinder/+/964492 | 18:20 |
| winiciusallan[m] | hey cinder o/ | 18:21 |
| winiciusallan[m] | I've noticed some differences between the snapshot API v2 and v3 documentation. I saw that V2 allows the use of the name field when listing, but V3 does not | 18:22 |
| winiciusallan[m] | at least according to the documentation. when testing in my environment, I was to able to filter it. is there some msising documentation on V3's side? | 18:22 |
| rosmaita | winiciusallan[m]: i think the model for how we document the filtering changed between v2 and v3. In v3, we have "resource filters" that i think an operator can customize, but which also allow discovery of what filters are available:https://docs.openstack.org/api-ref/block-storage/v3/#resource-filters-resource-filters | 18:27 |
| winiciusallan[m] | Oh, ok. I see. I compared two environments and they look like to have the same filters by default for the snapshot resource | 18:30 |
| rosmaita | winiciusallan[m]: yes, i think the default set is supposed to be compatible with v2 | 18:31 |
| winiciusallan[m] | rosmaita: Is there somewhere I can check what are these fields? I mean, in code or elsewhere | 18:31 |
| rosmaita | https://opendev.org/openstack/cinder/src/branch/master/etc/cinder/resource_filters.json | 18:31 |
| winiciusallan[m] | rosmaita: yeah yeah, that makes sense. | 18:31 |
| rosmaita | that's the default set | 18:31 |
| rosmaita | but you can also make that api request in a particular cloud | 18:32 |
| rosmaita | our docs look pretty lacking in this regard, i don't know how you feel about adding some clarifications to the api-ref ? | 18:32 |
| winiciusallan[m] | not sure about the time to commit to this, but I have one friend who is starting on contributions. it is a good opportunity to get involved. what do you think? | 18:34 |
| winiciusallan[m] | the resource filter API answear why I was not finding the list schema here https://github.com/openstack/cinder/blob/master/cinder/api/schemas/snapshots.py | 18:39 |
| winiciusallan[m] | hehehe | 18:39 |
| rosmaita | yeah, i don't think cinder allows listing of any schemas, they're just used for request validation | 18:40 |
| rosmaita | winiciusallan[m]: i think updating the docs around this would be a good contribution, i will write up a bug real quick, and then you can point your friend at it | 18:41 |
| winiciusallan[m] | rosmaita: that's really appreciated. thanks | 18:42 |
| rosmaita | winiciusallan[m]: turns out there was an existing bug: https://bugs.launchpad.net/cinder/+bug/1943689 | 19:11 |
| winiciusallan[m] | rosmaita: good. I noticed that you have changed the description, it is reliable with the needed changes, right? | 19:14 |
| rosmaita | yep | 19:14 |
| winiciusallan[m] | perfect! I'll send it to my friend. many thanks, rosmaita | 19:15 |
| rosmaita | np | 19:15 |
| opendevreview | Ghanshyam Maan proposed openstack/cinder master: Enable SRBAC by default https://review.opendev.org/c/openstack/cinder/+/986472 | 19:19 |
| opendevreview | Ghanshyam Maan proposed openstack/cinder master: Enable SRBAC by default https://review.opendev.org/c/openstack/cinder/+/986472 | 19:50 |
| opendevreview | Merged openstack/cinder master: Enable threads mode in cinder-volume https://review.opendev.org/c/openstack/cinder/+/954650 | 20:25 |
| opendevreview | Merged openstack/cinder master: cinder/monkey_patch.py for threading mode https://review.opendev.org/c/openstack/cinder/+/970332 | 20:25 |
| opendevreview | Merged openstack/cinder master: Use futurist-based thread pool for volume manager https://review.opendev.org/c/openstack/cinder/+/982319 | 20:25 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Migrate setup.cfg to pyproject.toml https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/987284 | 21:07 |
| opendevreview | Ivan Anfimov proposed openstack/rbd-iscsi-client master: Cleanup setup.py https://review.opendev.org/c/openstack/rbd-iscsi-client/+/971734 | 21:25 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Migrate setup.cfg to pyproject.toml https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/987284 | 21:54 |
| opendevreview | Eric Harney proposed openstack/cinder master: Add tpool.Proxy wrapper for eventlet mode https://review.opendev.org/c/openstack/cinder/+/984951 | 22:19 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD changes for eventlet removal https://review.opendev.org/c/openstack/cinder/+/959385 | 22:19 |
| *** erlon2 is now known as erlon | 22:39 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!