<div dir="ltr">It does not seem very ReSTful--or very usable, for that matter--for a resource to be permanently modified when you a PUT fails. So I don't think we need the 'killed' status. It was purposefully left out of v2 images, which is not just a reskin of v1.<div>
<br></div><div>Perhaps there is still a bug where an image is getting stuck in 'saving' or some other state when a PUT fails? </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jan 26, 2014 at 5:10 AM, David Koo <span dir="ltr"><<a href="mailto:kpublicmail@gmail.com" target="_blank">kpublicmail@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Hi Fei,<br>
<br>
    Thanks for the confirmation.<br>
<div class="im"><br>
> I think you're right. The 'killed' status should be set in method upload()<br>
> if there is an upload failure, see<br>
> <a href="https://github.com/openstack/glance/blob/master/glance/common/utils.py#L244" target="_blank">https://github.com/openstack/glance/blob/master/glance/common/utils.py#L244</a><br>
<br>
</div>I think you meant:<br>
<br>
    <a href="https://github.com/openstack/glance/blob/master/glance/api/v1/upload_utils.py#L244" target="_blank">https://github.com/openstack/glance/blob/master/glance/api/v1/upload_utils.py#L244</a><br>
<br>
(the safe_kill() call) right?<br>
<br>
--<br>
Koo<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> ------------------ Original ------------------<br>
> From:  "David Koo"<<a href="mailto:kpublicmail@gmail.com">kpublicmail@gmail.com</a>>;<br>
> Date:  Jan 26, 2014<br>
> To:  "OpenStack Development Mailing<br>
> List"<<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>>;<br>
> Subject:  [openstack-dev] [Glance] Is the 'killed' state ever set in v2?<br>
><br>
> Hi All,<br>
><br>
> While trying to work on a bug I was trying to simulate some image<br>
> download failures and found that apparently the 'killed' state is never<br>
> set using v2 APIs.<br>
><br>
> If I understand correctly, a file upload goes to<br>
> api.v2.image_data.ImageDataController.upload and goes all the way to<br>
> store.ImageProxy.set_data which proceeds to write to the backend store.<br>
><br>
> If the backend store raises an exception it is simply propagated all the<br>
> way up. The notifier re-encodes the exceptions (which is the bug I was<br>
> looking at) but doesn't do anything about the image status.<br>
><br>
> Nowhere does the image status seem to get set to 'killed'.<br>
><br>
> Before I log a bug I just wanted to confirm with everybody whether or<br>
> not I've missed out on something.<br>
><br>
> Thanks.<br>
><br>
> --<br>
> Koo<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div>