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

Use heroku/builder:22 in example-basics integration test #775

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

Malax
Copy link
Member

@Malax Malax commented Feb 9, 2024

See code comment for details. This became a blocker for #773 since cnbs/sample-builder@sha256:da5ff69191919f1ff30d5e28859affff8e39f23038137c7751e24a42e919c1ab doesn't contain a CNB lifecycle that is compatible with buildpack API 0.10.

@Malax Malax changed the title Use heroku/heroku:22 in example-basics integration test Use heroku/heroku:22 in example-basics integration test Feb 9, 2024
@Malax Malax changed the title Use heroku/heroku:22 in example-basics integration test Use heroku/builder:22 in example-basics integration test Feb 9, 2024
@Malax Malax marked this pull request as ready for review February 9, 2024 10:36
@Malax Malax requested a review from a team as a code owner February 9, 2024 10:36
@Malax Malax merged commit ff8203b into main Feb 9, 2024
4 checks passed
@Malax Malax deleted the malax/ci-changes branch February 9, 2024 10:50
edmorley added a commit that referenced this pull request Oct 25, 2024
In CI a `pack build` is performed using one of libcnb's example CNBs
and the upstream `cnb/sample-builder:alpine` builder, in order to test
that our static MUSL compile works with non-libc base images.

However, in #594 the version of that builder had to be pinned to
an older release, to work around a temporary incompatibility
between the latest version of the builder and the latest stable
Pack CLI:
buildpacks/samples#170

In addition, in #775 the image was switched to our builder image
since the example images didn't use a CNB lifecycle that was
compatible with Buildpack API 0.13.

Both issues have since been resolved, so we can switch back to
the latest version of the upstream example alpine builder,
giving us musl coverage again.

GUS-W-14086879.
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