Skip to content

Commit ac412ea

Browse files
authored
Remove RemoveResource from Create and Update (#2694)
1 parent d1a8f26 commit ac412ea

File tree

3 files changed

+3
-19
lines changed

3 files changed

+3
-19
lines changed

internal/service/encryptionatrest/resource.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -384,12 +384,8 @@ func (r *encryptionAtRestRS) Update(ctx context.Context, req resource.UpdateRequ
384384
return
385385
}
386386
projectID := encryptionAtRestState.ProjectID.ValueString()
387-
atlasEncryptionAtRest, atlasResp, err := connV2.EncryptionAtRestUsingCustomerKeyManagementApi.GetEncryptionAtRest(context.Background(), projectID).Execute()
387+
atlasEncryptionAtRest, _, err := connV2.EncryptionAtRestUsingCustomerKeyManagementApi.GetEncryptionAtRest(context.Background(), projectID).Execute()
388388
if err != nil {
389-
if resp != nil && atlasResp.StatusCode == http.StatusNotFound {
390-
resp.State.RemoveResource(ctx)
391-
return
392-
}
393389
resp.Diagnostics.AddError("error when getting encryption at rest resource during update", fmt.Sprintf(project.ErrorProjectRead, projectID, err.Error()))
394390
return
395391
}

internal/service/project/resource_project.go

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -402,12 +402,8 @@ func (r *projectRS) Create(ctx context.Context, req resource.CreateRequest, resp
402402
}
403403

404404
projectID := project.GetId()
405-
projectRes, atlasResp, err := connV2.ProjectsApi.GetProject(ctx, projectID).Execute()
405+
projectRes, _, err := connV2.ProjectsApi.GetProject(ctx, projectID).Execute()
406406
if err != nil {
407-
if resp != nil && atlasResp.StatusCode == http.StatusNotFound {
408-
resp.State.RemoveResource(ctx)
409-
return
410-
}
411407
resp.Diagnostics.AddError("error when getting project after create", fmt.Sprintf(ErrorProjectRead, projectID, err.Error()))
412408
return
413409
}
@@ -529,12 +525,8 @@ func (r *projectRS) Update(ctx context.Context, req resource.UpdateRequest, resp
529525
return
530526
}
531527

532-
projectRes, atlasResp, err := connV2.ProjectsApi.GetProject(ctx, projectID).Execute()
528+
projectRes, _, err := connV2.ProjectsApi.GetProject(ctx, projectID).Execute()
533529
if err != nil {
534-
if resp != nil && atlasResp.StatusCode == http.StatusNotFound {
535-
resp.State.RemoveResource(ctx)
536-
return
537-
}
538530
resp.Diagnostics.AddError("error when getting project after create", fmt.Sprintf(ErrorProjectRead, projectID, err.Error()))
539531
return
540532
}

internal/service/pushbasedlogexport/resource.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,6 @@ func (r *pushBasedLogExportRS) Create(ctx context.Context, req resource.CreateRe
6363
resp.Diagnostics.AddError("Error when unconfiguring push-based log export configuration", err.Error())
6464
return
6565
}
66-
67-
resp.State.RemoveResource(ctx)
6866
return
6967
}
7068

@@ -83,8 +81,6 @@ func (r *pushBasedLogExportRS) Create(ctx context.Context, req resource.CreateRe
8381
resp.Diagnostics.AddError("Error when unconfiguring push-based log export configuration", err.Error())
8482
return
8583
}
86-
87-
resp.State.RemoveResource(ctx)
8884
return
8985
}
9086

0 commit comments

Comments
 (0)