docs: Update KMS docs #1204
docs: Update KMS docs #1204
Conversation
Vercel Previews Deployed
|
Broken Link CheckerNo broken links found! 🎉 |
| This example shows an IBM KMS stanza in the Boundary configuration file: | ||
|
|
||
| ````hcl | ||
| kms "ibmkp" { |
There was a problem hiding this comment.
@mikemountain @dkanney I'm a little confused about the naming conventions. Was it intentional to go with ibmkp instead of ibmkms? What convention do other kms's go with?
There was a problem hiding this comment.
That's how the package is named/exported in the the wrapper itself
There was a problem hiding this comment.
Ok. To maintain consistency, I think we need to rename the header above from
# ibmkms KMS
-->
# ibmkp KMS
cc: @Dan-Heath
There was a problem hiding this comment.
Yep, that makes sense. Thanks @moduli ! Just updated it.
There was a problem hiding this comment.
Similarly, I don't know if that affects anything in docs-nav-data.json?
There was a problem hiding this comment.
I updated the navigation pane to reflect the proper product names and make all the KMS provider names consistent. Thanks for thinking that through with me!
5 participants
This PR updates the Boundary KMS docs.
View the update in the preview deployment: