<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-cite-prefix">On 06/15/2014 07:18 PM, Aline Manera
wrote:<br>
</div>
<blockquote cite="mid:539D8106.9090008@linux.vnet.ibm.com"
type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
<div class="moz-cite-prefix">On 06/13/2014 06:16 AM, Hongliang
Wang wrote:<br>
</div>
<blockquote
cite="mid:1402651017-2940-2-git-send-email-hlwang@linux.vnet.ibm.com"
type="cite">
<pre wrap="">When user tries to create a resource with an existing name, we should return
HTTP 409 status code to tell the user and let him/her choose another one.
Signed-off-by: Hongliang Wang <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:hlwang@linux.vnet.ibm.com"><hlwang@linux.vnet.ibm.com></a>
---
src/kimchi/control/base.py | 2 ++
src/kimchi/exception.py | 4 ++++
2 files changed, 6 insertions(+)
diff --git a/src/kimchi/control/base.py b/src/kimchi/control/base.py
index f8a5210..a110ac2 100644
--- a/src/kimchi/control/base.py
+++ b/src/kimchi/control/base.py
@@ -276,6 +276,8 @@ class Collection(object):
raise cherrypy.HTTPError(400, e.message)
except NotFoundError, e:
raise cherrypy.HTTPError(404, e.message)
+ except ResourceAlreadyExists, e:
+ raise cherrypy.HTTPError(409, e.message)
except OperationFailed, e:
raise cherrypy.HTTPError(500, e.message)
except KimchiException, e:
diff --git a/src/kimchi/exception.py b/src/kimchi/exception.py
index fcf60cc..38daeab 100644
--- a/src/kimchi/exception.py
+++ b/src/kimchi/exception.py
@@ -91,3 +91,7 @@ class IsoFormatError(KimchiException):
class TimeoutExpired(KimchiException):
pass
+
+
+class ResourceAlreadyExists(KimchiException):
+ pass
</pre>
</blockquote>
<br>
<font face="DejaVu Sans Mono">And let's keep using
InvalidParameter for this proposal and if we get agreement 409
is better we change it for all the resources.<br>
That way we keep consistence between them.<br>
</font></blockquote>
<font face="DejaVu Sans Mono">ACK.</font><br>
<br>
</body>
</html>