Tuesday, 2022-05-24

opendevreviewGhanshyam proposed openstack/cinder-tempest-plugin master: DNM: Testing tempest pin on stable/ussuri  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/84309102:10
opendevreviewGhanshyam proposed openstack/cinder stable/ussuri: DNM: Testing tempest pin on stable/ussuri  https://review.opendev.org/c/openstack/cinder/+/84309202:11
opendevreviewOleg proposed openstack/cinder master: PowerMax Docs - Known issues section  https://review.opendev.org/c/openstack/cinder/+/84311808:32
tobias-urdinfriendly ping for https://review.opendev.org/c/openstack/cinder/+/840289 (review request for swift backup driver topic of last meeting)08:42
opendevreviewRaghavendra Tilay proposed openstack/cinder stable/yoga: HPE 3PAR: In multi host env, fix multi-detach operation  https://review.opendev.org/c/openstack/cinder/+/84294709:00
kevkohello, 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
opendevreviewOleg proposed openstack/cinder master: PowerMax Docs - Known issues section  https://review.opendev.org/c/openstack/cinder/+/84311809:15
opendevreviewBenny Kopilov proposed openstack/cinder-tempest-plugin master: Refactor rbac cinder tests  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/84295409:43
*** dviroel|out is now known as dviroel11:21
opendevreviewMerged openstack/cinder master: db: Remove unnecessary session management  https://review.opendev.org/c/openstack/cinder/+/84108311:34
opendevreviewMatheus Andrade proposed openstack/cinder master: Fix NetApp NFS storage migration between backends  https://review.opendev.org/c/openstack/cinder/+/84301812:14
opendevreviewMerged openstack/cinder master: db: Migrate "volume metadata" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83009612:17
opendevreviewMerged openstack/cinder master: db: Migrate "snapshot" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83009712:17
opendevreviewMerged openstack/cinder master: db: Migrate "snapshot metadata" APIs to enginefacade  https://review.opendev.org/c/openstack/cinder/+/83009812:24
opendevreviewEric Harney proposed openstack/cinder master: Tests: Extend RBD deferred deletion interval  https://review.opendev.org/c/openstack/cinder/+/84315114:28
opendevreviewEric Harney proposed openstack/cinder master: tox.ini: combine functional-py* envs  https://review.opendev.org/c/openstack/cinder/+/84315314:48
rosmaitaeharney: good idea ^^14:55
ricolinwhoami-rajat__: got time for another review for  https://review.opendev.org/c/openstack/cinder/+/839793 ? :) Thanks14:56
whoami-rajat__ricolin, sure, will add it to my review list14:57
ricolinwhoami-rajat__: thanks:)15:41
opendevreviewMerged openstack/cinder master: NetApp: Support custom igroups  https://review.opendev.org/c/openstack/cinder/+/73093615:43
*** whoami-rajat__ is now known as whoami-rajat15:50
*** dviroel is now known as dviroel|lunch15:58
jungleboyjwhoami-rajat:  I won't be able to make the video meeting tomorrow.  Have another video meeting that conflicts.16:43
rosmaitajungleboyj: btw, you did not miss the review festival ... it was moved to *this* friday16:45
jungleboyjOh.  Cool.16:45
jungleboyjrosmaita:  I do have some conflicts but should be able to join for a little bit.16:46
*** dviroel|lunch is now known as dviroel16:58
hemnaanyone familiar with generic volume groups?17:20
hemnalooks like in order to create a group you have to have a group type and volume type17:21
hemnanot sure the purpose of requiring a group type17:21
whoami-rajatjungleboyj, sure, no problem17:29
whoami-rajathemna, i think it's for group_specs, similar to extra_specs in volume type17:32
whoami-rajatbut not 100% sure17:32
hemnaI get that, just don't understand the purpose17:33
hemnasince you have to assign a volume type to a group as well17:33
hemnawe don't have a group type created in our deployment17:33
hemnaand a customer wants to create a group so they can snap the whole group17:34
hemnabut it's impossible without creating a placeholder group type that has no specs17:34
hemnaI don't understand the need for requiring a group type or even a volume type for that matter to create a group17:34
hemnadocumentation on it is useless17:35
hemnaseems like cinder should always have a default group type created, like the default volume type17:35
hemnaor customers/users can never create a group17:35
whoami-rajathemna, 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#L6817:50
whoami-rajati 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 capabilities17:50
whoami-rajatso group type is useful, for the backends that make use of it17:50
whoami-rajatfor others, not so much17:50
whoami-rajatbut 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 types17:51
hemnacurrently the command line requires a group type to even create a group17:53
whoami-rajatok, so it's already enforced there, hmm17:53
hemnayah :(17:53
whoami-rajatcan't provide a resolution atm but how about a topic for this at the midcycle? https://etherpad.opendev.org/p/cinder-zed-midcycles17:55
opendevreviewMerged openstack/cinder master: pylint: Fix vmdk driver use before definition  https://review.opendev.org/c/openstack/cinder/+/84212917:56
*** dviroel is now known as dviroel|out21:11
opendevreviewGhanshyam proposed openstack/cinder stable/ussuri: DNM: Testing tempest pin on stable/ussuri  https://review.opendev.org/c/openstack/cinder/+/84309223:22

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!