Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

chore: Update snyk monitor scan to use Grafana's GH action#5561

Merged
musculman merged 3 commits intodevfrom
ijulve/snyk-changes
Jul 1, 2025
Merged

chore: Update snyk monitor scan to use Grafana's GH action#5561
musculman merged 3 commits intodevfrom
ijulve/snyk-changes

Conversation

@musculman
Copy link
Copy Markdown
Contributor

What this PR does

Changes the Snyk Monitoring to use Grafana's Action
The current setup prevents Snyk from running on forks and blocks contributions because of the id-token: write permission

@musculman musculman requested a review from a team as a code owner July 1, 2025 07:11
@musculman musculman added the pr:no public docs Added to a PR that does not require public documentation updates label Jul 1, 2025
@musculman musculman requested review from matiasb and paulcoghlan July 1, 2025 07:11
@musculman musculman added the release:patch PR will be added to "Other Changes" section of release notes label Jul 1, 2025
Copy link
Copy Markdown
Contributor

@paulcoghlan paulcoghlan left a comment

Choose a reason for hiding this comment

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

LGTM 😄

# will need to refactor .github/workflows/build-engine-docker-image-and-publish-to-dockerhub.yml
# to be a composable action instead of a workflow
snyk-scan-ci:
uses: 'grafana/security-github-actions/.github/workflows/snyk_monitor.yml@main'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we're only pinning versions for external GitHub actions, so this LGTM.

@musculman musculman added this pull request to the merge queue Jul 1, 2025
Merged via the queue into dev with commit eddb861 Jul 1, 2025
29 checks passed
@musculman musculman deleted the ijulve/snyk-changes branch July 1, 2025 13:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

pr:no public docs Added to a PR that does not require public documentation updates release:patch PR will be added to "Other Changes" section of release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants