Skip to content
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

Improve heuristic for opening docker-library/official-images PRs #233

Open
cap10morgan opened this issue May 28, 2024 · 1 comment
Open

Comments

@cap10morgan
Copy link
Collaborator

This pull request is a good example of why our current heuristic for whether or not we should automatically open a PR in docker-library/official-images is overly simplistic.

Currently we only open the official-images PR if the generated manifest file changes somewhere other than the git SHA.

However, PR #232 fixes a bug in the lein images but only bumps the git SHA in the manifest. It probably still makes sense to not always open an official-images PR on every change we merge here, though. But maybe just doing that is the simplest and most reliable option?

One option would be to build (some of*) the images in our CI pipeline here and detect when they change and base the decision on that. That would be tricky and slow, but is probably the most correct heuristic.

Another option would be to detect a string in the git commit message that forces an official-images PR (e.g. [force-official-images-pr]). We would just have to remember to add it when we needed it.

* It would likely be slow and unreliable to build every architecture in GitHub Actions' free tier infrastructure.

@cap10morgan
Copy link
Collaborator Author

And, of course, another option would be to do nothing and just continue manually opening official-images PRs in these cases. It's not a huge hassle.

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

No branches or pull requests

1 participant