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

[Enhancement]: Add RotateImmediately to aws_secretsmanager_secret_rotation #34961

Closed
sakojun opened this issue Dec 18, 2023 · 5 comments · Fixed by #35105 or #35174
Closed

[Enhancement]: Add RotateImmediately to aws_secretsmanager_secret_rotation #34961

sakojun opened this issue Dec 18, 2023 · 5 comments · Fixed by #35105 or #35174
Labels
enhancement Requests to existing resources that expand the functionality or scope. good first issue Call to action for new contributors looking for a place to start. Smaller or straightforward issues. service/secretsmanager Issues and PRs that pertain to the secretsmanager service.
Milestone

Comments

@sakojun
Copy link

sakojun commented Dec 18, 2023

Description

aws_secretsmanager_secret_rotation must accept the RotateImmediately parameter.
If this parameter is not specified, immediate rotation will be performed.
The SDK supports this parameter, but Terraform does not.

Affected Resource(s) and/or Data Source(s)

aws_secretsmanager_secret_rotation

Potential Terraform Configuration

resource "aws_secretsmanager_secret_rotation" "example" {
  secret_id           = aws_secretsmanager_secret.example.id
  rotation_lambda_arn = aws_lambda_function.example.arn

  rotate_immediately = false

  rotation_rules {
    automatically_after_days = 30
  }
}

References

https://docs.aws.amazon.com/secretsmanager/latest/apireference/API_RotateSecret.html
https://pkg.go.dev/github.com/aws/aws-sdk-go-v2/service/secretsmanager#Client.RotateSecret
https://pkg.go.dev/github.com/aws/aws-sdk-go-v2/service/secretsmanager#RotateSecretInput

Would you like to implement a fix?

None

@sakojun sakojun added the enhancement Requests to existing resources that expand the functionality or scope. label Dec 18, 2023
Copy link

Community Note

Voting for Prioritization

  • Please vote on this issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize this request.
  • Please see our prioritization guide for information on how we prioritize.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request.

Volunteering to Work on This Issue

  • If you are interested in working on this issue, please leave a comment.
  • If this would be your first contribution, please review the contribution guide.

@github-actions github-actions bot added the service/secretsmanager Issues and PRs that pertain to the secretsmanager service. label Dec 18, 2023
@terraform-aws-provider terraform-aws-provider bot added the needs-triage Waiting for first response or review from a maintainer. label Dec 18, 2023
@galiceau
Copy link

galiceau commented Dec 18, 2023

Hello

I add my issue because I think that it could be the same topic : disable the password rotation (in AWS secret ) of RDS in terraform. It is only possible from console or API.
In this answer it was precised that "As noted in the aws_secretsmanager_secret resource rotation example, adding rotation_lambda_arn will enable rotation. Removing that argument will disable rotation."
=> It doesn't work.

module "difa-rds" {
  source  = "terraform-aws-modules/rds/aws"
  version = "6.1.0"

}

resource "aws_secretsmanager_secret_rotation" "db_instance_master_user_secret_rotation" {
  secret_id               = module.difa-rds.db_instance_master_user_secret_arn
  rotation_rules {
    automatically_after_days = 30
  } 
}```


regards

@justinretzolk justinretzolk added good first issue Call to action for new contributors looking for a place to start. Smaller or straightforward issues. and removed needs-triage Waiting for first response or review from a maintainer. labels Dec 18, 2023
@cbhoffman
Copy link

I wouldn't mind having a look at this and see if I can get something in to fix.

Copy link

This functionality has been released in v5.33.0 of the Terraform AWS Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

Copy link

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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope. good first issue Call to action for new contributors looking for a place to start. Smaller or straightforward issues. service/secretsmanager Issues and PRs that pertain to the secretsmanager service.
Projects
None yet
4 participants