Skip to content

Commit

Permalink
Use from k8s in the vtexplain image and fix Makefile
Browse files Browse the repository at this point in the history
Signed-off-by: Florent Poinsard <[email protected]>
  • Loading branch information
frouioui committed Apr 2, 2024
1 parent 6ccdf9b commit ad32595
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ endif
install: build
# binaries
mkdir -p "$${PREFIX}/bin"
cp "$${VTROOTBIN}/"{mysqlctl,mysqlctld,vtorc,vtadmin,vtctld,vtctlclient,vtctldclient,vtgate,vttablet,vtbackup} "$${PREFIX}/bin/"
cp "$${VTROOTBIN}/"{mysqlctl,mysqlctld,vtorc,vtadmin,vtctld,vtctlclient,vtctldclient,vtgate,vttablet,vtbackup,vtexplain} "$${PREFIX}/bin/"

# Will only work inside the docker bootstrap for now
cross-install: cross-build
Expand Down
1 change: 1 addition & 0 deletions docker/k8s/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ COPY --from=lite /vt/bin/vttablet /vt/bin/
COPY --from=lite /vt/bin/vtbackup /vt/bin/
COPY --from=lite /vt/bin/vtadmin /vt/bin/
COPY --from=lite /vt/bin/vtorc /vt/bin/
COPY --from=lite /vt/bin/vtexplain /vt/bin/

# copy web admin files
COPY --from=lite $VTROOT/web /vt/web/
Expand Down
4 changes: 2 additions & 2 deletions docker/k8s/vtexplain/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
ARG VT_BASE_VER=latest
ARG DEBIAN_VER=stable-slim

FROM vitess/lite:${VT_BASE_VER} AS lite
FROM vitess/k8s:${VT_BASE_VER} AS k8s

FROM debian:${DEBIAN_VER}

Expand All @@ -26,7 +26,7 @@ ENV VTROOT /vt
RUN mkdir -p /vt/bin && mkdir -p /vtdataroot

# Copy binaries
COPY --from=lite /vt/bin/vtexplain /vt/bin/
COPY --from=k8s /vt/bin/vtexplain /vt/bin/

# add vitess user/group and add permissions
RUN groupadd -r --gid 2000 vitess && \
Expand Down

0 comments on commit ad32595

Please sign in to comment.