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

bootstrap: cdk v2 bootstrap cli created AWS managed key instead of Customer-managed key #21992

Closed
yayfeng opened this issue Sep 10, 2022 · 5 comments
Assignees
Labels
bug This issue is a bug. documentation This is a problem with documentation. effort/small Small work item – less than a day of effort p1 package/tools Related to AWS CDK Tools or CLI

Comments

@yayfeng
Copy link

yayfeng commented Sep 10, 2022

Describe the bug

cdk bootstrap aws://xxx/xxx created AWS managed key.
image

According to documentation it should be Customer-managed key?

image

cdk --version
2.41.0 (build 6ad48a3)

Expected Behavior

either documentation or cdk cli should be updated to match each other.

Current Behavior

wrong kms key was used

Reproduction Steps

cdk bootstrap aws://xxx/xxx

Possible Solution

No response

Additional Information/Context

No response

CDK CLI Version

2.41.0 (build 6ad48a3)

Framework Version

No response

Node.js Version

v14.18.0

OS

Mac OS

Language

Typescript

Language Version

No response

Other information

No response

@yayfeng yayfeng added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Sep 10, 2022
@github-actions github-actions bot added the package/tools Related to AWS CDK Tools or CLI label Sep 10, 2022
@peterwoodworth
Copy link
Contributor

thanks for reporting this @yayfeng,

I believe this is an issue with our docs. creating a CMK incurs a cost, so we default to the option that incurs less cost.

@jerry-aws could you look into fixing this in the devguide? Thanks 🙂

@peterwoodworth peterwoodworth added p1 effort/small Small work item – less than a day of effort documentation This is a problem with documentation. and removed needs-triage This issue or PR still needs to be triaged. bug This issue is a bug. labels Sep 14, 2022
@peterwoodworth peterwoodworth assigned ghost and unassigned rix0rrr Sep 14, 2022
@lestephane
Copy link

How do I bootstrap without the cost-incurring KMS key? And how do I apply the change to already bootstrapped environments that are using a cost-incurring KMS key?

@tim-finnigan
Copy link

Submitted a ticket for this via the Provide feedback link from the documentation page. To my knowledge this is most direct approach for escalating documentation issues in the Developer Guide. And for general questions we recommend using Discussions.

@peterwoodworth peterwoodworth added the bug This issue is a bug. label May 2, 2023
@vinayak-kukreja vinayak-kukreja self-assigned this May 7, 2024
@comcalvi comcalvi assigned scanlonp and unassigned vinayak-kukreja May 28, 2024
@scanlonp
Copy link
Contributor

Docs are updated!

Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. documentation This is a problem with documentation. effort/small Small work item – less than a day of effort p1 package/tools Related to AWS CDK Tools or CLI
Projects
None yet
Development

No branches or pull requests

7 participants