[Bug 1323660] Re: [SRU] Glance image properties not copied to cinder volume with glance V2 API

Edward Hope-Morley edward.hope-morley at canonical.com
Thu Mar 23 11:18:05 UTC 2017


** Tags removed: sts-sru
** Tags added: sts-sru-needed

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to Ubuntu Cloud Archive.
https://bugs.launchpad.net/bugs/1323660

Title:
  [SRU] Glance image properties not copied to cinder volume with glance
  V2 API

Status in Cinder:
  Fix Released
Status in Ubuntu Cloud Archive:
  Invalid
Status in Ubuntu Cloud Archive icehouse series:
  Fix Committed
Status in Ubuntu Cloud Archive kilo series:
  Fix Released
Status in Glance Client:
  Fix Released
Status in Glance Client kilo series:
  Fix Committed
Status in cinder package in Ubuntu:
  Fix Released
Status in python-glanceclient package in Ubuntu:
  Fix Released
Status in cinder source package in Trusty:
  Fix Released
Status in python-glanceclient source package in Trusty:
  Fix Released

Bug description:
  [Description]
  When a cinder volume is created from a glance image, the "properties" attached to the glance image are not copied to the cinder volume (in the volume_image_metadata) if the glance V2 API is used. If the glance V1 API is used, the properties are copied correctly.

  [Test Case]
  To illustrate, here is a glance image with a hw_vif_model property defined:

  [root at controller-0 log(keystone_admin)]# glance image-show wrl5-avp
  +-------------------------+--------------------------------------+
  | Property                | Value                                |
  +-------------------------+--------------------------------------+
  | Property 'hw_vif_model' | avp                                  |
  | checksum                | 2386997c1a85709759c3d26ba0f11192     |
  | container_format        | bare                                 |
  | created_at              | 2014-05-23T15:25:01.804247           |
  | deleted                 | False                                |
  | disk_format             | raw                                  |
  | id                      | 78e935d4-3b8e-4cc4-ab9f-c24959492c22 |
  | is_public               | True                                 |
  | min_disk                | 0                                    |
  | min_ram                 | 0                                    |
  | name                    | wrl5-avp                             |
  | owner                   | 376000978ffb43159aa3ec554d62b7df     |
  | protected               | False                                |
  | size                    | 507510784                            |
  | status                  | active                               |
  | updated_at              | 2014-05-23T15:25:03.852936           |
  +-------------------------+--------------------------------------+

  When a volume is created (using glance V1 API) from this image (with "cinder create --image-id 2386997c1a85709759c3d26ba0f11192 --display-name=wrl5-avp-from-volume 1"), the volume_image_metadata for the resulting volume is:
  {u'container_format': u'bare', u'min_ram': u'0', u'hw_vif_model': u'avp', u'disk_format': u'qcow2', u'image_name': u'wrl5-avp', u'image_id': u'8f1b7941-cbac-484e-8ef4-5714e01f5c66', u'checksum': u'2386997c1a85709759c3d26ba0f11192', u'min_disk': u'0', u'size': u'507510784'}

  When a volume is created using the glance V2 API, the volume_image_metadata is missing the hw_vif_model:
  {u'container_format': u'bare', u'min_ram': u'0', u'disk_format': u'qcow2', u'image_name': u'wrl5-avp', u'image_id': u'8f1b7941-cbac-484e-8ef4-5714e01f5c66', u'checksum': u'2386997c1a85709759c3d26ba0f11192', u'min_disk': u'0', u'size': u'507510784'}

  With debug logs turned on, the results of the image get from glance
  are different. For the glance V1 API:

  May 25 12:49:48 localhost 2014-05-25 12:49:48.221 1773 DEBUG
  glanceclient.common.http [-]  HTTP/1.1 200 OK content-length: 0 x
  -image-meta-status: active x-image-meta-owner:
  621793db5e664ca390af686640b24543 x-image-meta-name: wrl5-avp x-image-
  meta-container_format: bare x-image-meta-created_at:
  2014-05-22T17:59:05.446468 etag: 2386997c1a85709759c3d26ba0f11192
  location: http://192.168.204.2:9292/v1/images/8f1b7941-cbac-484e-
  8ef4-5714e01f5c66 x-image-meta-min_ram: 0 x-image-meta-updated_at:
  2014-05-22T17:59:11.446979 x-image-meta-id: 8f1b7941-cbac-484e-
  8ef4-5714e01f5c66 date: Sun, 25 May 2014 12:49:48 GMT x-openstack-
  request-id: req-884ad959-fe85-4653-81e9-97a9205db3b8 x-image-meta-
  deleted: False x-image-meta-checksum: 2386997c1a85709759c3d26ba0f11192
  x-image-meta-property-hw_vif_model: avp x-image-meta-protected: False
  x-image-meta-min_disk: 0 x-image-meta-size: 507510784 x-image-meta-
  is_public: True content-type: text/html; charset=UTF-8 x-image-meta-
  disk_format: qcow2  log_http_response /usr/lib64/python2.7/site-
  packages/glanceclient/common/http.py:152

  With the glance V2 API:

  May 25 12:07:15 localhost 2014-05-25 12:07:15.589 12689 DEBUG
  glanceclient.common.http [-]  HTTP/1.1 200 OK date: Sun, 25 May 2014
  12:07:15 GMT content-length: 543 content-type: application/json;
  charset=UTF-8 x-openstack-request-id: req-
  5c8b6f6c-0880-4054-ba03-4fc89326f64e  {"status": "active", "name":
  "wrl5-avp", "tags": [], "container_format": "bare", "created_at":
  "2014-05-22T17:59:05Z", "hw_vif_model": "avp", "disk_format": "qcow2",
  "updated_at": "2014-05-22T17:59:11Z", "visibility": "public", "self":
  "/v2/images/8f1b7941-cbac-484e-8ef4-5714e01f5c66", "protected": false,
  "id": "8f1b7941-cbac-484e-8ef4-5714e01f5c66", "file":
  "/v2/images/8f1b7941-cbac-484e-8ef4-5714e01f5c66/file", "checksum":
  "2386997c1a85709759c3d26ba0f11192", "min_disk": 0, "size": 507510784,
  "min_ram": 0, "schema": "/v2/schemas/image"}  log_http_response
  /usr/lib64/python2.7/site-packages/glanceclient/common/http.py:152

  The difference (other than the switch to the json encoding) is that
  the hw_vif_model is returned as a property in the V1 case and returned
  as a regular field in the V2 case. From a look at the cinder code, I
  suspect that the problem happens because the _extract_attributes
  function (in cinder/image/glance.py) is using a whitelist to copy
  certain attributes and then copying the "properties" from the image
  transparently. In the case of the glance V2 API, the hw_vif_model
  isn't a "property", so it isn't getting copied. Here is the code (from
  Havana):

  def _extract_attributes(image):
      IMAGE_ATTRIBUTES = ['size', 'disk_format', 'owner',
                          'container_format', 'checksum', 'id',
                          'name', 'created_at', 'updated_at',
                          'deleted_at', 'deleted', 'status',
                          'min_disk', 'min_ram', 'is_public']
      output = {}
      for attr in IMAGE_ATTRIBUTES:
          output[attr] = getattr(image, attr, None)

      output['properties'] = getattr(image, 'properties', {})

      return output

  I don't know why the glance V2 API is returning the properties as
  regular attributes, but I expect that is what is causing the problem.

  I am testing this in Havana, using the Ceph backend, but I don't
  expect the backend matters here.

  [Regression Potential]
  Regression potential is low. Patch is cherry-picked from upstream stable/liberty branch without any changes.

To manage notifications about this bug go to:
https://bugs.launchpad.net/cinder/+bug/1323660/+subscriptions



More information about the Ubuntu-openstack-bugs mailing list