Skip to content

Commit

Permalink
Merge pull request #96 from che-incubator/fixdockerfiles
Browse files Browse the repository at this point in the history
fix: Don't copy /etc/passpwd from the builder since appuser is added in runtime container.
  • Loading branch information
tolusha authored Aug 3, 2023
2 parents bd75807 + f66fed9 commit b8f633b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion build/dockerfiles/brew.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ RUN microdnf -y install shadow-utils && \
echo "Installed Packages" && rpm -qa | sort -V && echo "End Of Installed Packages"

USER appuser
COPY --from=builder /etc/passwd /etc/passwd
COPY --from=builder /usr/local/bin/configbump /usr/local/bin/configbump
ENTRYPOINT [ "/usr/local/bin/configbump" ]

Expand Down
1 change: 0 additions & 1 deletion build/dockerfiles/rhel.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,5 @@ RUN microdnf -y install shadow-utils && \
echo "Installed Packages" && rpm -qa | sort -V && echo "End Of Installed Packages"

USER appuser
COPY --from=builder /etc/passwd /etc/passwd
COPY --from=builder /usr/local/bin/configbump /usr/local/bin/configbump
ENTRYPOINT [ "/usr/local/bin/configbump" ]

0 comments on commit b8f633b

Please sign in to comment.