-
Notifications
You must be signed in to change notification settings - Fork 4
fix: strip version from kmodule image digest #271
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
Conversation
ocofaigh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you are sure that this does not support passing an image tag (which is not the case for any of the other images used here) we should add variable validation to the input to only allow the digest format.
|
@ocofaigh docs suggest they're both enabled by default We are sunsetting application checks in favor of [Monitoring Integrations](https://docs.sysdig.com/en/integration-library/). |
ocofaigh
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see comments
|
/run pipeline |
1 similar comment
|
/run pipeline |
|
/run pipeline |
|
🎉 This PR is included in version 1.19.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |

Description
Sysdig charts don't support passing an image tag to the digest field as we get:
instead of:
We need to maintain the version
14.2.5for renovate automation, so instead we created a local that strips at the@symbol from14.2.5@sha256:0345968a77f8eb64a00f2b06af3c40bf3df78d34125b46c9788a9e73f8ddbb1aso we end up only passingsha256:0345968a77f8eb64a00f2b06af3c40bf3df78d34125b46c9788a9e73f8ddbb1a.Release required?
x.x.X)x.X.x)X.x.x)Release notes content
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
Checklist for reviewers
For mergers