Skip to content

Commit

Permalink
[improve][docker] fix yaml and dockerfile (apache#4186)
Browse files Browse the repository at this point in the history
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: apache#4185
  • Loading branch information
zwOvO authored Jan 29, 2024
1 parent 7b1912f commit 1eceb5d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
1 change: 0 additions & 1 deletion deploy/kubernetes/gke/bookkeeper.statefulset.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,6 @@ metadata:
app: bookkeeper
component: bookie
spec:
ports:
ports:
- name: bookie
port: 3181
Expand Down
8 changes: 4 additions & 4 deletions tests/docker-images/statestore-image/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ RUN set -x \
&& apt-get install -y --no-install-recommends python3 pip \
&& ln -s /usr/bin/python3 /usr/bin/python \
&& 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 \
&& cd /opt
Expand Down

0 comments on commit 1eceb5d

Please sign in to comment.