Skip to content

fix: allow auth_token_update_strategy=DELETE to migrate AUTH to RBAC/ACL#274

Merged
yangci (oycyc) merged 1 commit into
cloudposse:mainfrom
LuisMiranda10:fix/auth-token-delete-strategy-rbac-migration
Jun 14, 2026
Merged

fix: allow auth_token_update_strategy=DELETE to migrate AUTH to RBAC/ACL#274
yangci (oycyc) merged 1 commit into
cloudposse:mainfrom
LuisMiranda10:fix/auth-token-delete-strategy-rbac-migration

Conversation

@LuisMiranda10

Copy link
Copy Markdown
Contributor

what

  • Updated the ElastiCache replication group resource to allow auth_token_update_strategy = "DELETE" when auth_token is set to null.
  • Fixed the migration path from AUTH token authentication to RBAC/ACL (user_group_ids) by correctly forwarding the DELETE strategy to AWS.
  • Preserved existing behavior for resources using SET or ROTATE, as well as resources without an authentication token.

why

  • During migrations from AUTH to RBAC/ACL, AWS requires auth_token_update_strategy = "DELETE" to remove an existing authentication token.
  • The module previously set auth_token_update_strategy to null whenever auth_token was null, preventing the provider from sending the required DELETE operation.
  • As a result, the authentication token was never removed and Terraform applies failed, blocking the migration to RBAC/ACL.
  • This change ensures that the DELETE strategy is passed through when explicitly requested, allowing successful token removal and completion of the migration process.
  • The change is backward compatible and does not alter behavior for existing configurations using the default strategy or an active auth_token.

references

  • Related to ElastiCache AUTH → RBAC/ACL migration support.
  • Terraform validation (terraform validate) and formatting (terraform fmt) completed successfully.

@oycyc

Copy link
Copy Markdown
Contributor

/terratest

@oycyc yangci (oycyc) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

@mergify mergify Bot removed the triage Needs triage label Jun 14, 2026
@oycyc yangci (oycyc) merged commit c8d9b38 into cloudposse:main Jun 14, 2026
40 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

These changes were released in v2.1.0.

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.

3 participants