Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning related with deprecated argument #38

Closed
intention-attention opened this issue Mar 20, 2023 · 1 comment · Fixed by #44
Closed

Warning related with deprecated argument #38

intention-attention opened this issue Mar 20, 2023 · 1 comment · Fixed by #44

Comments

@intention-attention
Copy link

Describe the Feature

The warning related with account assignments

│ Warning: Argument is deprecated

│ with module.sso_account_assignments.data.aws_identitystore_user.this["[email protected]"],
│ on .terraform/modules/sso_account_assignments/modules/account-assignments/main.tf line 21, in data "aws_identitystore_user" "this":
│ 21: data "aws_identitystore_user" "this" {

│ Use the alternate_identifier attribute instead.

Expected Behavior

No warning displayed

Use Case

When applying account assignments.

Describe Ideal Solution

Update resource to use recommended/updated argument

Alternatives Considered

No response

Additional Context

No response

@jagnk
Copy link

jagnk commented May 26, 2023

Hi I am having the same issue but now it's failing instead of throwing a warning doesn't matter which version of the module I use.

TF version 1.2.2
AWS provider version v5.0.0
module version 1.1.0

╷
│ Error: Unsupported block type
│ 
│   on .terraform/modules/account_assignments/modules/account-assignments/main.tf line 25, in data "aws_identitystore_user" "this":
│   25:   filter {
│ 
│ Blocks of type "filter" are not expected here.

You merged recently a fix for data "aws_identitystore_group" in https://github.com/cloudposse/terraform-aws-sso/pull/35/files . This need to be fixed for data "aws_identitystore_user" too.

Could you fix it please as it is blocking my work.

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 a pull request may close this issue.

2 participants