-
Notifications
You must be signed in to change notification settings - Fork 57
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
Add TrustedRootJSON to TrustRootSpec #1705
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Cody Soyland <[email protected]>
Signed-off-by: Cody Soyland <[email protected]>
@malancas @hectorj2f @vaikas what do you think of this one? |
Regarding the format, I think you can create the json as a string as a user and not worry about the indentation , so not sure if it's worth doing the roundtrip to/from base64, but I don't really feel strongly about that. The validation here should catch any indentation errors: Can you add a test for this, something like: Another way to do this might be to hoist the api definition to v1beta1 for trustroots, and at that point change the previous field(s) to not support the old version, and only support the TrustedRootJSON. As in, the trust root is still in v1alpha1, and I'm curious if that should be hoisted to v1beta1, and get a deprecation plan for getting rid of v1alpha1 (obviously not part of this PR, but just thinking out loud). |
👍🏻
No strong feelings here... perhaps the deprecation plan would be to support both in v1alpha1 and only trustroot json in v1beta1? I'm okay with either option! |
Yeah, since the old 'trustroot' was before the proto came to be, I don't think carrying that forward is a good move, and overall deprecating the v1alpha1 is probably the right thing to do sooner rather than later so folks have time to move off of it to v1beta1: So, I'd vote for trustrootjson in v1beta1, deprecate the whole v1alpha1 (all the crds, not just the trustroot). |
This adds
TrustedRootJSON
to theTrustRootSpec
to allow using of theTrustedRootJSON
spec in air-gapped environments.usage:
Note this uses type
string
. We could alternatively use type[]byte
for this, which would mean that thetrustedRootJSON
value would need to be base64-encoded. That would be perhaps less prone to human-error in indentation. I'm happy to make that change if deemed better by the maintainers.Fixes #1704
Summary
Release Note
Documentation