Tuesday, 2023-09-26

stephenfin\o/ The SQLAlchemy 2.0 test patch is finally passing08:44
* stephenfin hums a Queen song...08:44
frickler.oO(Another one bites the dust) *scnr*09:26
opendevreviewYian Zong proposed openstack/os-brick master: Dell PowerFlex: Unnecessary login happen  https://review.opendev.org/c/openstack/os-brick/+/87628409:33
Roamer`hi whoami-rajat, whenever you have a minute... you might remember that last year I brought up an issue with the StorPool driver and Cinder-backed Glance images: the StorPool driver has something called "templates" that it exposes as Cinder volume pools, but it is actually a) possible and b) not always that expensive to move or clone a volume from one template into another. Thus, we keep Glance 11:01
Roamer`images in one template (Cinder pool), but then users want to create Cinder volumes in other Cinder pools (StorPool templates), and Glance + Cinder do not understand that they can clone a volume into a different template. My first attempt at fixing that was reimplementing much of Cinder's own logic into the StorPool driver; at a Cinder meeting some time ago you said that was suboptimal and it 11:01
Roamer`might be better handled in other ways, and you suggested a driver capability. I did that in https://review.opendev.org/c/openstack/cinder/+/847131 and https://review.opendev.org/c/openstack/cinder/+/847132 ...and now it seems that simondodsley has an objection to including what they see as a vendor-specific feature in the support matrix (see his comments on 847131). What are your thoughts on the 11:01
Roamer`matter? Should this be advertised as a driver capability that _clone_image_volume() can test for? Should it be added to the support matrix? (actually, I just realized that I may have misunderstood simondodsley's comments - are the objections *only* to adding that to the support matrix, or to adding the driver capability at all?)11:01
Roamer`sorry for the wall of text, and thanks in advance :)11:01
hemna-can we get another +2 on this please ?  https://review.opendev.org/c/openstack/python-openstackclient/+/89625312:57
opendevreviewKonrad Gube proposed openstack/cinder master: Support waiting for volume extend completion  https://review.opendev.org/c/openstack/cinder/+/89160214:43
opendevreviewKonrad Gube proposed openstack/cinder master: NFS: Add online extend support  https://review.opendev.org/c/openstack/cinder/+/87368614:43
opendevreviewKonrad Gube proposed openstack/cinder master: Netapp NFS: Add online extend support  https://review.opendev.org/c/openstack/cinder/+/87388914:43
opendevreviewArnaud Morin proposed openstack/cinder master: Add GMR to cinder wsgi  https://review.opendev.org/c/openstack/cinder/+/88542815:00
opendevreviewRajat Dhasmana proposed openstack/cinder master: WIP: Fix create volume from image with signature_verified key  https://review.opendev.org/c/openstack/cinder/+/89658419:23
opendevreviewRajat Dhasmana proposed openstack/cinder master: WIP: Fix create volume from image with signature_verified key  https://review.opendev.org/c/openstack/cinder/+/89658419:23
*** jbernard_ is now known as jbernard20:26
opendevreviewBrian Rosmaita proposed openstack/cinder-tempest-plugin master: zuul: use only RE2 syntax  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/89659420:52

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