8000 feat: add trivy by dv-hossam-radwan · Pull Request #164 · datavisyn/github-workflows · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: add trivy #164

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

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

feat: add trivy #164

wants to merge 33 commits into from

Conversation

dv-hossam-radwan
Copy link
Contributor

No description provided.

@dv-hossam-radwan dv-hossam-radwan marked this pull request as draft April 15, 2025 11:54
@dv-hossam-radwan dv-hossam-radwan marked this pull request as ready for review April 16, 2025 09:32
@puehringer puehringer changed the title Feat/add trivy feat: add trivy Apr 22, 2025
@dv-hossam-radwan
Copy link
Contributor Author

PR ready for review @puehringer @dvvanessastoiber @dvviktordelev

@puehringer
Copy link
Contributor

@dv-hossam-radwan did a test run with this PR and visynKB in case you want to take a look: https://github.com/datavisyn/visyn_kb/actions/runs/15003505026

8000

exit-code: '1'
ignore-unfixed: false
vuln-type: 'os,library'
severity: 'MEDIUM,HIGH,CRITICAL'
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure if 57A7 we should treat medium ones...

Suggested change
severity: 'MEDIUM,HIGH,CRITICAL'
severity: 'HIGH,CRITICAL'

Copy link
Contributor

Choose a reason for hiding this comment

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

High and critical ones should be sufficient as we did the same in the old pipeline 👍 interestingly, GSK only checks for critical vulnerabilites, but I would like to keep the high ones as well 😅

image

Copy link
Contributor
@dvvanessastoiber dvvanessastoiber left a comment

Choose a reason for hiding this comment

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

Looks good to me code-wise 👍 let's also wait for approval by @puehringer and @dvviktordelev

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.

3 participants
0