| opendevreview | Ghanshyam proposed openstack/cinder-tempest-plugin master: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/843091 | 02:10 |
|---|---|---|
| opendevreview | Ghanshyam proposed openstack/cinder stable/ussuri: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/cinder/+/843092 | 02:11 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Docs - Known issues section https://review.opendev.org/c/openstack/cinder/+/843118 | 08:32 |
| tobias-urdin | friendly ping for https://review.opendev.org/c/openstack/cinder/+/840289 (review request for swift backup driver topic of last meeting) | 08:42 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder stable/yoga: HPE 3PAR: In multi host env, fix multi-detach operation https://review.opendev.org/c/openstack/cinder/+/842947 | 09:00 |
| kevko | hello, just quick question ...if i am running cinder-api via apache mod_wsgi is osapi_volume used ? (i think no, am I right ?) | 09:02 |
| opendevreview | Oleg proposed openstack/cinder master: PowerMax Docs - Known issues section https://review.opendev.org/c/openstack/cinder/+/843118 | 09:15 |
| opendevreview | Benny Kopilov proposed openstack/cinder-tempest-plugin master: Refactor rbac cinder tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/842954 | 09:43 |
| *** dviroel|out is now known as dviroel | 11:21 | |
| opendevreview | Merged openstack/cinder master: db: Remove unnecessary session management https://review.opendev.org/c/openstack/cinder/+/841083 | 11:34 |
| opendevreview | Matheus Andrade proposed openstack/cinder master: Fix NetApp NFS storage migration between backends https://review.opendev.org/c/openstack/cinder/+/843018 | 12:14 |
| opendevreview | Merged openstack/cinder master: db: Migrate "volume metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830096 | 12:17 |
| opendevreview | Merged openstack/cinder master: db: Migrate "snapshot" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830097 | 12:17 |
| opendevreview | Merged openstack/cinder master: db: Migrate "snapshot metadata" APIs to enginefacade https://review.opendev.org/c/openstack/cinder/+/830098 | 12:24 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Extend RBD deferred deletion interval https://review.opendev.org/c/openstack/cinder/+/843151 | 14:28 |
| opendevreview | Eric Harney proposed openstack/cinder master: tox.ini: combine functional-py* envs https://review.opendev.org/c/openstack/cinder/+/843153 | 14:48 |
| rosmaita | eharney: good idea ^^ | 14:55 |
| ricolin | whoami-rajat__: got time for another review for https://review.opendev.org/c/openstack/cinder/+/839793 ? :) Thanks | 14:56 |
| whoami-rajat__ | ricolin, sure, will add it to my review list | 14:57 |
| ricolin | whoami-rajat__: thanks:) | 15:41 |
| opendevreview | Merged openstack/cinder master: NetApp: Support custom igroups https://review.opendev.org/c/openstack/cinder/+/730936 | 15:43 |
| *** whoami-rajat__ is now known as whoami-rajat | 15:50 | |
| *** dviroel is now known as dviroel|lunch | 15:58 | |
| jungleboyj | whoami-rajat: I won't be able to make the video meeting tomorrow. Have another video meeting that conflicts. | 16:43 |
| rosmaita | jungleboyj: btw, you did not miss the review festival ... it was moved to *this* friday | 16:45 |
| jungleboyj | Oh. Cool. | 16:45 |
| jungleboyj | rosmaita: I do have some conflicts but should be able to join for a little bit. | 16:46 |
| rosmaita | :) | 16:46 |
| *** dviroel|lunch is now known as dviroel | 16:58 | |
| hemna | anyone familiar with generic volume groups? | 17:20 |
| hemna | looks like in order to create a group you have to have a group type and volume type | 17:21 |
| hemna | not sure the purpose of requiring a group type | 17:21 |
| whoami-rajat | jungleboyj, sure, no problem | 17:29 |
| whoami-rajat | hemna, i think it's for group_specs, similar to extra_specs in volume type | 17:32 |
| whoami-rajat | but not 100% sure | 17:32 |
| hemna | I get that, just don't understand the purpose | 17:33 |
| hemna | since you have to assign a volume type to a group as well | 17:33 |
| hemna | we don't have a group type created in our deployment | 17:33 |
| hemna | and a customer wants to create a group so they can snap the whole group | 17:34 |
| hemna | but it's impossible without creating a placeholder group type that has no specs | 17:34 |
| hemna | I don't understand the need for requiring a group type or even a volume type for that matter to create a group | 17:34 |
| hemna | documentation on it is useless | 17:35 |
| hemna | seems like cinder should always have a default group type created, like the default volume type | 17:35 |
| hemna | or customers/users can never create a group | 17:35 |
| whoami-rajat | hemna, the only usage of that i found is in dell drivers for replication https://github.com/openstack/cinder/blob/393c2e4ad90c05ebf28cc3a2c65811d7e1e0bc18/cinder/volume/drivers/dell_emc/vnx/replication.py#L68 | 17:50 |
| whoami-rajat | i think this is sort of the similar issue where a volume can be created without a volume type but it won't possess backend specific capabilities | 17:50 |
| whoami-rajat | so group type is useful, for the backends that make use of it | 17:50 |
| whoami-rajat | for others, not so much | 17:50 |
| whoami-rajat | but it's more of a question of whether we want to enforce these types on resources or not, we enforced having a volume type for every volume, similarly we can discuss for group types | 17:51 |
| hemna | currently the command line requires a group type to even create a group | 17:53 |
| whoami-rajat | ok, so it's already enforced there, hmm | 17:53 |
| hemna | yah :( | 17:53 |
| whoami-rajat | can't provide a resolution atm but how about a topic for this at the midcycle? https://etherpad.opendev.org/p/cinder-zed-midcycles | 17:55 |
| opendevreview | Merged openstack/cinder master: pylint: Fix vmdk driver use before definition https://review.opendev.org/c/openstack/cinder/+/842129 | 17:56 |
| *** dviroel is now known as dviroel|out | 21:11 | |
| opendevreview | Ghanshyam proposed openstack/cinder stable/ussuri: DNM: Testing tempest pin on stable/ussuri https://review.opendev.org/c/openstack/cinder/+/843092 | 23:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!