Enable usage of tags in unleash_feature_v2

Author: jdelucaa

docs/resources/feature_v2.md

@@ -44,6 +44,16 @@ resource "unleash_feature_v2" "with_env_strategies" {
       }
     }
   }
+
+  tag {
+    type  = "simple"
+    value = "foo"
+  }
+
+  tag {
+    type  = "simple"
+    value = "bar"
+  }
 }
 ```
 
@@ -61,6 +71,7 @@ resource "unleash_feature_v2" "with_env_strategies" {
 - `archive_on_destroy` (Boolean) Whether to archive the feature toggle on destroy. Default is `true`. When `false`, it will permanently delete the feature toggle.
 - `description` (String) Feature description
 - `environment` (Block List) Use this to enable a feature in an environment and add strategies (see [below for nested schema](#nestedblock--environment))
+- `tag` (Block List) Tag to add to the feature (see [below for nested schema](#nestedblock--tag))
 - `variant` (Block List) Feature variant (see [below for nested schema](#nestedblock--variant))
 
 ### Read-Only
@@ -77,7 +88,7 @@ Required:
 Optional:
 
 - `enabled` (Boolean) Whether the feature is on/off in the environment. Default is `true` (on)
-- `strategy` (Block Set) Strategy to add in the environment (see [below for nested schema](#nestedblock--environment--strategy))
+- `strategy` (Block List) Strategy to add in the environment (see [below for nested schema](#nestedblock--environment--strategy))
 
 <a id="nestedblock--environment--strategy"></a>
 ### Nested Schema for `environment.strategy`
@@ -96,6 +107,18 @@ Read-Only:
 
 
 
+<a id="nestedblock--tag"></a>
+### Nested Schema for `tag`
+
+Required:
+
+- `value` (String) Tag value.
+
+Optional:
+
+- `type` (String) Tag type. Default is `simple`.
+
+
 <a id="nestedblock--variant"></a>
 ### Nested Schema for `variant`
 

examples/resources/unleash_feature_v2/resource.tf

@@ -29,4 +29,14 @@ resource "unleash_feature_v2" "with_env_strategies" {
       }
     }
   }
+
+  tag {
+    type  = "simple"
+    value = "foo"
+  }
+
+  tag {
+    type  = "simple"
+    value = "bar"
+  }
 }

go.mod

@@ -5,7 +5,7 @@ go 1.18
 require (
 	github.com/hashicorp/terraform-plugin-docs v0.14.1
 	github.com/hashicorp/terraform-plugin-sdk/v2 v2.26.1
-	github.com/philips-labs/go-unleash-api v1.0.2
+	github.com/philips-labs/go-unleash-api v1.0.5
 )
 
 require (

go.sum

@@ -184,8 +184,8 @@ github.com/mitchellh/reflectwalk v1.0.2/go.mod h1:mSTlrgnPZtwu0c4WaC2kGObEpuNDbx
 github.com/niemeyer/pretty v0.0.0-20200227124842-a10e7caefd8e/go.mod h1:zD1mROLANZcx1PVRCS0qkT7pwLkGfwJo4zjcN/Tysno=
 github.com/oklog/run v1.0.0 h1:Ru7dDtJNOyC66gQ5dQmaCa0qIsAUFY3sFpK1Xk8igrw=
 github.com/oklog/run v1.0.0/go.mod h1:dlhp/R75TPv97u0XWUtDeV/lRKWPKSdTuV0TZvrmrQA=
-github.com/philips-labs/go-unleash-api v1.0.2 h1:GTbeJ+LfohrpXzj/qg89yyZHupEAPX8Wxq3z4sigzi8=
-github.com/philips-labs/go-unleash-api v1.0.2/go.mod h1:7iJbdxlvMu9VlRLpitWFavCifH8IsfYPc7EkDY3PFv4=
+github.com/philips-labs/go-unleash-api v1.0.5 h1:eESdVQbH7szoLfe8l0JjCuQWoL7uKVJ4u2Fjt15B0Tw=
+github.com/philips-labs/go-unleash-api v1.0.5/go.mod h1:7iJbdxlvMu9VlRLpitWFavCifH8IsfYPc7EkDY3PFv4=
 github.com/pkg/errors v0.8.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
 github.com/pkg/errors v0.9.1/go.mod h1:bwawxfHBFNV+L2hUp1rHADufV3IMtnDRdf1r5NINEl0=
 github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=

internal/provider/resource_feature_v2.go

@@ -168,6 +168,26 @@ func resourceFeatureV2() *schema.Resource {
 					},
 				},
 			},
+			"tag": {
+				Description: "Tag to add to the feature",
+				Type:        schema.TypeList,
+				Optional:    true,
+				Elem: &schema.Resource{
+					Schema: map[string]*schema.Schema{
+						"type": {
+							Description: "Tag type. Default is `simple`.",
+							Type:        schema.TypeString,
+							Optional:    true,
+							Default:     "simple",
+						},
+						"value": {
+							Description: "Tag value.",
+							Type:        schema.TypeString,
+							Required:    true,
+						},
+					},
+				},
+			},
 		},
 	}
 }
@@ -228,6 +248,19 @@ func resourceFeatureV2Create(ctx context.Context, d *schema.ResourceData, meta i
 			}
 		}
 	}
+	if t, ok := d.GetOk("tag"); ok {
+		tfTags := t.([]interface{})
+		for _, tfTag := range tfTags {
+			tag := toFeatureTag(tfTag.(map[string]interface{}))
+			_, resp, err := client.FeatureTags.CreateFeatureTags(feature.Name, tag)
+			if resp == nil || err != nil {
+				client.FeatureToggles.ArchiveFeature(feature.Project, feature.Name)
+				client.FeatureToggles.DeleteArchivedFeature(feature.Name)
+				return diag.FromErr(err)
+			}
+		}
+
+	}
 
 	d.SetId(createdFeature.Name)
 	readDiags := resourceFeatureV2Read(ctx, d, meta)
@@ -273,6 +306,23 @@ func resourceFeatureV2Read(ctx context.Context, d *schema.ResourceData, meta int
 		_ = d.Set("environment", flattenEnvironments(toSave))
 	}
 
+	if t, ok := d.GetOk("tag"); ok {
+		featureTags, _, err := client.FeatureTags.GetAllFeatureTags(feature.Name)
+		if err != nil {
+			return diag.FromErr(err)
+		}
+		toSave := []api.FeatureTag{}
+		for _, tfTag := range t.([]interface{}) {
+			for _, tag := range featureTags.Tags {
+				if tfTag.(map[string]interface{})["value"] == tag.Value {
+					toSave = append(toSave, tag)
+				}
+			}
+		}
+
+		_ = d.Set("tag", flattenTags(toSave))
+	}
+
 	return diags
 }
 
@@ -311,6 +361,34 @@ func resourceFeatureV2Update(ctx context.Context, d *schema.ResourceData, meta i
 		}
 	}
 
+	if d.HasChange("tag") {
+		o, a := d.GetChange("tag")
+		old := o.([]interface{})
+		new := a.([]interface{})
+
+		toAdd := []api.FeatureTag{}
+		toRemove := []api.FeatureTag{}
+
+		for _, newTag := range new {
+			newFeatureTag := toFeatureTag(newTag.(map[string]interface{}))
+			if !isTagIn(newFeatureTag, old) {
+				toAdd = append(toAdd, newFeatureTag)
+			}
+		}
+
+		for _, oldTag := range old {
+			oldFeatureTag := toFeatureTag(oldTag.(map[string]interface{}))
+			if !isTagIn(oldFeatureTag, new) {
+				toRemove = append(toRemove, oldFeatureTag)
+			}
+		}
+
+		_, _, err := client.FeatureTags.UpdateFeatureTags(feature.Name, toAdd, toRemove)
+		if err != nil {
+			return diag.FromErr(err)
+		}
+	}
+
 	if d.HasChange("environment") {
 		o, a := d.GetChange("environment")
 		old := o.([]interface{})
@@ -429,6 +507,16 @@ func isEnvIn(name string, envs []interface{}) bool {
 	return false
 }
 
+func isTagIn(tag api.FeatureTag, tags []interface{}) bool {
+	for _, t := range tags {
+		tfTag := toFeatureTag(t.(map[string]interface{}))
+		if tfTag.Type == tag.Type && tfTag.Value == tag.Value {
+			return true
+		}
+	}
+	return false
+}
+
 func isStratIn(id string, strats []api.FeatureStrategy) bool {
 	for _, strat := range strats {
 		if strat.ID == id {
@@ -529,3 +617,26 @@ func flattenEnvironments(environments []api.Environment) []interface{} {
 
 	return tfEnvironments
 }
+
+func flattenTags(tags []api.FeatureTag) []interface{} {
+	if tags == nil {
+		return []interface{}{}
+	}
+
+	tfTags := []interface{}{}
+	for _, tag := range tags {
+		tfTag := map[string]interface{}{}
+		tfTag["type"] = tag.Type
+		tfTag["value"] = tag.Value
+
+		tfTags = append(tfTags, tfTag)
+	}
+	return tfTags
+}
+
+func toFeatureTag(tfTag map[string]interface{}) api.FeatureTag {
+	tag := api.FeatureTag{}
+	tag.Type = tfTag["type"].(string)
+	tag.Value = tfTag["value"].(string)
+	return tag
+}

internal/provider/resource_feature_v2_test.go

@@ -32,6 +32,8 @@ func TestAccResourceFeatureV2(t *testing.T) {
 					resource.TestCheckResourceAttr("unleash_feature_v2.foo", "environment.1.strategy.1.parameters.rollout", "68"),
 					resource.TestCheckResourceAttr("unleash_feature_v2.foo", "environment.1.strategy.1.parameters.stickiness", "random"),
 					resource.TestCheckResourceAttr("unleash_feature_v2.foo", "environment.1.strategy.1.parameters.groupId", "toggle"),
+					resource.TestCheckResourceAttr("unleash_feature_v2.foo", "tag.0.type", "simple"),
+					resource.TestCheckResourceAttr("unleash_feature_v2.foo", "tag.0.value", "value"),
 				),
 			},
 		},
@@ -70,5 +72,9 @@ resource "unleash_feature_v2" "foo" {
 		}
 	  }
 	}
+	tag {
+		type = "simple"
+		value = "value"
+	}
   }
 `, utils.RandomString(4))