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


##########
docker/pulsar/Dockerfile:
##########
@@ -69,7 +69,9 @@ RUN mkdir -p /etc/apt/keyrings \
      && echo "deb [signed-by=/etc/apt/keyrings/adoptium.asc] 
https://packages.adoptium.net/artifactory/deb $(awk -F= 
'/^VERSION_CODENAME/{print$2}' /etc/os-release) main" | tee 
/etc/apt/sources.list.d/adoptium.list \
      && apt-get update \
      && apt-get -y dist-upgrade \
-     && apt-get -y install temurin-17-jdk
+     && apt-get -y install temurin-17-jdk \
+     && export architecture=$(uname -m | sed -r 's/aarch64/arm64/g' |  awk 
'!/arm64/{$0="amd64"}1') \

Review Comment:
   Resolved at 
https://github.com/apache/pulsar/pull/17733/commits/1909f25075609c1537afb397a0626c9b810058b0.



##########
tests/docker-images/java-test-image/Dockerfile:
##########
@@ -46,10 +46,9 @@ RUN mkdir -p /etc/apt/keyrings \
      && echo "deb [signed-by=/etc/apt/keyrings/adoptium.asc] 
https://packages.adoptium.net/artifactory/deb $(awk -F= 
'/^VERSION_CODENAME/{print$2}' /etc/os-release) main" | tee 
/etc/apt/sources.list.d/adoptium.list \
      && apt-get update \
      && apt-get -y dist-upgrade \
-     && apt-get -y install temurin-17-jdk
-
-ENV JAVA_HOME /usr/lib/jvm/temurin-17-jdk-amd64
-RUN echo networkaddress.cache.ttl=1 >> 
/usr/lib/jvm/temurin-17-jdk-amd64/conf/security/java.security
+     && apt-get -y install temurin-17-jdk \
+     && export architecture=$(uname -m | sed -r 's/aarch64/arm64/g' |  awk 
'!/arm64/{$0="amd64"}1') \

Review Comment:
   Resolved at 
https://github.com/apache/pulsar/pull/17733/commits/1909f25075609c1537afb397a0626c9b810058b0.



##########
tests/docker-images/latest-version-image/Dockerfile:
##########
@@ -29,13 +29,8 @@ RUN apt-get install -y procps curl git build-essential
 
 ENV GOLANG_VERSION 1.15.8
 
-RUN curl -sSL https://golang.org/dl/go$GOLANG_VERSION.linux-amd64.tar.gz \
-               | tar -C /usr/local -xz
-
-# RUN wget https://dl.google.com/go/go1.13.3.linux-amd64.tar.gz && tar -xvf 
go1.13.3.linux-amd64.tar.gz && mv go /usr/local
-# RUN export GOROOT=/usr/local/go && export GOPATH=$HOME/go && export 
PATH=$GOPATH/bin:$GOROOT/bin:$PATH
-# RUN echo "export GOROOT=/usr/local/go" >> ~/.profile && echo "export 
GOPATH=$HOME/go" >> ~/.profile && echo "export 
PATH=$GOPATH/bin:$GOROOT/bin:$PATH" >> ~/.profile
-
+RUN export architecture=$(uname -m | sed -r 's/aarch64/arm64/g' |  awk 
'!/arm64/{$0="amd64"}1') \

Review Comment:
   Resolved at 
https://github.com/apache/pulsar/pull/17733/commits/1909f25075609c1537afb397a0626c9b810058b0.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to