This is an automated email from the ASF dual-hosted git repository.

nicoloboschi pushed a commit to branch branch-4.16
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git

commit 017e7118640fc135566ac67d44a8e8458b6ead96
Author: 赤月 <[email protected]>
AuthorDate: Mon Jan 29 20:19:44 2024 +0800

    [improve][docker] fix yaml and dockerfile (#4186)
    
    Descriptions of the changes in this PR:
    1. There are multiple spec.ports fields in sts yaml
    2. The RUN command specified in dockerfile is not standardized
    
    Master Issue: #4185
    
    (cherry picked from commit 1eceb5dd2180ab57e5b21be968206292bb825177)
---
 deploy/kubernetes/gke/bookkeeper.statefulset.yml | 1 -
 tests/docker-images/statestore-image/Dockerfile  | 8 ++++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/deploy/kubernetes/gke/bookkeeper.statefulset.yml 
b/deploy/kubernetes/gke/bookkeeper.statefulset.yml
index 49548f09d0..ca32b91137 100644
--- a/deploy/kubernetes/gke/bookkeeper.statefulset.yml
+++ b/deploy/kubernetes/gke/bookkeeper.statefulset.yml
@@ -124,7 +124,6 @@ metadata:
     app: bookkeeper
     component: bookie
 spec:
-  ports:
   ports:
     - name: bookie
       port: 3181
diff --git a/tests/docker-images/statestore-image/Dockerfile 
b/tests/docker-images/statestore-image/Dockerfile
index dc48fd5120..5a7a168885 100644
--- a/tests/docker-images/statestore-image/Dockerfile
+++ b/tests/docker-images/statestore-image/Dockerfile
@@ -42,10 +42,10 @@ RUN set -x \
     && ln -s /usr/bin/python3 /usr/bin/python \
     && apt-get -y install netcat dnsutils less procps iputils-ping \
     && apt-get install -y --no-install-recommends gpg gpg-agent wget sudo \
-     && apt-get -y --purge autoremove \
-     && apt-get autoclean \
-     && apt-get clean \
-     && rm -rf /var/lib/apt/lists/*
+    && apt-get -y --purge autoremove \
+    && apt-get autoclean \
+    && apt-get clean \
+    && rm -rf /var/lib/apt/lists/* \
     && pip install zk-shell \
     && mkdir -pv /opt \
     && JAVA_HOME=$(dirname $(dirname $(readlink -f $(which java)))) \

Reply via email to