support deprecating variables

Author: DanielMSchmidt

internal/configs/named_values.go

@@ -48,6 +48,10 @@ type Variable struct {
 	Nullable    bool
 	NullableSet bool
 
+	Deprecated      string
+	DeprecatedSet   bool
+	DeprecatedRange hcl.Range
+
 	DeclRange hcl.Range
 }
 
@@ -186,6 +190,13 @@ func decodeVariableBlock(block *hcl.Block, override bool) (*Variable, hcl.Diagno
 		v.Default = val
 	}
 
+	if attr, exists := content.Attributes["deprecated"]; exists {
+		valDiags := gohcl.DecodeExpression(attr.Expr, nil, &v.Deprecated)
+		diags = append(diags, valDiags...)
+		v.DeprecatedSet = true
+		v.DeprecatedRange = attr.Range
+	}
+
 	for _, block := range content.Blocks {
 		switch block.Type {
 
@@ -509,6 +520,9 @@ var variableBlockSchema = &hcl.BodySchema{
 		{
 			Name: "nullable",
 		},
+		{
+			Name: "deprecated",
+		},
 	},
 	Blocks: []hcl.BlockHeaderSchema{
 		{

internal/configs/named_values_test.go

@@ -54,8 +54,8 @@ func TestOutputDeprecation(t *testing.T) {
 		output "foo" {
 			value = "bar"
 			deprecated = "This output is deprecated"
-		}
-	`
+			}
+		`
 
 	hclF, diags := hclsyntax.ParseConfig([]byte(src), "test.tf", hcl.InitialPos)
 	if diags.HasErrors() {
@@ -66,7 +66,6 @@ func TestOutputDeprecation(t *testing.T) {
 	if diags.HasErrors() {
 		t.Fatalf("unexpected error: %q", diags)
 	}
-
 	if !b.Outputs[0].DeprecatedSet {
 		t.Fatalf("expected output to be deprecated")
 	}
@@ -75,3 +74,30 @@ func TestOutputDeprecation(t *testing.T) {
 		t.Fatalf("expected output to have deprecation message")
 	}
 }
+
+func TestVariableDeprecation(t *testing.T) {
+	src := `
+		variable "foo" {
+			type = string
+			deprecated = "This variable is deprecated, use bar instead"
+		}
+	`
+
+	hclF, diags := hclsyntax.ParseConfig([]byte(src), "test.tf", hcl.InitialPos)
+	if diags.HasErrors() {
+		t.Fatal(diags.Error())
+	}
+
+	b, diags := parseConfigFile(hclF.Body, nil, false, false)
+	if diags.HasErrors() {
+		t.Fatalf("unexpected error: %q", diags)
+	}
+
+	if !b.Variables[0].DeprecatedSet {
+		t.Fatalf("expected variable to be deprecated")
+	}
+
+	if b.Variables[0].Deprecated != "This variable is deprecated, use bar instead" {
+		t.Fatalf("expected variable to have deprecation message")
+	}
+}

internal/terraform/context_plan2_test.go

@@ -7577,3 +7577,147 @@ resource "test_resource" "test" {
 		},
 	))
 }
+
+func TestContext2Plan_deprecated_variable(t *testing.T) {
+	m := testModuleInline(t, map[string]string{
+		"mod/main.tf": `
+variable "old-and-used" {
+	type = string
+	deprecated = "module variable deprecation 1"
+	default = "optional"
+}
+
+variable "old-and-unused" {
+	type = string
+	deprecated = "module variable deprecation 2"
+	default = "optional"
+}
+
+variable "new" {
+    type = string
+    default = "optional"
+}
+
+output "use-everything" {
+    value = {
+      used = var.old-and-used
+      unused = var.old-and-unused
+      new = var.new
+    }
+}
+`,
+		"main.tf": `
+variable "root-old-and-used" {
+	type = string
+	deprecated = "root variable deprecation 1"
+	default = "optional"
+}
+
+variable "root-old-and-unused" {
+	type = string
+	deprecated = "root variable deprecation 2"
+	default = "optional"
+}
+
+variable "new" {
+    type = string
+    default = "new"
+}
+
+module "old-mod" {
+    source = "./mod"
+    old-and-used = "old"
+}
+
+module "new-mod" {
+    source = "./mod"
+    new = "new"
+}
+
+output "use-everything" {
+    value = {
+      used = var.root-old-and-used
+      unused = var.root-old-and-unused
+      new = var.new
+    }
+}
+`,
+	})
+
+	p := new(testing_provider.MockProvider)
+	p.GetProviderSchemaResponse = getProviderSchemaResponseFromProviderSchema(&providerSchema{
+		ResourceTypes: map[string]*configschema.Block{
+			"test_resource": {
+				Attributes: map[string]*configschema.Attribute{
+					"attr": {
+						Type:     cty.String,
+						Computed: true,
+					},
+				},
+			},
+		},
+	})
+
+	ctx := testContext2(t, &ContextOpts{
+		Providers: map[addrs.Provider]providers.Factory{
+			addrs.NewDefaultProvider("test"): testProviderFuncFixed(p),
+		},
+	})
+
+	vars := InputValues{
+		"root-old-and-used": {
+			Value: cty.StringVal("root-old-and-used"),
+		},
+		"root-old-and-unused": {
+			Value: cty.NullVal(cty.String),
+		},
+		"new": {
+			Value: cty.StringVal("new"),
+		},
+	}
+
+	// We can find module variable deprecation during validation
+	diags := ctx.Validate(m, &ValidateOpts{})
+	var expectedValidateDiags tfdiags.Diagnostics
+	expectedValidateDiags = expectedValidateDiags.Append(
+		&hcl.Diagnostic{
+			Severity: hcl.DiagWarning,
+			Summary:  "Deprecated variable got a value",
+			Detail:   "module variable deprecation 1",
+			Subject: &hcl.Range{
+				Filename: filepath.Join(m.Module.SourceDir, "main.tf"),
+				Start:    hcl.Pos{Line: 21, Column: 20, Byte: 350},
+				End:      hcl.Pos{Line: 21, Column: 25, Byte: 355},
+			},
+		},
+	)
+	tfdiags.AssertDiagnosticsMatch(t, diags, expectedValidateDiags)
+
+	_, diags = ctx.Plan(m, states.NewState(), SimplePlanOpts(plans.NormalMode, vars))
+	var expectedPlanDiags tfdiags.Diagnostics
+	expectedPlanDiags = expectedPlanDiags.Append(
+		&hcl.Diagnostic{
+			Severity: hcl.DiagWarning,
+			Summary:  "Deprecated variable got a value",
+			Detail:   "root variable deprecation 1",
+			Subject: &hcl.Range{
+				Filename: filepath.Join(m.Module.SourceDir, "main.tf"),
+				Start:    hcl.Pos{Line: 4, Column: 2, Byte: 48},
+				End:      hcl.Pos{Line: 4, Column: 42, Byte: 90},
+			},
+		},
+	).Append(
+		&hcl.Diagnostic{
+			Severity: hcl.DiagWarning,
+			Summary:  "Deprecated variable got a value",
+			Detail:   "module variable deprecation 1",
+			Subject: &hcl.Range{
+				Filename: filepath.Join(m.Module.SourceDir, "main.tf"),
+				Start:    hcl.Pos{Line: 21, Column: 20, Byte: 350},
+				End:      hcl.Pos{Line: 21, Column: 25, Byte: 355},
+			},
+		},
+	)
+
+	tfdiags.AssertDiagnosticsMatch(t, diags, expectedPlanDiags)
+}

internal/terraform/node_module_variable.go

@@ -307,6 +307,15 @@ func (n *nodeModuleVariable) evalModuleVariable(ctx EvalContext, validateOnly bo
 	finalVal, moreDiags := PrepareFinalInputVariableValue(n.Addr, rawVal, n.Config)
 	diags = diags.Append(moreDiags)
 
+	if n.Config.DeprecatedSet && !givenVal.IsNull() {
+		diags = diags.Append(&hcl.Diagnostic{
+			Severity: hcl.DiagWarning,
+			Summary:  "Deprecated variable got a value",
+			Detail:   n.Config.Deprecated,
+			Subject:  n.Expr.Range().Ptr(),
+		})
+	}
+
 	return finalVal, diags.ErrWithWarnings()
 }
 

internal/terraform/node_root_variable.go

@@ -92,6 +92,15 @@ func (n *NodeRootVariable) Execute(ctx EvalContext, op walkOperation) tfdiags.Di
 		}
 	}
 
+	if n.Config.DeprecatedSet && !givenVal.Value.IsNull() && givenVal.Value.IsKnown() {
+		diags = diags.Append(&hcl.Diagnostic{
+			Severity: hcl.DiagWarning,
+			Summary:  "Deprecated variable got a value",
+			Detail:   n.Config.Deprecated,
+			Subject:  &n.Config.DeprecatedRange,
+		})
+	}
+
 	if n.Planning {
 		if checkState := ctx.Checks(); checkState.ConfigHasChecks(n.Addr.InModule(addrs.RootModule)) {
 			ctx.Checks().ReportCheckableObjects(