[GitHub] [pulsar] tisonkun commented on a diff in pull request #17733: [feat][build] Support ARM64-based docker images

[GitBox <gi...@apache.org>]

tisonkun commented on code in PR #17733:
URL: https://github.com/apache/pulsar/pull/17733#discussion_r975217728


##########
docker/pulsar/Dockerfile:
##########
@@ -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') \
      && 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

Review Comment:
   I don't know why. But this change causes local build fails with `returned a non-zero code: 100`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org