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

Remove heroku/buildpacks:18 from trusted/suggested builders lists #1350

Merged
merged 1 commit into from
Jan 3, 2022

Conversation

edmorley
Copy link
Contributor

@edmorley edmorley commented Dec 21, 2021

Summary

The heroku/buildpacks:18 builder is based upon Heroku-18 / Ubuntu 18.04, which are due to enter their deprecation cycle at Heroku in the coming April/May.

In addition, we currently only run heroku/buildpacks:20 on the Heroku platform, so whilst this builder does have some rudimentary CI coverage in our builder generation repository (https://github.com/heroku/pack-images), the older stack is unused and barely tested in practice. We've recently found a few internal developers who were inadvertently still using it, since they'd set it as the default builder some time ago and forgotten in the interim.

Lastly, we expected to release a new heroku/buildpacks:22 builder based on Ubuntu-22 next year, at which point we'd have three builders in the suggested builders list, which seems excessive :-)

As such, the older stack has been removed from the trusted and suggested builders list. This is a breaking change for anyone using that image, since they will need to explicitly add it to their trusted builders list (so this will need to be in a new major Pack CLI release). However we expect this to affect very few users in practice.

Output

See diff in output excerpt in tests.

Documentation

  • Should this change be documented?
    • Yes, see #___
    • No

@edmorley edmorley requested a review from a team as a code owner December 21, 2021 17:28
@github-actions github-actions bot added this to the 0.23.0 milestone Dec 21, 2021
@github-actions github-actions bot added type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement. labels Dec 21, 2021
@edmorley
Copy link
Contributor Author

cc @hone @Malax

The `heroku/buildpacks:18` builder is based upon Heroku-18 / Ubuntu 18.04,
which are due to enter their deprecation cycle In the coming April/May.

In addition, we currently only run `heroku/buildpacks:20` on the Heroku platform,
so whilst this builder does have some rudimentary CI coverage in our builder generation
repository (`heroku/pack-images`), the older stack is unused and barely tested in
practice. We've recently found a few developers who were inadvertently still using
it, since they'd set it as the default builder set some time ago.

Lastly, we expected to release a new `heroku/buildpacks:22`builder based on Ubuntu-22
next year, at which point we'd have three builders in the suggested builders list, which
seems excessive :-)

As such, the older stack has been removed from the trusted and suggested builders
list. This is a breaking change for anyone using that image, since they will need to
explicitly add it to their trusted builders list (so this will need to be in a new major
Pack release).

Signed-off-by: Ed Morley <501702+edmorley@users.noreply.github.com>
Copy link
Member

@dfreilich dfreilich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change! I'm sorry for the time it took to get to this.

@dfreilich dfreilich merged commit 9e71e32 into buildpacks:main Jan 3, 2022
@edmorley
Copy link
Contributor Author

edmorley commented Jan 3, 2022

No problem at all! Thank you for the review/merge :-)

@edmorley edmorley deleted the remove-heroku-buildpacks-18 branch January 3, 2022 16:00
@dfreilich dfreilich modified the milestones: 0.23.0, 0.24.0 Jan 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/chore Issue that requests non-user facing changes. type/enhancement Issue that requests a new feature or improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants