]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
_translate_from_glance() can cause an unnecessary HTTP request
authorChris Buccella <buccella@linux.vnet.ibm.com>
Thu, 3 Apr 2014 04:02:45 +0000 (04:02 +0000)
committerChris Buccella <buccella@linux.vnet.ibm.com>
Thu, 3 Apr 2014 04:02:45 +0000 (04:02 +0000)
commitda13c6285bb0aee55cfbc93f55ce2e2b7d6a28f2
tree2a4d26fa1a1d02eed94a89d3da555d920ec213f1
parentc72cc4b2345e9e271205d1de00c9812d0a2684ec
_translate_from_glance() can cause an unnecessary HTTP request

After returning from a get() call to python-glanceclient, cinder runs a
translation function on the returned Image to get the data it wants. Part of
this process is checking for an expected set of attributes, one of which is
the deletion time ('deleted_at'). However, if the image has not been deleted,
deleted_at key will not exist. This forces another call to glance to occur for
the same image. A similar problem exists for the checksum attribute, which does
not exist before an image is active. The fix here is to only consider
deleted_at and checksum if they are expected to be present.

This change was made in nova as change I67b7dd16

Change-Id: Iedc16cb9316f9610fdb8ac03f448bc375a4e6bfa
Closes-Bug: #1275173
cinder/image/glance.py