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

Use retagged trivy-db image #176

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Use retagged trivy-db image #176

merged 1 commit into from
Nov 5, 2024

Conversation

fhielpos
Copy link
Member

@fhielpos fhielpos requested a review from a team as a code owner October 31, 2024 15:19
@fhielpos fhielpos self-assigned this Oct 31, 2024
@@ -37,3 +37,6 @@ trivy:
privileged: false
readOnlyRootFilesystem: true
allowPrivilegeEscalation: false

trivy:
dbRepository: gsoci.azurecr.io/giantswarm/trivy-db
Copy link
Contributor

Choose a reason for hiding this comment

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

Same Aliyun question here

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes but here we don't have it split in registry and repository. Will check what we can do

Copy link
Member Author

Choose a reason for hiding this comment

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

Unfortunately here there is now way for us to split this template into two, so we need to overwrite the value directly at the catalog level

Copy link
Contributor

@stone-z stone-z left a comment

Choose a reason for hiding this comment

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

Scans are already not pulling from aliyun, so this at least isn't a regression. I'm fine to do it at the catalog level if you want, or let eventual zot be the solution here.

@fhielpos fhielpos merged commit 286ffab into main Nov 5, 2024
3 of 5 checks passed
@fhielpos fhielpos deleted the retag-dbs branch November 5, 2024 15:42
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