From 0decd46dd069285e95278bc4f6f2a50fd4ba0e3a Mon Sep 17 00:00:00 2001
From: Matthew Farrellee <matt@redhat.com>
Date: Mon, 20 Jan 2014 12:06:01 -0500
Subject: [PATCH] Add node-group-template-create to CLI

node-group-template-create takes a json data structure and passes it
on to the Savanna API. The data structure is read from stdin by
default or from a file provided by the --json parameter.

Change-Id: I9f885df84726d6390e57e39ccafee75935c86c9d
Implements: blueprint python-savannaclient-cli
---
 savannaclient/api/shell.py | 21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

diff --git a/savannaclient/api/shell.py b/savannaclient/api/shell.py
index 3168b6b1..6796cff9 100644
--- a/savannaclient/api/shell.py
+++ b/savannaclient/api/shell.py
@@ -13,8 +13,11 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
+import argparse
+import inspect
 import json
 from savannaclient.nova import utils
+import sys
 
 
 def _print_list_field(field):
@@ -266,7 +269,7 @@ def do_cluster_delete(cs, args):
 #
 # node-group-template-show --name <template>|--id <template_id> [--json]
 #
-# TODO(mattf): node-group-template-create
+# node-group-template-create [--json <file>]
 #
 # node-group-template-delete --name <template>|--id <template_id>
 #
@@ -301,6 +304,22 @@ def do_node_group_template_show(cs, args):
         _show_node_group_template(template)
 
 
+@utils.arg('--json',
+           default=sys.stdin,
+           type=argparse.FileType('r'),
+           help='JSON representation of node group template')
+def do_node_group_template_create(cs, args):
+    """Create a node group template."""
+     # TODO(mattf): improve template validation, e.g. template w/o name key
+    template = json.loads(args.json.read())
+    valid_args = inspect.getargspec(cs.node_group_templates.create).args
+    for name in template.keys():
+        if name not in valid_args:
+            # TODO(mattf): make this verbose - bug/1271147
+            del template[name]
+    _show_node_group_template(cs.node_group_templates.create(**template))
+
+
 # TODO(mattf): Add --name
 #@utils.arg('--name',
 #           metavar='<template>',