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

Switch tests from heroku/buildpacks:20 to heroku/builder:20 #38

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

edmorley
Copy link
Member

Since the heroku/buildpacks:20 image is about to be deprecated, and has been replaced by the newly added heroku/builder:20 for those that need to use a Heroku-20 based builder.

The build and run images used by heroku/builder:20 are the same as those used by heroku/buildpacks:20 - the only differences between the builders are the included buildpacks (which is somewhat irrelevant for these integration tests, since the buildpack under test will be injected instead).

The smoke tests in the heroku/builder GitHub repo are still testing against all image variants (until such time as we make the legacy images error with an EOL message), so we still have test coverage against them.

See:
heroku/cnb-builder-images#394
https://salesforce.quip.com/0JtbAYiWZYk6

GUS-W-14186015.

Since the `heroku/buildpacks:20` image is about to be deprecated,
and has been replaced by the newly added `heroku/builder:20` for
those that need to use a Heroku-20 based builder.

The build and run images used by `heroku/builder:20` are the same as
those used by `heroku/buildpacks:20` - the only differences between the
builders are the included buildpacks (which is somewhat irrelevant for
these integration tests, since the buildpack under test will be injected instead).

The smoke tests in the `heroku/builder` GitHub repo are still testing
against all image variants (until such time as we make the legacy images
error with an EOL message), so we still have test coverage against them.

See:
heroku/cnb-builder-images#394
https://salesforce.quip.com/0JtbAYiWZYk6

GUS-W-14186015.
@edmorley edmorley self-assigned this Sep 26, 2023
@edmorley edmorley marked this pull request as ready for review September 26, 2023 18:41
@edmorley edmorley requested a review from a team as a code owner September 26, 2023 18:41
@edmorley edmorley enabled auto-merge (squash) September 26, 2023 18:42
@edmorley edmorley merged commit 862570f into main Sep 27, 2023
6 checks passed
@edmorley edmorley deleted the edmorley/builder-20 branch September 27, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants