Image remains in queued status after location set via PATCH

[iain MacDonnell]

Pike release, with show_image_direct_url and show_multiple_locations enabled.

Attempting to create an image using the HTTP backend with the glance v2 API. I create a new/blank image (goes into "queued" status), then set the location with:

curl -g -i -X PATCH -H 'Accept-Encoding: gzip, deflate' -H 'Accept: */*' -H 'User-Agent: imacdonn-getting-dangerous' -H 'X-Auth-Token: xxx' -H 'Content-Type: application/openstack-images-v2.1-json-patch' -d '[{"op":"replace", "path": "/locations", "value": [{"url": "http://my_http_server/cirros.img", "metadata": {}}]}]' http://my_glance_api_endpoint:9292/v2/images/e5581f14-2d05-4ae7-8d78-9da42731a37e

This results in the direct_url getting set correctly, and the size of the image is correctly determined, but the image remains in "queued" status. It should become "active".

[Brian Rosmaita (brian-rosmaita)]

Slightly off topic, but it's kind of overkill to have to expose locations in order to use the http store in this way. Maybe what we need is a 'web-reference' (instead of web-download) import method to do this.

[iain MacDonnell (imacdonn)]

Yes, please! I was considering opening a separate bug to request something like that...

[Brian Rosmaita (brian-rosmaita)]

Also note that the checksum is None and there's no way to set it in the v2 API.

[Erno Kuvaja (jokke)]

By the looks of it the replace call indeed does not touch the image state. Add call 'thjough will change the state from 'queued' to 'active'. You might want to try out "op":"add","path":"/locations/0"

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/592775
Committed: https://git.openstack.org/cgit/openstack/glance/commit/?id=e2c54d58b33c7350c78629cc7726b5fa4509637b
Submitter: Zuul
Branch: master

commit e2c54d58b33c7350c78629cc7726b5fa4509637b
Author: imacdonn <email address hidden>
Date: Fri Aug 17 00:55:08 2018 +0000

    Update status to active when locations replaced

    Image status should be changed from 'queued' to 'active' when
    'locations' has been successfully replaced.

    Change-Id: I966688e67475e00a1eeb0cbe1632d5c811e41e4f
    Closes-Bug: 1750892

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/598204

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/598253

[OpenStack Infra (hudson-openstack)]

Change abandoned by iain MacDonnell (<email address hidden>) on branch: stable/rocky
Review: https://review.openstack.org/598204
Reason: Project can't decide if this is the right thing to do or not, so dropping backports.

[OpenStack Infra (hudson-openstack)]

Change abandoned by iain MacDonnell (<email address hidden>) on branch: stable/queens
Review: https://review.openstack.org/598253
Reason: Project can't decide if this is the right thing to do or not, so dropping backports.

[OpenStack Infra (hudson-openstack)]

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