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

feat(trivy): allow to give trivy github token via secret #1290

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sylvainOL
Copy link
Contributor

Instead of setting the trivy github token via a parameter in the helm chart,
provide the ability to give it via an existing secret.

Instead of setting the trivy github token via a parameter in the helm chart,
provide the ability to give it via an existing secret.

Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com>
@sylvainOL
Copy link
Contributor Author

@zyyw , @MinerYang can you take a look?

@rizzza
Copy link

rizzza commented Jan 6, 2023

@zyyw , @MinerYang can you take a look?

would love to have this. When can we expect some movement?

@zyyw
Copy link
Collaborator

zyyw commented Jan 12, 2023

closing it and then reopen it to trigger ci

@zyyw zyyw closed this Jan 12, 2023
@zyyw zyyw reopened this Jan 12, 2023
@@ -1,4 +1,4 @@
# Helm Chart for Harbor
****# Helm Chart for Harbor
Copy link
Collaborator

Choose a reason for hiding this comment

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

@sylvainOL , could you please justify why we need change of this line? Could you please remove the change regarding this line, thanks

Copy link

github-actions bot commented Feb 8, 2024

This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days.

@github-actions github-actions bot added the Stale label Feb 8, 2024
@Vad1mo Vad1mo removed the Stale label Feb 14, 2024
Copy link

This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days.

@github-actions github-actions bot added the Stale label Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants