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

[KMS] The rotation state of key is not disabled, when key in pending delete state. #1821

Closed
anton-sidelnikov opened this issue Jul 19, 2022 · 0 comments
Assignees

Comments

@anton-sidelnikov
Copy link
Member

The rotation state of key is not disabled, when key in pending delete state.


module "kms" {
  source = "../modules/terraform-opentelekomcloud-kms"

  kms_keys = {
    key1 = { alias = "test" }
    key2 = { alias = "test-2", description = "A test key", is_rotation_enabled = true, rotation_interval = 90 }
  }
}

This leads to a follow-up error: {"error":{"error_msg":"The rotation state of key is not disabled.","error_code":"KMS.2901"}}.

@anton-sidelnikov anton-sidelnikov self-assigned this Jul 19, 2022
otc-zuul bot pushed a commit that referenced this issue Jul 19, 2022
[KMS] fix kms deletion with  enabled rotation

Summary of the Pull Request
When reenable key, which was created with rotation: {"error":{"error_msg":"The rotation state of key is not disabled.","error_code":"KMS.2901"}}.
PR Checklist

 Refers to: #1804 #1821
 Tests added/passed.
 Documentation updated.
 Schema updated.
 Release notes added.

Acceptance Steps Performed
=== RUN   TestAccKmsKeyV1_basic
--- PASS: TestAccKmsKeyV1_basic (83.19s)
=== RUN   TestAccKmsKey_isEnabled
--- PASS: TestAccKmsKey_isEnabled (116.61s)
=== RUN   TestAccKmsKey_rotation
--- PASS: TestAccKmsKey_rotation (51.10s)
=== RUN   TestAccKmsKey_cancelDeletion
--- PASS: TestAccKmsKey_cancelDeletion (50.83s)
=== RUN   TestAccKmsKey_cancelDeletionWithRotation
--- PASS: TestAccKmsKey_cancelDeletionWithRotation (52.38s)
PASS


Process finished with the exit code 0

Reviewed-by: Vladimir Vshivkov <None>
Reviewed-by: Artem Lifshits <None>
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

No branches or pull requests

1 participant