Merge pull request #1244 from Tinyblargon/1182

feat: return same case for MAC

proxmox/Internal/resource/guest/qemu/network/schema.go

@@ -65,11 +65,6 @@ func Schema() *schema.Schema {
 					Type:     schema.TypeString,
 					Optional: true,
 					Computed: true,
-					DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
-						oldMAC, _ := net.ParseMAC(old)
-						newMAC, _ := net.ParseMAC(new)
-						return oldMAC.String() == newMAC.String()
-					},
 					ValidateDiagFunc: func(i interface{}, p cty.Path) diag.Diagnostics {
 						v := i.(string)
 						if _, err := net.ParseMAC(v); err != nil {

proxmox/Internal/resource/guest/qemu/network/terraform.go

@@ -1,13 +1,20 @@
 package network
 
 import (
+	"net"
+
 	pveAPI "github.com/Telmate/proxmox-api-go/proxmox"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 )
 
 // Converts the SDK configuration to the Terraform configuration
 func Terraform(config pveAPI.QemuNetworkInterfaces, d *schema.ResourceData) {
 	paramArray := make([]interface{}, len(config))
+	tfConfig := d.Get(Root).([]interface{})
+	tfMap := make(map[int]interface{}, len(tfConfig))
+	for i := range tfConfig {
+		tfMap[tfConfig[i].(map[string]interface{})[schemaID].(int)] = tfConfig[i]
+	}
 	var index int
 	for i := 0; i < AmountNetworkInterfaces; i++ {
 		v, ok := config[pveAPI.QemuNetworkInterfaceID(i)]
@@ -26,7 +33,17 @@ func Terraform(config pveAPI.QemuNetworkInterfaces, d *schema.ResourceData) {
 			params[schemaFirewall] = *v.Firewall
 		}
 		if v.MAC != nil {
-			params[schemaMAC] = v.MAC.String()
+			if vv, ok := tfMap[i]; ok {
+				tfMAC := vv.(map[string]interface{})[schemaMAC].(string)
+				mac, _ := net.ParseMAC(tfMAC)
+				if mac.String() == v.MAC.String() {
+					params[schemaMAC] = tfMAC
+				} else {
+					params[schemaMAC] = v.MAC.String()
+				}
+			} else {
+				params[schemaMAC] = v.MAC.String()
+			}
 		}
 		if v.MTU != nil {
 			if v.MTU.Inherit {