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

Fix the admission controller README to reflect the updated chart and its credential handling #74

Open
zhill opened this issue Oct 16, 2020 · 0 comments

Comments

@zhill
Copy link
Member

zhill commented Oct 16, 2020

The updated chart will create a credentials secret by default with empty content unless existingCredentialsSecret is used, but the README references the older key, so its easy to create an external secret but not have it actually mounted to the controller and thus all the policy reference lookups fail.

One way to confirm this condition is to:

kubectl exec -t <controller pod> /bin/cat /credentials/credentials.json

If that returns {} then you're probably using the old method and it isn't in the pod properly.

Btodhunter pushed a commit that referenced this issue May 11, 2023
* feat: updates for vunnel integration

Signed-off-by: Weston Steimel <[email protected]>

* chore: bump chart version

Signed-off-by: Weston Steimel <[email protected]>

* feat: allow for config of nvd api key

Signed-off-by: Weston Steimel <[email protected]>

---------

Signed-off-by: Weston Steimel <[email protected]>
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

No branches or pull requests

1 participant