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

HDDS-11915. Netty OpenSsl not available on arm64 #36

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Netty OpenSsl is not available on arm64:

UnsatisfiedLinkError: /tmp/liborg_apache_ratis_thirdparty_netty_tcnative_linux_aarch_6412907517136400085340.so: libcrypt.so.1: cannot open shared object file: No such file or directory

Install libxcrypt-compat explicitly to fix it. (Previously it was automatically installed as dependency on amd64, but not on arm64.)

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-11915

How was this patch tested?

Tested the image built on push to my fork:

$ docker run -it --rm -v ~/.m2:/opt/hadoop/.m2 \
  ghcr.io/adoroszlai/ozone-runner:052dedfe508e9ea459824846d7c8ee317fb47ef9 \
  bash -c "echo 'org.apache.ratis.thirdparty.io.netty.handler.ssl.OpenSsl.isAvailable()' | jshell --class-path ~/.m2/repository/org/apache/ratis/ratis-thirdparty-misc/1.0.6/ratis-thirdparty-misc-1.0.6.jar"
...

jshell> org.apache.ratis.thirdparty.io.netty.handler.ssl.OpenSsl.isAvailable()$1 ==> true

Copy link
Contributor

@dombizita dombizita left a comment

Choose a reason for hiding this comment

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

Thank you for investigating this and quickly fixing it @adoroszlai!

Copy link

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

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

@adoroszlai adoroszlai merged commit 19d55c4 into apache:master Dec 12, 2024
1 check passed
@adoroszlai adoroszlai deleted the libcrypt branch December 12, 2024 16:13
@adoroszlai
Copy link
Contributor Author

Thanks @dombizita, @jojochuang for the review.

adoroszlai added a commit to adoroszlai/ozone-docker-runner that referenced this pull request Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants