Open Stack

On 03/13/2014 03:04 PM, Josh Durgin wrote:
> These reverts are still confusing me. The use of glance's v2 api
> is very limited and easy to protect from errors.
> 
> These patches use the v2 glance api for exactly one call - to get
> image locations. This has been available and used by other
> features in nova and cinder since 2012.
> 
> Jay's patch fixed the one issue that was found, and added tests for
> several other cases. No other calls to glance v2 are used. The method
> Jay fixed is the only one that accesses the response from glanceclient.
> Furthermore, it's trivial to guard against more incompatibilities and
> fall back to downloading normally if any errors occur. This already
> happens if glance does not expose image locations.

There was some use of the v2 API, but not by default.  These patches
changed that, and it was broken.  We went from not requiring the v2 API
to requiring it, without a complete view for what that means, including
a severe lack of testing of that API.

I think it's the right call to block any non-optional use of the API
until it's properly tested, and ideally, supported more generally in nova.

> Can we consider adding this safety valve and un-reverting these patches?

No.  We're already well into the freeze and we can't afford risk or
distraction.  The time it took to deal with and discuss the issue this
caused is exactly why we're hesitant to approve FFEs at all.  It's a
distraction during critical time as we work toward the RC.

The focus right now has to be on high/critical priority bugs and
regressions.  We can revisit this properly in Juno.

-- 
Russell Bryant