-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Can the Regex for importing aws_appautoscaling_policy be relaxed to allow /
symbols.
#8306
Comments
I'm facing exactly the same issue. According to the documentation, the name of the policy of a DynamoDB table will always be in a format similar to: I found the code is treating dynamoDB target resource id as ECS/RDS (with no slashes (/) in the policy name), and it interpolates in a wrong way. This might work for ECS and RDS scaling, but it fails for DynamoDB.
I sent pull request #8397 that fixes this issue. Let's see if it's accepted :-) |
The fix to the |
The fix to the For further feature requests or bug reports with this resource, please create a new GitHub issue following the template for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Terraform Version
Affected Resource(s)
Panic Output
Expected Behavior
Actual Behavior
the required id
<service-namespace>/<resource-id>/<scalable-dimension>/<policy-name>
doesn't validate correctly ifpolicy-name
also includes/
Steps to Reproduce
terraform import module.data_global.module.sisyphus_autoscaling_usw2.aws_appautoscaling_policy.dynamodb_table_read_policy dynamodb/table/test/dynamodb:table:ReadCapacityUnits/DynamoDBReadCapacityUtilization:table/test
The text was updated successfully, but these errors were encountered: