Fix RAPI network tag handling
authorHrvoje Ribicic <riba@google.com>
Mon, 16 Dec 2013 13:21:04 +0000 (14:21 +0100)
committerHrvoje Ribicic <riba@google.com>
Mon, 16 Dec 2013 15:31:37 +0000 (16:31 +0100)
The network tags were absent from an if check used to actually list
tags. The patch fixes the oversight, and adds a proper error message in
case the issue occurs again for a new tag type.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Michele Tartara <mtartara@google.com>

lib/rapi/rlib2.py

index 321ddb5..9b3dbaa 100644 (file)
@@ -1500,7 +1500,8 @@ class _R_Tags(baserlib.OpcodeResource):
 
     if kind in (constants.TAG_INSTANCE,
                 constants.TAG_NODEGROUP,
-                constants.TAG_NODE):
+                constants.TAG_NODE,
+                constants.TAG_NETWORK):
       if not self.name:
         raise http.HttpBadRequest("Missing name on tag request")
 
@@ -1513,6 +1514,9 @@ class _R_Tags(baserlib.OpcodeResource):
       ssc = ssconf.SimpleStore()
       tags = ssc.GetClusterTags()
 
+    else:
+      raise http.HttpBadRequest("Unhandled tag type!")
+
     return list(tags)
 
   def GetPutOpInput(self):