From 9488f6f4a43bf07878d8f693270ce5705139cde5 Mon Sep 17 00:00:00 2001
From: Grant Gongaware <grant@telmate.com>
Date: Tue, 1 Aug 2017 16:00:55 -0700
Subject: [PATCH] reconnect provisioner and more docs

---
 README.md              | 43 +++++++++++++++++++++++++++++++++++++++---
 proxmox/provisioner.go | 14 ++++++++++++++
 2 files changed, 54 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 38b1cf8..2774ab0 100644
--- a/README.md
+++ b/README.md
@@ -37,7 +37,7 @@ resource "proxmox_vm_qemu" "test" {
 	name = "tftest1.xyz.com"
 	desc = "tf description"
 	target_node = "proxmox1-xx"
-	
+
 	clone = "terraform-ubuntu1404-template"
 	storage = "local"
 	cores = 3
@@ -50,10 +50,10 @@ resource "proxmox_vm_qemu" "test" {
 	ssh_user = "terraform"
 	ssh_private_key = <<EOF
 -----BEGIN RSA PRIVATE KEY-----
-private ssh key terraform 
+private ssh key terraform
 -----END RSA PRIVATE KEY-----
 EOF
-	
+
 	os_type = "ubuntu"
 	os_network_config = <<EOF
 auto eth0
@@ -72,5 +72,42 @@ EOF
 }
 
 ```
+### Provider usage
+You can start from either an ISO or clone an existing VM.
+
+Optimally, you could create a VM resource you will use a clone base with an ISO, and make the rest of the VM resources depend on that base "template" and clone it.
+
+Interesting parameters:
+
+**ssh_forward_ip** - should be the IP or hostname of the target node or bridge IP. This is where proxmox will create a port forward to your VM with via a user_net.
+
+**os_type** - ubuntu (https://github.com/Telmate/terraform-ubuntu-proxmox-iso) or centos (TODO: centos iso template)
 
 
+### Preprovision (internal)
+
+There is a pre-provision phase which is used to set a hostname, intialize eth0, and resize the VM disk to available space. This is done over SSH with the ssh_forward_ip, ssh_user and ssh_private_key.
+
+Disk resize is done if the file /etc/auto_resize_vda.sh exists. Source: https://github.com/Telmate/terraform-ubuntu-proxmox-iso/blob/master/auto_resize_vda.sh
+
+### Provisioner usage
+
+
+Remove the temporary net1 adapter.
+Inside the VM this usually triggers the routes back to the provisioning machine on net0.
+```
+	provisioner "proxmox" {
+		action = "sshbackward"
+	}
+
+```
+
+Replace the temporary net1 adapter with a new persistent net1.
+```
+	provisioner "proxmox" {
+		action = "reconnect"
+		net1 = "virtio,bridge=vmbr0,tag=99"
+	}
+
+```
+If net1 needs a config other than DHCP you should prior to this use provisioner "remote-exec" to modify the network config.
diff --git a/proxmox/provisioner.go b/proxmox/provisioner.go
index 17b6937..186a072 100644
--- a/proxmox/provisioner.go
+++ b/proxmox/provisioner.go
@@ -6,6 +6,7 @@ import (
 	pxapi "github.com/Telmate/proxmox-api-go/proxmox"
 	"github.com/hashicorp/terraform/helper/schema"
 	"github.com/hashicorp/terraform/terraform"
+	"time"
 )
 
 func Provisioner() terraform.ResourceProvisioner {
@@ -51,6 +52,19 @@ func applyFn(ctx context.Context) error {
 	switch act {
 	case "sshbackward":
 		return pxapi.RemoveSshForwardUsernet(vmr, client)
+
+	case "reconnect":
+		err = pxapi.RemoveSshForwardUsernet(vmr, client)
+		if err != nil {
+			return err
+		}
+		time.Sleep(5 * time.Second)
+		vmParams := map[string]string{
+			"net1": data.Get("net1").(string),
+		}
+		_, err = client.SetVmConfig(vmr, vmParams)
+
+		return err
 	default:
 		return fmt.Errorf("Unkown action: %s", act)
 	}