Skip to content

Conversation

@fpletz
Copy link
Contributor

@fpletz fpletz commented Oct 30, 2020

Terraform 0.13 will throw the following warning:

Warning: Interpolation-only expressions are deprecated
on .terraform/modules/dbpool.deploy_nixos/deploy_nixos/main.tf line
104, in data "external" "nixos-instantiate":
104: "--argstr", "system", "${var.target_system}"
Terraform 0.11 and earlier required all non-constant expressions to be
provided via interpolation syntax, but this pattern is now deprecated. To
silence this warning, remove the "${ sequence from the start and the }"
sequence from the end of this expression, leaving just the inner expression.
Template interpolation syntax is still used to construct strings from
expressions when the template includes multiple interpolation sequences or a
mixture of literal strings and interpolations. This deprecation applies only
to templates that consist entirely of a single interpolation sequence.
(and 3 more similar warnings elsewhere)

Since this syntax is already supported in 0.12 and everybody should have
upgraded to that till now, this change should be safe.

Terraform 0.13 will throw the following warning:

> Warning: Interpolation-only expressions are deprecated
>   on .terraform/modules/dbpool.deploy_nixos/deploy_nixos/main.tf line
>   104, in data "external" "nixos-instantiate":
>    104:     "--argstr", "system", "${var.target_system}"
>    Terraform 0.11 and earlier required all non-constant expressions to be
>    provided via interpolation syntax, but this pattern is now deprecated. To
>    silence this warning, remove the "${ sequence from the start and the }"
> sequence from the end of this expression, leaving just the inner expression.
> Template interpolation syntax is still used to construct strings from
> expressions when the template includes multiple interpolation sequences or a
> mixture of literal strings and interpolations. This deprecation applies only
> to templates that consist entirely of a single interpolation sequence.
> (and 3 more similar warnings elsewhere)

Since this syntax is already supported in 0.12 and everybody should have
upgraded to that till now, this change should be safe.
@fpletz fpletz force-pushed the fix-tf-0.13-syntax branch from 143ed04 to 8c8bef9 Compare October 30, 2020 12:53
@zimbatm zimbatm merged commit d055d21 into nix-community:master Nov 6, 2020
@zimbatm
Copy link
Member

zimbatm commented Nov 6, 2020

thanks Franz! :)

@fpletz fpletz deleted the fix-tf-0.13-syntax branch November 6, 2020 14:49
@fpletz fpletz restored the fix-tf-0.13-syntax branch November 6, 2020 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants