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.

Sign up for GitHub

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

[feat][build] Support ARM64-based docker images #17733

Merged
merged 8 commits into from
Sep 21, 2022
+17 −16
Merged

[feat][build] Support ARM64-based docker images #17733

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
6a2d129
detect arch for Golang installation
tisonkun Sep 20, 2022
e0d465a
detect arch for JDK installation
tisonkun Sep 20, 2022
b2d5605
fix python client build on ARM64
tisonkun Sep 20, 2022
9fe71d9
fix java_home path
tisonkun Sep 20, 2022
a13e8cb
fix resolve architecture
tisonkun Sep 20, 2022
1ae7709
revert chained run commands
tisonkun Sep 20, 2022
1909f25
architecture -> ARCH
tisonkun Sep 20, 2022
2793885
install build-essential python3-dev only for ARM64
tisonkun Sep 20, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions docker/pulsar/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,11 +72,14 @@ RUN mkdir -p /etc/apt/keyrings \
&& apt-get -y install temurin-17-jdk \
&& export architecture=$(uname -m | sed -r 's/aarch64/arm64/g' | awk '!/arm64/{$0="amd64"}1') \
tisonkun marked this conversation as resolved.
Show resolved Hide resolved
&& echo networkaddress.cache.ttl=1 >> /usr/lib/jvm/temurin-17-jdk-$architecture/conf/security/java.security \
&& apt-get -y --purge autoremove \

# Cleanup apt
RUN apt-get -y --purge autoremove \
&& apt-get autoclean \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/* \
&& pip3 install pyyaml==5.4.1
&& rm -rf /var/lib/apt/lists/*

RUN pip3 install pyyaml==5.4.1
Copy link
Member Author

@tisonkun tisonkun Sep 20, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

I don't know why. But this change causes local build fails with returned a non-zero code: 100. So keep it as is and revert intermediate changes.


# Pulsar currently writes to the below directories, assuming the default configuration.
# Note that number 4 is the reason that pulsar components need write access to the /pulsar directory.
Expand Down