Merge pull request #291 from terraform-providers/soft-delete

Support for Gitlab EE soft-delete of groups and projects

Author: ringods

.github/workflows/acceptance-tests.yml

@@ -9,7 +9,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v1
       with:
-        go-version: 1.12
+        go-version: 1.14
       id: go
 
     - name: Check out code repository source code
@@ -36,7 +36,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v1
       with:
-        go-version: 1.12
+        go-version: 1.14
       id: go
 
     - name: Check out code repository source code
@@ -59,6 +59,7 @@ jobs:
         echo "::set-env name=GO111MODULE::on"
         echo "::set-env name=GITLAB_TOKEN::20char-testing-token"
         echo "::set-env name=GITLAB_BASE_URL::http://127.0.0.1:8080/api/v4"
+        echo "::set-env name=TF_LOG::DEBUG"
 
     - name: Start Gitlab and run acceptance tests
       run: |

.github/workflows/unit-tests.yml

@@ -7,7 +7,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        go: [1.12, 1.13, 1.14]
+        go: [1.13, 1.14]
         os: [ubuntu-latest, macos-latest, windows-latest]
         make_target: [test, vet]
 

.github/workflows/website.yml

@@ -9,7 +9,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v1
       with:
-        go-version: 1.12
+        go-version: 1.14
       id: go
 
     # https://help.github.com/en/actions/reference/workflow-commands-for-github-actions

.travis.yml

@@ -1,4 +1,11 @@
 ## 2.5.1 (Unreleased)
+
+BUGFIXES:
+* Support for soft-delete of groups and projects in Gitlab Enterprise Edition ([GH-282][GH-283][GH-285][GH-291])
+
+ENHANCEMENTS:
+* Switched from Travis CI to Github Actions ([GH-216])
+
 ## 2.5.0 (December 05, 2019)
 
 ENHANCEMENTS:

CHANGELOG.md

@@ -1,4 +1,4 @@
-TEST?=$$(go list ./... |grep -v 'vendor')
+TEST?=./gitlab
 GOFMT_FILES?=$$(find . -name '*.go' |grep -v vendor)
 WEBSITE_REPO=github.com/hashicorp/terraform-website
 PKG_NAME=gitlab
@@ -14,7 +14,7 @@ test: fmtcheck
 		xargs -t -n4 go test $(TESTARGS) -timeout=30s -parallel=4
 
 testacc: fmtcheck
-	TF_ACC=1 go test $(TEST) -v $(TESTARGS) -timeout 120m
+	TF_ACC=1 go test -v $(TEST) $(TESTARGS) -timeout 40m
 
 vet:
 	@echo "go vet ."

GNUmakefile

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"log"
 	"net/http"
+	"strings"
 	"time"
 
 	"github.com/hashicorp/terraform-plugin-sdk/helper/resource"
@@ -127,6 +128,11 @@ func resourceGitlabGroupRead(d *schema.ResourceData, meta interface{}) error {
 		}
 		return err
 	}
+	if group.MarkedForDeletionOn != nil {
+		log.Printf("[DEBUG] gitlab group %s is marked for deletion", d.Id())
+		d.SetId("")
+		return nil
+	}
 
 	d.SetId(fmt.Sprintf("%d", group.ID))
 	d.Set("name", group.Name)
