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: Set TRIVY_USERNAME and TRIVY_PASSWORD environment variables to consume secret keys #141

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

danielpacak
Copy link
Contributor

@danielpacak danielpacak commented Sep 2, 2020

So far we were passing registry credentials as values. In this PR we change it to use secretKeyRefs:

Starboard HLD - Find vulnerabilities with private image

Signed-off-by: Daniel Pacak [email protected]

@danielpacak danielpacak force-pushed the fix/pass_registry_creds_as_secret_refs branch from f463fb7 to 9eb74a5 Compare September 2, 2020 09:10
@codecov
Copy link

codecov bot commented Sep 2, 2020

Codecov Report

Merging #141 into master will decrease coverage by 3.32%.
The diff coverage is 15.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #141      +/-   ##
==========================================
- Coverage   43.19%   39.86%   -3.33%     
==========================================
  Files          33       33              
  Lines        1623     1801     +178     
==========================================
+ Hits          701      718      +17     
- Misses        846     1003     +157     
- Partials       76       80       +4     
Impacted Files Coverage Δ
pkg/find/vulnerabilities/trivy/scanner.go 0.00% <0.00%> (ø)
pkg/kube/pod/manager.go 0.00% <0.00%> (ø)
pkg/kube/secrets/secrets.go 77.27% <68.00%> (-12.21%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b5cc5a...1007f24. Read the comment docs.

@danielpacak danielpacak force-pushed the fix/pass_registry_creds_as_secret_refs branch from 9eb74a5 to 4691680 Compare September 2, 2020 10:23
@danielpacak danielpacak force-pushed the fix/pass_registry_creds_as_secret_refs branch from 4691680 to 1007f24 Compare September 2, 2020 10:44
@danielpacak danielpacak requested a review from lizrice September 2, 2020 11:02
@danielpacak danielpacak added this to the Release v0.3.0 milestone Sep 2, 2020
Copy link
Contributor

@lizrice lizrice left a comment

Choose a reason for hiding this comment

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

lgtm

@danielpacak danielpacak merged commit e5963e5 into master Sep 3, 2020
@danielpacak danielpacak deleted the fix/pass_registry_creds_as_secret_refs branch September 3, 2020 15:51
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