fix: migrate the level in the approval expression

ecmadao · ecmadao · commit 3a09a6580f77 · 2025-10-31T11:07:54.000+08:00
diff --git a/VERSION b/VERSION
@@ -1 +1 @@
-3.11.3
+3.11.4
diff --git a/api/setting.go b/api/setting.go
diff --git a/examples/database/main.tf b/examples/database/main.tf
@@ -2,7 +2,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/database_group/main.tf b/examples/database_group/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/environments/main.tf b/examples/environments/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/groups/main.tf b/examples/groups/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/iamPolicy/main.tf b/examples/iamPolicy/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/instances/main.tf b/examples/instances/main.tf
@@ -2,7 +2,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/policies/main.tf b/examples/policies/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/projects/main.tf b/examples/projects/main.tf
@@ -2,7 +2,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/risk/main.tf b/examples/risk/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/roles/main.tf b/examples/roles/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/settings/main.tf b/examples/settings/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/setup/main.tf b/examples/setup/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/sql_review/main.tf b/examples/sql_review/main.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/examples/users/main.tf b/examples/users/main.tf
@@ -2,7 +2,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }
diff --git a/provider/data_source_setting.go b/provider/data_source_setting.go
@@ -558,10 +558,10 @@ func getWorkspaceApprovalSetting(computed bool) *schema.Schema {
 											Computed: computed,
 											Optional: true,
 											ValidateFunc: validation.StringInSlice([]string{
-												string(api.RiskLevelDefault),
-												string(api.RiskLevelLow),
-												string(api.RiskLevelModerate),
-												string(api.RiskLevelHigh),
+												v1pb.RiskLevel_LOW.String(),
+												v1pb.RiskLevel_MODERATE.String(),
+												v1pb.RiskLevel_HIGH.String(),
+												v1pb.RiskLevel_RISK_LEVEL_UNSPECIFIED.String(),
 											}, false),
 										},
 									},
@@ -678,22 +678,8 @@ func parseApprovalExpression(callExpr *v1alpha1.Expr_Call) ([]map[string]interfa
 
 			argName := argExpr.Args[0].GetIdentExpr().Name
 			switch argName {
-			case "source":
+			case "source", "level":
 				resp[argName] = argExpr.Args[1].GetConstExpr().GetStringValue()
-			case "level":
-				levelNumber := argExpr.Args[1].GetConstExpr().GetInt64Value()
-				switch int(levelNumber) {
-				case api.RiskLevelDefault.Int():
-					resp[argName] = api.RiskLevelDefault
-				case api.RiskLevelLow.Int():
-					resp[argName] = api.RiskLevelLow
-				case api.RiskLevelModerate.Int():
-					resp[argName] = api.RiskLevelModerate
-				case api.RiskLevelHigh.Int():
-					resp[argName] = api.RiskLevelHigh
-				default:
-					return nil, errors.Errorf("unknown risk level: %v", levelNumber)
-				}
 			default:
 				return nil, errors.Errorf("unsupport arg: %v", argName)
 			}
diff --git a/provider/resource_setting.go b/provider/resource_setting.go
@@ -359,8 +359,7 @@ func convertToV1ApprovalSetting(d *schema.ResourceData) (*v1pb.WorkspaceApproval
 		buildCondition := []string{}
 		for _, condition := range conditionList {
 			rawCondition := condition.(map[string]interface{})
-			rawLevel := rawCondition["level"].(string)
-			buildCondition = append(buildCondition, fmt.Sprintf(`source == "%s" && level == %d`, rawCondition["source"].(string), api.RiskLevel(rawLevel).Int()))
+			buildCondition = append(buildCondition, fmt.Sprintf(`(source == "%s" && level == "%s")`, rawCondition["source"].(string), rawCondition["level"].(string)))
 		}
 		expression := strings.Join(buildCondition, " || ")
 
diff --git a/tutorials/0-provider.tf b/tutorials/0-provider.tf
@@ -1,7 +1,7 @@
 terraform {
   required_providers {
     bytebase = {
-      version = "3.11.3"
+      version = "3.11.4"
       # For local development, please use "terraform.local/bytebase/bytebase" instead
       source = "registry.terraform.io/bytebase/bytebase"
     }

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`terraform {`
`3`	`3`	`required_providers {`
`4`	`4`	`bytebase = {`
`5`		`- version = "3.11.3"`
	`5`	`+ version = "3.11.4"`
`6`	`6`	`# For local development, please use "terraform.local/bytebase/bytebase" instead`
`7`	`7`	`source = "registry.terraform.io/bytebase/bytebase"`
`8`	`8`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`terraform {`
`2`	`2`	`required_providers {`
`3`	`3`	`bytebase = {`
`4`		`- version = "3.11.3"`
	`4`	`+ version = "3.11.4"`
`5`	`5`	`# For local development, please use "terraform.local/bytebase/bytebase" instead`
`6`	`6`	`source = "registry.terraform.io/bytebase/bytebase"`
`7`	`7`	`}`