added fix for cascading failure

Author: Uddipaan Hazarika

internal/provider/commons.go

@@ -51,6 +51,7 @@ var updatableVdbKeys = map[string]bool{
 	"new_dbid":                      true,
 	"mount_point":                   true,
 	"tags":                          true,
+	"database_name":                 true,
 }
 
 var isDestructiveVdbUpdate = map[string]bool{

internal/provider/resource_vdb.go

@@ -92,8 +92,8 @@ func resourceVdb() *schema.Resource {
 			},
 			"database_name": {
 				Type:     schema.TypeString,
-				Optional: true,
 				Computed: true,
+				Optional: true,
 			},
 			"cdb_id": {
 				Type:     schema.TypeString,
@@ -1594,6 +1594,7 @@ func resourceVdbRead(ctx context.Context, d *schema.ResourceData, meta interface
 	if !*result.IsAppdata {
 		d.Set("database_name", result.GetDatabaseName())
 	}
+
 	d.Set("tags", flattenTags(result.GetTags()))
 	d.Set("vdb_restart", result.GetVdbRestart())
 

internal/provider/utility.go

@@ -5,6 +5,7 @@ import (
 	"io"
 	"math"
 	"net/http"
+	"reflect"
 	"strconv"
 	"time"
 
@@ -285,7 +286,9 @@ func enableVDB(ctx context.Context, client *dctapi.APIClient, vdbId string) diag
 func revertChanges(d *schema.ResourceData, changedKeys []string) {
 	for _, key := range changedKeys {
 		old, _ := d.GetChange(key)
-		d.Set(key, old)
+		if !reflect.ValueOf(old).IsZero() { // so that a previously optional param is not set to blank erroraneously
+			d.Set(key, old)
+		}
 	}
 }