From 94501f400fda6b3c196f79c8f34ab814f54fa037 Mon Sep 17 00:00:00 2001
From: Joshua Boniface <joshua@boniface.me>
Date: Fri, 11 Oct 2019 23:52:20 -0400
Subject: [PATCH] Remove vm add command

This is likely not going to be used with the planned implementation of
the automatic provisioning daemon, which will be either an API client or
direct Python binding client.
---
 client-cli/pvc.py | 51 -----------------------------------------------
 1 file changed, 51 deletions(-)

diff --git a/client-cli/pvc.py b/client-cli/pvc.py
index eaa81a14..88cb8288 100755
--- a/client-cli/pvc.py
+++ b/client-cli/pvc.py
@@ -211,56 +211,6 @@ def cli_vm():
     """
     pass
 
-###############################################################################
-# pvc vm add
-###############################################################################
-@click.command(name='add', short_help='Add a new virtual machine to the provisioning queue.')
-@click.option(
-    '--target', 'target_node',
-    help='Home node for this domain; autodetect if unspecified.'
-)
-@click.option(
-    '--cluster', 'is_cluster',
-    is_flag=True,
-    help='Create a cluster VM.'
-)
-@click.option(
-    '--system-template', 'system_template',
-    required=True,
-    help='System resource template for this domain.'
-)
-@click.option(
-    '--network-template', 'network_template',
-    required=True,
-    help='Network resource template for this domain.'
-)
-@click.option(
-    '--storage-template', 'storage_template',
-    required=True,
-    help='Storage resource template for this domain.'
-)
-@click.argument(
-    'vmname'
-)
-def vm_add(vmname, target_node, is_cluster, system_template, network_template, storage_template):
-    """
-    Add a new VM VMNAME to the provisioning queue.
-
-    Note: Cluster VMs are those which will only run on Coordinator hosts. Usually, these VMs will use the 'cluster' network template, or possibly a custom template including the upstream network as well. Use these sparingly, as they are designed primarily for cluster control or upstream bridge VMs.
-    """
-
-    zk_conn = pvc_common.startZKConnection(zk_host)
-    retcode, retmsg = pvc_provisioner.add_vm(
-        zk_conn,
-        vmname=vmname,
-        target_node=target_node,
-        is_cluster=is_cluster,
-        system_template=system_template,
-        network_template=network_template,
-        storage_template=storage_template
-    )
-    cleanup(retcode, retmsg, zk_conn)
-
 ###############################################################################
 # pvc vm define
 ###############################################################################
@@ -1823,7 +1773,6 @@ cli_node.add_command(node_unflush)
 cli_node.add_command(node_info)
 cli_node.add_command(node_list)
 
-cli_vm.add_command(vm_add)
 cli_vm.add_command(vm_define)
 cli_vm.add_command(vm_modify)
 cli_vm.add_command(vm_undefine)