stephenfin | \o/ The SQLAlchemy 2.0 test patch is finally passing | 08:44 |
---|---|---|
* stephenfin hums a Queen song... | 08:44 | |
frickler | .oO(Another one bites the dust) *scnr* | 09:26 |
opendevreview | Yian Zong proposed openstack/os-brick master: Dell PowerFlex: Unnecessary login happen https://review.opendev.org/c/openstack/os-brick/+/876284 | 09: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/+/896253 | 12:57 |
opendevreview | Konrad Gube proposed openstack/cinder master: Support waiting for volume extend completion https://review.opendev.org/c/openstack/cinder/+/891602 | 14:43 |
opendevreview | Konrad Gube proposed openstack/cinder master: NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873686 | 14:43 |
opendevreview | Konrad Gube proposed openstack/cinder master: Netapp NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873889 | 14:43 |
opendevreview | Arnaud Morin proposed openstack/cinder master: Add GMR to cinder wsgi https://review.opendev.org/c/openstack/cinder/+/885428 | 15:00 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Fix create volume from image with signature_verified key https://review.opendev.org/c/openstack/cinder/+/896584 | 19:23 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Fix create volume from image with signature_verified key https://review.opendev.org/c/openstack/cinder/+/896584 | 19:23 |
*** jbernard_ is now known as jbernard | 20:26 | |
opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: zuul: use only RE2 syntax https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896594 | 20:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!