Skip to content

feat(dpav-2563): add trivy vulnerability check to management-node pipeline#49

Open
ethiggins wants to merge 25 commits intodevelopfrom
feature/dpav-2563/add-trivy-to-pipeline
Open

feat(dpav-2563): add trivy vulnerability check to management-node pipeline#49
ethiggins wants to merge 25 commits intodevelopfrom
feature/dpav-2563/add-trivy-to-pipeline

Conversation

@ethiggins
Copy link

@ethiggins ethiggins commented Mar 12, 2026

Sensitive Credential Checks

  • As the author of these changes, I have checked for any sensitive credentials prior to this review being requested.
  • As a reviewer of these changes, I have checked for any sensitive credentials prior to approving this merge.

Motivation and Context

Description

  • Added Trivy vulnerability scanner to the automated tests workflow, to run as part of release pipeline.

How Has This Been Tested?

Screenshots (if appropriate):

Checklist:

  • It contains only changes required by issue (does not contain other PR)
  • Includes link to an issue (if apply)
  • I have added tests to cover my changes.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

✅ OSS Checks Passed

All tracked OSS checks passed in this run.

📊 Total Files 🟢 Passed 🔴 Failed 🧮 Score
13 13 0 100%

Results from commit 235be75, view the full job summary↗️ for detailed results.

♻️ This comment has been updated with latest results.

Copy link

@KumailKamranIS KumailKamranIS left a comment

Choose a reason for hiding this comment

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

Overall looks good just request a quick change in the same vein as the dockerfile image update

Choose a reason for hiding this comment

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

can you also make this update in the -dev docker file?

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants