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.

Sign up for GitHub

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

Jump to bottom

CCE Node Data Encryption for Node Pools #1157

Closed
iitsDelbruegger opened this issue Jun 24, 2021 · 0 comments · Fixed by #1165
Closed

CCE Node Data Encryption for Node Pools #1157

iitsDelbruegger opened this issue Jun 24, 2021 · 0 comments · Fixed by #1165
Assignees
@lego963
Labels
enhancement
Milestone
v1.24.3

Comments

@iitsDelbruegger
Copy link

iitsDelbruegger commented Jun 24, 2021 •
edited
Loading

Since 1.24.1, it is possible to encrypt the data volumes of CCE nodes with a KMS key: https://registry.terraform.io/providers/opentelekomcloud/opentelekomcloud/latest/docs/resources/cce_node_v3.
However, I am not able to specify such a key in a node pool. See https://registry.terraform.io/providers/opentelekomcloud/opentelekomcloud/latest/docs/resources/cce_node_pool_v3
@iitsDelbruegger iitsDelbruegger changed the title CCE Node Data Encryption and Node Pools CCE Node Data Encryption for Node Pools Jun 24, 2021
@lego963 lego963 self-assigned this Jun 24, 2021
@lego963 lego963 mentioned this issue Jun 25, 2021
Merged
4 tasks
@lego963 lego963 added this to the v1.24.3 milestone Jun 25, 2021
@otc-zuul otc-zuul bot closed this as completed in #1165 Jun 25, 2021
otc-zuul bot pushed a commit that referenced this issue Jun 25, 2021
@lego963
[CCE] Add missing KMS config (#1165)
9edb040 
[CCE] Add missing `KMS` config

Summary of the Pull Request
Add missing kms config in resource/opentelekomcloud_cce_node_pool_v3
Fixes: #1161
Fixes: #1157
PR Checklist

 Refers to: #1161, #1157
 Tests added/passed.
 Documentation updated.
 Schema updated.

Acceptance Steps Performed
=== RUN   TestAccCCENodePoolsV3_basic
--- PASS: TestAccCCENodePoolsV3_basic (1017.89s)
=== RUN   TestAccCCENodePoolsV3_randomAZ
--- PASS: TestAccCCENodePoolsV3_randomAZ (793.57s)
=== RUN   TestAccCCENodePoolsV3EncryptedVolume
--- PASS: TestAccCCENodePoolsV3EncryptedVolume (720.49s)
PASS

Process finished with the exit code 0

Reviewed-by: Anton Sidelnikov <None>
Reviewed-by: Anton Kachurin <[email protected]>
Reviewed-by: None <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lego963 lego963

Labels
enhancement
Projects
None yet
Milestone
v1.24.3
Development

Successfully merging a pull request may close this issue.

[CCE] Add missing KMS config opentelekomcloud/terraform-provider-opentelekomcloud
2 participants
@lego963 @iitsDelbruegger