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

Stop publishing non-functions images to the internal registry #428

Merged
merged 1 commit into from
Oct 27, 2023

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Oct 27, 2023

The heroku/buildpacks:20 and heroku/builder:22 builders are now only published to Docker Hub, rather than also being published to the internal registry (where they were tagged as heroku-20:builder and heroku-22:builder respectively).

Kodon has not defaulted to these non-functions images from the internal registry for some time, and as of heroku/kodon#410 these images (and the internal registry) are no longer on the builder allow list, so it's not possible to manually select them using project.toml.

This change does not affect the functions-only image, which continues to be published to (and consumed by the functions-specific Kodon instance from) the internal registry.

GUS-W-14378185.

The `heroku/buildpacks:20` and `heroku/builder:22` builders are now
only published to Docker Hub, rather than also being published to the
internal registry (under the `heroku-20:builder` and `heroku-22:builder`
tags respectively).

Kodon has not defaulted to these non-functions images from the internal registry
for some time, and as of heroku/kodon#410 these images (and the internal
registry) are no longer on the builder allow list, so it's not possible to manually
select them using `project.toml`.

This change does not affect the functions-only image, which continues to be
published to (and consumed by Kodon from) the internal registry.
@edmorley edmorley self-assigned this Oct 27, 2023
@edmorley edmorley marked this pull request as ready for review October 27, 2023 10:40
@edmorley edmorley requested a review from a team as a code owner October 27, 2023 10:40
@edmorley edmorley enabled auto-merge (squash) October 27, 2023 10:40
@edmorley edmorley disabled auto-merge October 27, 2023 10:41
@edmorley edmorley enabled auto-merge (squash) October 27, 2023 10:42
@edmorley edmorley merged commit a968eb3 into main Oct 27, 2023
42 checks passed
@edmorley edmorley deleted the edmorley/publish-less branch October 27, 2023 11:21
@edmorley
Copy link
Member Author

edmorley commented Oct 27, 2023

I've also removed the old images, so people don't accidentally start using them in the future for new use-cases not realising they are no longer updated:
https://github.com/heroku/cnb-builder-images/actions/runs/6667064995/job/18119816685#step:4:33

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

Successfully merging this pull request may close these issues.

2 participants