Make provision to pass store information

Bug #1857094 reported by Abhishek Kekane
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Brian Rosmaita
oslo.vmware
Fix Released
Undecided
Damon Li

Bug Description

Cinder is now planning to support multiple stores configured in glance. Using the same cinder can now upload the volume as a image (copy-volume-to-image) to specific glance store using volume-type.

Using this patch [1] cinder has now added this support to it's vmware driver, but oslo_vmware further needs to pass it to glance.

[1] https://review.opendev.org/#/c/661676/16

Damon Li (damonl1)
Changed in oslo.vmware:
assignee: nobody → Damon Li (damonl1)
Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

See the comment here:

https://review.opendev.org/#/c/661676/16/cinder/tests/unit/volume/drivers/vmware/test_vmware_vmdk.py@1214

for a suggestion about what needs to be changed.

Revision history for this message
Brian Rosmaita (brian-rosmaita) wrote :

The Cinder part of this bug is to update the Cinder requirements and lower-constraints to use versions of oslo.vmware that contain this fix (or to add a 'known issues' release note if the new version isn't released until after the Ussuri library freeze).

Changed in cinder:
assignee: nobody → Brian Rosmaita (brian-rosmaita)
status: New → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.vmware (master)

Fix proposed to branch: master
Review: https://review.opendev.org/701671

Changed in oslo.vmware:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.vmware (master)

Reviewed: https://review.opendev.org/701671
Committed: https://git.openstack.org/cgit/openstack/oslo.vmware/commit/?id=60d23fb1e8d4a85ba40162ca8fcc511973f57a09
Submitter: Zuul
Branch: master

commit 60d23fb1e8d4a85ba40162ca8fcc511973f57a09
Author: Damon Li <email address hidden>
Date: Wed Jan 8 22:58:48 2020 -0800

    Pass "store_id" to backend when transfer volume to image

    Glance supports multiple stores now. In order to use this feature, on
    patch #661676, it added parameter "store_id" when call function
    "image_transfer.upload_image". We need to parse this parameter and pass
    it to "image_service.update" function.

    Closes-Bug: #1857094

    Change-Id: I03747f54545ae3f91bf10de76d07de6828a419d6

Changed in oslo.vmware:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/oslo.vmware 2.35.0

This issue was fixed in the openstack/oslo.vmware 2.35.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/703077

Changed in cinder:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/703077
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=d9cf1f439bb2b82ab713dcbe72327ef67f93e9a8
Submitter: Zuul
Branch: master

commit d9cf1f439bb2b82ab713dcbe72327ef67f93e9a8
Author: Brian Rosmaita <email address hidden>
Date: Mon Jan 13 14:19:19 2020 -0500

    Update oslo.vmware version

    Version 2.35.0 contains a fix needed for the "Support multiple stores
    of Glance" feature.

    Change-Id: I64937ba8abdb0b0c5b6c1aab6ed08d67bac13590
    Closes-bug: #1857094

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 16.0.0.0b1

This issue was fixed in the openstack/cinder 16.0.0.0b1 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.