fungible driver should be using volume_utils to upload volume as an image

Bug #2063448 reported by Brian Rosmaita
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Triaged
Low
Unassigned

Bug Description

Noticed this in master (Dalmatian development branch). A few cycles back, volume_utils.upload_volume() was introduced as a wrapper for image_utils.upload_volume(); it does some manipulation of volume glance metadata before calling the image_utils function.

The fungible driver is using image_utils:
https://opendev.org/openstack/cinder/src/commit/79629c346e001e88a03a63f3a713b7af6ee8193c/cinder/volume/drivers/fungible/driver.py

This should be corrected, because fungible volumes uploaded as images will be missing the base image ref, among other things. (And, of course, they'll also miss out on any future enhancements to the volume_utils function.)

tags: added: fungible
Rajesh Tailor (ratailor)
Changed in cinder:
assignee: nobody → Rajesh Tailor (ratailor)
assignee: Rajesh Tailor (ratailor) → nobody
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.