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

Add libssl install to udeps task #6777

Merged
merged 2 commits into from
Jan 10, 2025
Merged

Conversation

eserilev
Copy link
Collaborator

@eserilev eserilev commented Jan 9, 2025

I'm not really sure what happened between now and yesterday, but I do know libssl1.1 has reached EOL recently. The udeps ci task wants to access libssl1.1 for some reason, so I've added a step to install it. It's not the cleanest fix, if anyone has better ideas please let me know

Here's the original error message
error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

And some informational links
https://stackoverflow.com/questions/72133316/libssl-so-1-1-cannot-open-shared-object-file-no-such-file-or-directory
https://openssl-library.org/post/2023-03-28-1.1.1-eol/

@eserilev eserilev added infra-ci ready-for-review The code is ready for review labels Jan 9, 2025
Copy link
Member

@jimmygchen jimmygchen 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 looking into this 🙏

I don't mind merging this to unblock CI!

I think our options are:

  1. install libssl 1.1 on the runner (this PR)
  2. revert to ubuntu-22.04 - this is probably not a long term solution either
  3. investigate possibility of removing ssl 1.1 dependency complelely (likely from reqwest?)

Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

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

Agree we should just merge this to unblock CI, and then hope that cargo-udeps is updated so as not to require this.

@jimmygchen jimmygchen added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Jan 10, 2025
@michaelsproul michaelsproul mentioned this pull request Jan 10, 2025
@michaelsproul
Copy link
Member

@mergify queue

Copy link

mergify bot commented Jan 10, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at a244aa3

mergify bot added a commit that referenced this pull request Jan 10, 2025
@mergify mergify bot merged commit a244aa3 into sigp:unstable Jan 10, 2025
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra-ci ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants