-
Notifications
You must be signed in to change notification settings - Fork 3
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
feature request(ci): Upload built images as artifacts in PRs #5
Comments
I think you're suggesting that PRs have an automated build and push to a public image registry, which can be reachable by any reviewer. It's a good idea but with large scale open source repos, that image registry could swell substantially and someone would need to manage that container registry. Some things to consider about:
I do something in my organization's fork which allows us to include linting and CVE scanning using trivy, as well as making the image available. |
Hey @Souheil-Yazji, what I 'm suggesting is not to publishd to a public registry but rather push them as artifacts in the CI's run. This way, they are also scoped in the PR rather than a public registry, which could also imply that they are published for use there. An example can be seen in this PR's runs https://github.com/canonical/pipelines-rocks/actions/runs/11816173378 where |
ah I understand, thanks for that example! This is a much better suggestion, and there doesn't seem to be immediate challenges. +1 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
On pull requests, workflows should upload the built images as artifacts. This way,
This is not a dashboard-specific thing, but I didn't know where to open this.
The text was updated successfully, but these errors were encountered: