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 back to the example alpine builder in CI #668

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Sep 7, 2023

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.

@edmorley edmorley self-assigned this Sep 7, 2023
@edmorley
Copy link
Member Author

edmorley commented Sep 7, 2023

Depends on buildpacks/samples#170

@edmorley edmorley force-pushed the edmorley/ci-alpine-latest branch 2 times, most recently from 2e0c776 to 14d5115 Compare September 25, 2023 12:52
@Malax
Copy link
Member

Malax commented Feb 12, 2024

@edmorley should we close this as it won't apply cleanly anymore after #775. Or do you want to keep this around as a loud reminder?

@edmorley
Copy link
Member Author

I'm happy to close this for now; we have the TODO in the workflow file, plus I'm still subscribed to the upstream issue in their examples repo.

@edmorley edmorley closed this Feb 12, 2024
@edmorley edmorley deleted the edmorley/ci-alpine-latest branch February 12, 2024 14:13
@edmorley edmorley restored the edmorley/ci-alpine-latest branch October 25, 2024 09:32
@edmorley edmorley reopened this Oct 25, 2024
@edmorley edmorley marked this pull request as ready for review October 25, 2024 09:38
@edmorley edmorley requested a review from a team as a code owner October 25, 2024 09:38
@edmorley edmorley changed the title Unpin version of the example alpine builder used in CI Switch back to the example alpine builder in CI 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.
@edmorley edmorley enabled auto-merge (squash) October 25, 2024 09:42
@edmorley edmorley merged commit 2a3299c into main Oct 25, 2024
4 checks passed
@edmorley edmorley deleted the edmorley/ci-alpine-latest branch October 25, 2024 11:38
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