<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 5, 2014 at 4:14 AM, Jay Pipes <span dir="ltr"><<a href="mailto:jaypipes@gmail.com" target="_blank">jaypipes@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Stackers,<br>
<br>
I'm looking to get consensus on a proposed API for server instance tagging in Nova:<br>
<br>
<a href="https://review.openstack.org/#/c/91444/" target="_blank">https://review.openstack.org/#<u></u>/c/91444/</a><br>
<br>
In the proposal, the REST API for the proposed server instance tagging looks like so:<br>
<br>
Remove a tag on a server:<br>
<br>
DELETE /v2/{project_id}/servers/{<u></u>server_id}/tags/{tag}<br>
<br>
It is this last API call that has drawn the attention of John Garbutt (cc'd). In Glance v2 API, if you attempt to delete a tag that does not exist, then a 404 Not Found is returned. In my proposal, if you attempt to delete a tag that does not exist for the server, a 204 No Content is returned.<br>

<br></blockquote><div><br></div><div>I think attempting to delete a tag that doesn't exist should return a 404. The user can always ignore the error if they know there's a<br>chance that the tag they want to get rid of doesn't exist. But for those that believe it should exist its an error is more useful to them as<br>
it may be an indicator that something is wrong with their logic.<br></div><div><br></div><div>Chris<br></div></div></div></div>