Fix RAPI network tag handling

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>

diff --git a/lib/rapi/rlib2.py b/lib/rapi/rlib2.py
index 321ddb5..9b3dbaa 100644 (file)
--- a/lib/rapi/rlib2.py
+++ b/lib/rapi/rlib2.py
@@ -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):