community images breaks Images v1 API

Bug #1658164 reported by Brian Rosmaita on 2017-01-20
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ian Cordasco

Bug Description

The community images code has broken some v1 functionality.

CI was reverted today by Change-Id: I03636966c6912af8fdc9fcfe49da3788e7316150

OSC reported failing functional tests. Test in question does an image-update in v1 to make an image public; test fails when the is_public value for the updated image is still false.

An exception isn't raised in the test, so it's not a permissions problem. Looks like the call is accepted but the visibility value isn't changed properly.

Fix proposed to branch: master

Changed in glance:
status: Triaged → In Progress

Submitter: Jenkins
Branch: master

commit 18acc704a1573346f751a2b8260720fd09e664cd
Author: Ian Cordasco <email address hidden>
Date: Fri Jan 20 19:21:52 2017 +0000

    Fix regression introduced by Community Images

    Updating an image in v1 skipped the work to ensure that the image dictionary
    would be Image v2 compliant. It was hidden inside an else clause and was only
    run when there was no image id provided. This meant that only sometimes would
    is_public be appropriately converted to visibility.

    A functional test has been added to prevent regression and the code has been
    mildly altered to fix the issue.

    Closes-bug: #1658164
    Change-Id: I996fbed2e31df8559c025cca31e5e12c4fb76548

Changed in glance:
status: In Progress → Fix Released

This issue was fixed in the openstack/glance development milestone.

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

Other bug subscribers