openstack
/
deb-glance
Code Issues Proposed changes

Remove state transition from active to queued 

After commit Ieb03aaba887492819f9c58aa67f7acfcea81720e it is no
longer necessary to maintain a state transition from ACTIVE to
QUEUED. The only time this happened was when someone removed
the last location from an image. Since this was removed we
don't need it anymore.

Also, updates the image state transition images and docs to
represent the current state transitions.

Closes-Bug: #1555448

Change-Id: I390ae2a7110f06046f38914e946139b3b729bd2d
This commit is contained in:
Sabari Kumar Murugesan 2016-03-09 15:28:31 -08:00 committed by Sabari
parent f17fcd1a43
commit 93b81315c6
4 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

BIN
doc/source/images/image_status_transition.png
View File

Binary file not shown.
Side by Side Swipe Overlay

Before

Width:  |  Height:  |  Size: 153 KiB

After

Width:  |  Height:  |  Size: 164 KiB

4
doc/source/images_src/image_status_transition.dot
View File

@ -34,10 +34,10 @@ digraph {
"queued" -> "deleted" [label="delete"];
"saving" -> "active" [label="upload succeed"];
"saving" -> "killed" [label="upload fail"];
"saving" -> "killed" [label="[v1] upload fail"];
"saving" -> "queued" [label="[v2] upload fail"];
"saving" -> "deleted" [label="delete"];
"active" -> "queued" [label="remove location*"];
"active" -> "pending_delete" [label="delayed delete"];
"active" -> "deleted" [label="delete"];
"active" -> "deactivated" [label="deactivate"];

3
doc/source/statuses.rst
View File

@ -71,9 +71,6 @@ Images in Glance can be in one the following statuses:
* Add location from zero to more than one.
* Remove location from one or more to zero by PATCH method which is only
supported in v2.
Task Statuses
==============

2
glance/domain/__init__.py
View File

@ -103,7 +103,7 @@ class Image(object):
# can be retried.
'queued': ('saving', 'active', 'deleted'),
'saving': ('active', 'killed', 'deleted', 'queued'),
'active': ('queued', 'pending_delete', 'deleted', 'deactivated'),
'active': ('pending_delete', 'deleted', 'deactivated'),
'killed': ('deleted',),
'pending_delete': ('deleted',),
'deleted': (),