Fix identity domain destruction by deactivating the identity domain before deletion

Author: F21

Diff for: internal/service/identity/identity_domain_resource.go

@@ -573,21 +573,42 @@ func (s *IdentityDomainResourceCrud) Update() error {
 }
 
 func (s *IdentityDomainResourceCrud) Delete() error {
-	request := oci_identity.DeleteDomainRequest{}
-
 	tmp := s.D.Id()
-	request.DomainId = &tmp
 
-	request.RequestMetadata.RetryPolicy = tfresource.GetRetryPolicy(s.DisableNotFoundRetries, "identity")
+	deactivateRequest := oci_identity.DeactivateDomainRequest{}
+
+	deactivateRequest.DomainId = &tmp
+
+	deactivateRequest.RequestMetadata.RetryPolicy = tfresource.GetRetryPolicy(s.DisableNotFoundRetries, "identity")
 
-	response, err := s.Client.DeleteDomain(context.Background(), request)
+	deactivateResponse, err := s.Client.DeactivateDomain(context.Background(), deactivateRequest)
 	if err != nil {
 		return err
 	}
 
-	workId := response.OpcWorkRequestId
+	deactivateWorkId := deactivateResponse.OpcWorkRequestId
+	// Wait until it finishes
+	_, deactivateWorkRequestErr := domainWaitForWorkRequest(deactivateWorkId, "domain",
+		oci_identity.IamWorkRequestResourceActionTypeUpdated, s.D.Timeout(schema.TimeoutDelete), s.DisableNotFoundRetries, s.Client)
+
+	if deactivateWorkRequestErr != nil {
+		return deactivateWorkRequestErr
+	}
+
+	deleteRequest := oci_identity.DeleteDomainRequest{}
+
+	deleteRequest.DomainId = &tmp
+
+	deleteRequest.RequestMetadata.RetryPolicy = tfresource.GetRetryPolicy(s.DisableNotFoundRetries, "identity")
+
+	deleteResponse, err := s.Client.DeleteDomain(context.Background(), deleteRequest)
+	if err != nil {
+		return err
+	}
+
+	deleteWorkId := deleteResponse.OpcWorkRequestId
 	// Wait until it finishes
-	_, delWorkRequestErr := domainWaitForWorkRequest(workId, "domain",
+	_, delWorkRequestErr := domainWaitForWorkRequest(deleteWorkId, "domain",
 		oci_identity.IamWorkRequestResourceActionTypeUpdated, s.D.Timeout(schema.TimeoutDelete), s.DisableNotFoundRetries, s.Client)
 	return delWorkRequestErr
 }