@@ -190,7 +196,7 @@ func resourceGitlabGroupDelete(d *schema.ResourceData, meta interface{}) error {
 	log.Printf("[DEBUG] Delete gitlab group %s", d.Id())
 
 	_, err := client.Groups.DeleteGroup(d.Id())
-	if err != nil {
+	if err != nil && !strings.Contains(err.Error(), "Group has been already marked for deletion") {
 		return fmt.Errorf("error deleting group %s: %s", d.Id(), err)
 	}
 
@@ -208,6 +214,10 @@ func resourceGitlabGroupDelete(d *schema.ResourceData, meta interface{}) error {
 				log.Printf("[ERROR] Received error: %#v", err)
 				return out, "Error", err
 			}
+			if out.MarkedForDeletionOn != nil {
+				// Represents a Gitlab EE soft-delete
+				return out, "Deleted", nil
+			}
 			return out, "Deleting", nil
 		},
 

gitlab/resource_gitlab_group.go

@@ -119,7 +119,9 @@ func testAccCheckGitlabGroupLabelDestroy(s *terraform.State) error {
 		group, resp, err := conn.Groups.GetGroup(rs.Primary.ID)
 		if err == nil {
 			if group != nil && fmt.Sprintf("%d", group.ID) == rs.Primary.ID {
-				return fmt.Errorf("Group still exists")
+				if group.MarkedForDeletionOn == nil {
+					return fmt.Errorf("Group still exists")
+				}
 			}
 		}
 		if resp.StatusCode != 404 {

gitlab/resource_gitlab_group_label_test.go

@@ -137,21 +137,22 @@ func TestAccGitlabGroup_nested(t *testing.T) {
 					}),
 				),
 			},
-			{
-				Config: testAccGitlabNestedGroupConfig(rInt),
-				Check: resource.ComposeTestCheckFunc(
-					testAccCheckGitlabGroupExists("gitlab_group.foo", &group),
-					testAccCheckGitlabGroupExists("gitlab_group.foo2", &group2),
-					testAccCheckGitlabGroupExists("gitlab_group.nested_foo", &nestedGroup),
-					testAccCheckGitlabGroupAttributes(&nestedGroup, &testAccGitlabGroupExpectedAttributes{
-						Name:        fmt.Sprintf("nfoo-name-%d", rInt),
-						Path:        fmt.Sprintf("nfoo-path-%d", rInt),
-						Description: "Terraform acceptance tests",
-						LFSEnabled:  true,
-						Parent:      &group,
-					}),
-				),
-			},
+			// TODO In EE version, re-creating on the same path where a previous group was soft-deleted doesn't work.
+			// {
+			// 	Config: testAccGitlabNestedGroupConfig(rInt),
+			// 	Check: resource.ComposeTestCheckFunc(
+			// 		testAccCheckGitlabGroupExists("gitlab_group.foo", &group),
+			// 		testAccCheckGitlabGroupExists("gitlab_group.foo2", &group2),
+			// 		testAccCheckGitlabGroupExists("gitlab_group.nested_foo", &nestedGroup),
+			// 		testAccCheckGitlabGroupAttributes(&nestedGroup, &testAccGitlabGroupExpectedAttributes{
+			// 			Name:        fmt.Sprintf("nfoo-name-%d", rInt),
+			// 			Path:        fmt.Sprintf("nfoo-path-%d", rInt),
+			// 			Description: "Terraform acceptance tests",
+			// 			LFSEnabled:  true,
+			// 			Parent:      &group,
+			// 		}),
+			// 	),
+			// },
 		},
 	})
 }
@@ -264,7 +265,9 @@ func testAccCheckGitlabGroupDestroy(s *terraform.State) error {
 		group, resp, err := conn.Groups.GetGroup(rs.Primary.ID)
 		if err == nil {
 			if group != nil && fmt.Sprintf("%d", group.ID) == rs.Primary.ID {
-				return fmt.Errorf("Group still exists")
+				if group.MarkedForDeletionOn == nil {
+					return fmt.Errorf("Group still exists")
+				}
 			}
 		}
 		if resp.StatusCode != 404 {

gitlab/resource_gitlab_group_test.go

@@ -119,7 +119,9 @@ func testAccCheckGitlabGroupVariableDestroy(s *terraform.State) error {
 		_, resp, err := conn.Groups.GetGroup(rs.Primary.ID)
 		if err == nil {
 			//if gotRepo != nil && fmt.Sprintf("%d", gotRepo.ID) == rs.Primary.ID {
-			//	return fmt.Errorf("Repository still exists")
+			//	if gotRepo.MarkedForDeletionAt == nil {
+			//		return fmt.Errorf("Repository still exists")
+			//	}
 			//}
 		}
 		if resp.StatusCode != 404 {