From 25b0a281ca2ed52ea2640e6e6a26047c4db6d921 Mon Sep 17 00:00:00 2001
From: Andreas Gruhler <andreas.gruhler@adfinis-sygroup.ch>
Date: Fri, 4 Oct 2019 08:17:23 +0200
Subject: [PATCH] add/remove compatibility functions

---
 proxmox/resource_lxc.go | 48 ++++++++++++++++++++++++-----------------
 proxmox/util.go         | 38 ++++++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+), 20 deletions(-)

diff --git a/proxmox/resource_lxc.go b/proxmox/resource_lxc.go
index f2ec98b..82e70d6 100644
--- a/proxmox/resource_lxc.go
+++ b/proxmox/resource_lxc.go
@@ -112,10 +112,6 @@ func resourceLxc() *schema.Resource {
 				Optional: true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
-						"id": {
-							Type:     schema.TypeInt,
-							Required: true,
-						},
 						"volume": {
 							Type:     schema.TypeString,
 							Required: true,
@@ -160,10 +156,6 @@ func resourceLxc() *schema.Resource {
 				Optional: true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
-						"id": {
-							Type:     schema.TypeInt,
-							Required: true,
-						},
 						"name": {
 							Type:     schema.TypeString,
 							Required: true,
@@ -342,14 +334,18 @@ func resourceLxcCreate(d *schema.ResourceData, meta interface{}) error {
 	// proxmox api allows multiple mountpoint sets,
 	// having a unique 'id' parameter foreach set
 	mountpoints := d.Get("mountpoint").(*schema.Set)
-	lxcMountpoints := DevicesSetToMap(mountpoints)
-	config.Mountpoints = lxcMountpoints
+	if len(mountpoints.List()) > 0 {
+		lxcMountpoints := DevicesSetToMapWithoutId(mountpoints)
+		config.Mountpoints = lxcMountpoints
+	}
 	config.Nameserver = d.Get("nameserver").(string)
 	// proxmox api allows multiple network sets,
 	// having a unique 'id' parameter foreach set
 	networks := d.Get("network").(*schema.Set)
-	lxcNetworks := DevicesSetToMap(networks)
-	config.Networks = lxcNetworks
+	if len(networks.List()) > 0 {
+		lxcNetworks := DevicesSetToMapWithoutId(networks)
+		config.Networks = lxcNetworks
+	}
 	config.OnBoot = d.Get("onboot").(bool)
 	config.OsType = d.Get("ostype").(string)
 	config.Password = d.Get("password").(string)
@@ -442,14 +438,18 @@ func resourceLxcUpdate(d *schema.ResourceData, meta interface{}) error {
 	// proxmox api allows multiple mountpoint sets,
 	// having a unique 'id' parameter foreach set
 	mountpoints := d.Get("mountpoint").(*schema.Set)
-	lxcMountpoints := DevicesSetToMap(mountpoints)
-	config.Mountpoints = lxcMountpoints
+	if len(mountpoints.List()) > 0 {
+		lxcMountpoints := DevicesSetToMapWithoutId(mountpoints)
+		config.Mountpoints = lxcMountpoints
+	}
 	config.Nameserver = d.Get("nameserver").(string)
 	// proxmox api allows multiple network sets,
 	// having a unique 'id' parameter foreach set
 	networks := d.Get("network").(*schema.Set)
-	lxcNetworks := DevicesSetToMap(networks)
-	config.Networks = lxcNetworks
+	if len(networks.List()) > 0 {
+		lxcNetworks := DevicesSetToMapWithoutId(networks)
+		config.Networks = lxcNetworks
+	}
 	config.OnBoot = d.Get("onboot").(bool)
 	config.OsType = d.Get("ostype").(string)
 	config.Password = d.Get("password").(string)
@@ -532,14 +532,22 @@ func resourceLxcRead(d *schema.ResourceData, meta interface{}) error {
 	d.Set("memory", config.Memory)
 
 	configMountpointSet := d.Get("mountpoint").(*schema.Set)
-	activeMountpointSet := UpdateDevicesSet(configMountpointSet, config.Mountpoints)
-	d.Set("mountpoint", activeMountpointSet)
+	configMountpointSet = AddIds(configMountpointSet)
+	if len(configMountpointSet.List()) > 0 {
+		activeMountpointSet := UpdateDevicesSet(configMountpointSet, config.Mountpoints)
+		activeMountpointSet = RemoveIds(activeMountpointSet)
+		d.Set("mountpoint", activeMountpointSet)
+	}
 
 	d.Set("nameserver", config.Nameserver)
 
 	configNetworksSet := d.Get("network").(*schema.Set)
-	activeNetworksSet := UpdateDevicesSet(configNetworksSet, config.Networks)
-	d.Set("network", activeNetworksSet)
+	configNetworksSet = AddIds(configNetworksSet)
+	if len(configNetworksSet.List()) > 0 {
+		activeNetworksSet := UpdateDevicesSet(configNetworksSet, config.Networks)
+		activeNetworksSet = RemoveIds(activeNetworksSet)
+		d.Set("network", activeNetworksSet)
+	}
 
 	d.Set("onboot", config.OnBoot)
 	d.Set("ostemplate", config.Ostemplate)
diff --git a/proxmox/util.go b/proxmox/util.go
index 9f00069..9bb884f 100644
--- a/proxmox/util.go
+++ b/proxmox/util.go
@@ -31,3 +31,41 @@ func UpdateDeviceConfDefaults(
 	return defaultDeviceConf
 }
 
+func DevicesSetToMapWithoutId(devicesSet *schema.Set) pxapi.QemuDevices {
+
+	devicesMap := pxapi.QemuDevices{}
+	i := 1
+	for _, set := range devicesSet.List() {
+		setMap, isMap := set.(map[string]interface{})
+		if isMap {
+			// setMap["id"] = i
+			devicesMap[i] = setMap
+			i += 1
+		}
+	}
+	return devicesMap
+}
+
+func AddIds(configSet *schema.Set) *schema.Set {
+	// add device config ids
+	var i = 1
+	for _, setConf := range configSet.List() {
+		configSet.Remove(setConf)
+		setConfMap := setConf.(map[string]interface{})
+		setConfMap["id"] = i
+		i += 1
+		configSet.Add(setConfMap)
+	}
+	return configSet
+}
+
+func RemoveIds(configSet *schema.Set) *schema.Set {
+	// remove device config ids
+	for _, setConf := range configSet.List() {
+		configSet.Remove(setConf)
+		setConfMap := setConf.(map[string]interface{})
+		delete(setConfMap, "id")
+		configSet.Add(setConfMap)
+	}
+	return configSet
+}
\ No newline at end of file