Skip to content

Conversation

@lmilleri
Copy link
Member

This PR updates the KBS configuration for being compatible with the latest all-in-one KBS image.

Please note the authentication key is provided as part of the kbs-config.yaml file, not sure how to consume the secret kbsAuthSecretName: kbs-auth-public-key

@lmilleri
Copy link
Member Author

/cc @bpradipt

"--config",
"--config-file",
"/etc/kbs-config/kbs-config.json",
"--auth-public-key",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So for the latest code, auth-public-key is not required ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah ok, it's part of the config.. Ignore my comment

Copy link
Member

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
@lmilleri can you please check the SoB line. Not sure why DCO error is coming.

Signed-off-by: Leonardo Milleri <[email protected]>
@lmilleri lmilleri force-pushed the lmilleri/update-config branch from 32ab645 to 86c20aa Compare January 30, 2024 15:46
@bpradipt bpradipt merged commit 82fa472 into confidential-containers:main Jan 31, 2024
@lmilleri lmilleri deleted the lmilleri/update-config branch May 15, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants