TER-240: Add force-destroy option

ghost/import_ghost_app_test.go

@@ -21,9 +21,10 @@ func TestAccGhostAppImportBasic(t *testing.T) {
 			},
 
 			resource.TestStep{
-				ResourceName:      "ghost_app.test",
-				ImportState:       true,
-				ImportStateVerify: true,
+				ResourceName:            "ghost_app.test",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"force_destroy"},
 			},
 		},
 	})

ghost/resource_ghost_app.go

@@ -471,6 +471,12 @@ func resourceGhostApp() *schema.Resource {
 				Type:     schema.TypeString,
 				Computed: true,
 			},
+			"force_destroy": {
+				Type:        schema.TypeBool,
+				Optional:    true,
+				Default:     false,
+				Description: "Force destroying the app even if it's has been updated from cloud-deploy",
+			},
 		},
 	}
 }
@@ -515,9 +521,16 @@ func resourceGhostAppUpdate(d *schema.ResourceData, meta interface{}) error {
 
 	log.Printf("[INFO] Updating Ghost app %s", d.Get("name").(string))
 
-	app := expandGhostApp(d)
+	app_updated := expandGhostApp(d)
+
+	// Get last etag
+	app, err := client.GetApp(d.Id())
+	if err != nil {
+		return fmt.Errorf("[ERROR] error updating Ghost app: %v", err)
+	}
+	d.Set("eve_etag", app.Etag)
 
-	eveMetadata, err := client.UpdateApp(&app, d.Id(), d.Get("eve_etag").(string))
+	eveMetadata, err := client.UpdateApp(&app_updated, d.Id(), d.Get("eve_etag").(string))
 	if err != nil {
 		return fmt.Errorf("[ERROR] error updating Ghost app: %v", err)
 	}
@@ -532,8 +545,22 @@ func resourceGhostAppDelete(d *schema.ResourceData, meta interface{}) error {
 
 	log.Printf("[INFO] Deleting Ghost app %s", d.Get("name").(string))
 
+	if d.Get("force_destroy").(bool) {
+		// Get last Etag
+		app, err := client.GetApp(d.Id())
+		if err != nil {
+			return fmt.Errorf("[ERROR] error deleting Ghost app: %v", err)
+		}
+		d.Set("eve_etag", app.Etag)
+	}
+
+	log.Printf("[INFO] %s\n\n\n\n", d.Get("eve_etag").(string))
 	err := client.DeleteApp(d.Id(), d.Get("eve_etag").(string))
 	if err != nil {
+		ec := err.Error()[len(err.Error())-3:]
+		if ec == "412" {
+			return fmt.Errorf("[ERROR] error deleting Ghost app: app has been updated, use force_destroy = true: %v", err)
+		}
 		return fmt.Errorf("[ERROR] error deleting Ghost app: %v", err)
 	}
 
@@ -574,7 +601,6 @@ func flattenGhostApp(d *schema.ResourceData, app ghost.App) error {
 	d.Set("instance_type", app.InstanceType)
 	d.Set("vpc_id", app.VpcID)
 	d.Set("instance_monitoring", app.InstanceMonitoring)
-	d.Set("eve_etag", app.Etag)
 
 	d.Set("modules", flattenGhostAppModules(app.Modules))
 	d.Set("build_infos", flattenGhostAppBuildInfos(app.BuildInfos))
@@ -990,3 +1016,10 @@ func flattenGhostAppStringList(strings []string) []interface{} {
 
 	return stringList
 }
+
+func resourceGhostAppImportState(d *schema.ResourceData, m interface{}) ([]*schema.ResourceData, error) {
+	// Force destroy can't be fetched from API calls so it needs a default value
+	d.Set("force_destroy", false)
+
+	return []*schema.ResourceData{d}, nil
+}