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

Fix CI pipeline #211

Merged
merged 6 commits into from
Feb 6, 2025
Merged

Fix CI pipeline #211

merged 6 commits into from
Feb 6, 2025

Conversation

navarone-feekery
Copy link
Collaborator

@navarone-feekery navarone-feekery commented Feb 6, 2025

This PR fixes two things:

1. Broken curl lib

Curl released an 8.12.0 yesterday. This is broken in Alpine Linux distros, calling curl with any sub command causes a crash.

Resolves CI failure likely caused by recent release of curl:

Step 6/22 : RUN mkdir /opt/jruby   && curl -fSL https://repo1.maven.org/maven2/org/jruby/jruby-dist/${JRUBY_VERSION}/jruby-dist-${JRUBY_VERSION}-bin.tar.gz -o /tmp/jruby.tar.gz   && echo "$JRUBY_SHA256 /tmp/jruby.tar.gz" | sha256sum -c -   && tar -zx --strip-components=1 -f /tmp/jruby.tar.gz -C /opt/jruby   && rm /tmp/jruby.tar.gz
 ---> Running in 9bf1d7e98f6c
curl: symbol lookup error: curl: undefined symbol: curl_easy_ssls_export
The command '/bin/sh -c mkdir /opt/jruby   && curl -fSL https://repo1.maven.org/maven2/org/jruby/jruby-dist/${JRUBY_VERSION}/jruby-dist-${JRUBY_VERSION}-bin.tar.gz -o /tmp/jruby.tar.gz   && echo "$JRUBY_SHA256 /tmp/jruby.tar.gz" | sha256sum -c -   && tar -zx --strip-components=1 -f /tmp/jruby.tar.gz -C /opt/jruby   && rm /tmp/jruby.tar.gz' returned a non-zero code: 127

2. Removal of obsolete Dockerfile

We had a second Dockerfile being used for CI testing. We should be doing this on the Dockerfile.wolfi. I've removed the old Dockerfile and changed pointers to towards Dockerfile.wolfi.

@navarone-feekery navarone-feekery changed the title Add libcurl to installed libs Pin curl version to 8.11.1 Feb 6, 2025
@navarone-feekery navarone-feekery changed the title Pin curl version to 8.11.1 Fixes to CI pipeline Feb 6, 2025
@navarone-feekery navarone-feekery changed the title Fixes to CI pipeline Fix to CI pipeline Feb 6, 2025
@navarone-feekery navarone-feekery changed the title Fix to CI pipeline Fix CI pipeline Feb 6, 2025
@navarone-feekery navarone-feekery marked this pull request as ready for review February 6, 2025 14:56
@navarone-feekery navarone-feekery requested a review from a team as a code owner February 6, 2025 14:56
@navarone-feekery navarone-feekery enabled auto-merge (squash) February 6, 2025 15:01
@navarone-feekery navarone-feekery merged commit 80e257f into main Feb 6, 2025
2 checks passed
@navarone-feekery navarone-feekery deleted the navarone/test-ci-curl-error branch February 6, 2025 15:08
github-actions bot pushed a commit that referenced this pull request Feb 6, 2025
Pin curl version and remove obsolete Dockerfile
Copy link

github-actions bot commented Feb 6, 2025

💚 Backport PR(s) successfully created

Status Branch Result
0.2 #212

This backport PR will be merged automatically after passing CI.

navarone-feekery added a commit that referenced this pull request Feb 6, 2025
Backports the following commits to 0.2:
 - Fix CI pipeline (#211)

Co-authored-by: Navarone Feekery <13634519+navarone-feekery@users.noreply.github.com>
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