-
Notifications
You must be signed in to change notification settings - Fork 54
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: rename kms to keyservice #943
Conversation
✅ Deploy Preview for constellation-docs canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me overall.
There is currently some inconsistency with the capitalization of KeyService
.
IMO, it should be either written as KeyService
, keyservice
, depending on the circumstances (Also see JoinService
, and joinservice
). Currently there are cases where its also written as Keyservice
.
CSI charts should also not be changed in this repo, as this wil conflict with the generate workflow, but needs to be updated upstream.
IMO, this has no high priority as the naming there is not user facing, but I will try to get it in with the next release.
bd535e3
to
52607c5
Compare
Regarding CSI charts: We have to add the changes in the constellation repo once they happened upstream, don't we? |
Just created |
003593f
to
1cf15a4
Compare
In the light of extending our eKMS support it will be helpful to have a tighter use of the word "KMS". KMS should refer to the actual component that manages keys. The keyservice, also called KMS in the constellation code, does not manage keys itself. It talks to a KMS backend, which in turn does the actual key management.
1cf15a4
to
6745cea
Compare
Proposed change(s)
Additional info
KMS should refer to the actual component that manages keys. The keyservice, also called KMS in the constellation code, does not manage keys itself. It talks to a KMS backend, which in turn does the actual key management.
Checklist