Skip to content

fix: move entrypoint scripts from /opt to home directory #5986

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from
Closed

fix: move entrypoint scripts from /opt to home directory #5986

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b6b12c2
Update standalone.Dockerfile
DeVikingMark Oct 7, 2025
10bdc45
Update multiplexer.Dockerfile
DeVikingMark Oct 7, 2025
931df8f
Update Dockerfile
DeVikingMark Oct 7, 2025
ee2df25
Merge branch 'main' into move-entrypoint-scripts-from-/opt-to-home-di…
rootulp Oct 7, 2025
d60d480
Update standalone.Dockerfile
DeVikingMark Oct 26, 2025
22e9939
Update multiplexer.Dockerfile
DeVikingMark Oct 26, 2025
5eeffe5
Merge branch 'main' into move-entrypoint-scripts-from-/opt-to-home-di…
DeVikingMark Oct 26, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docker/multiplexer.Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ RUN apk update && apk add --no-cache \
# Copy the celestia-appd binary from the builder into the final image.
COPY --from=builder /celestia-app/build/celestia-appd /bin/celestia-appd
# Copy the entrypoint script into the final image.
COPY --chown=${USER_NAME}:${USER_NAME} docker/entrypoint.sh /opt/entrypoint.sh
COPY --chown=${USER_NAME}:${USER_NAME} docker/entrypoint.sh ${CELESTIA_APP_HOME}/entrypoint.sh
# Set the user to celestia.
USER ${USER_NAME}
# Set the working directory to the home directory.
Expand All @@ -122,4 +122,4 @@ WORKDIR ${CELESTIA_APP_HOME}
# 26660 is the port used for Prometheus.
# 26661 is the port used for tracing.
EXPOSE 1317 9090 26656 26657 26660 26661
ENTRYPOINT [ "/bin/bash", "/opt/entrypoint.sh" ]
ENTRYPOINT [ "/bin/bash", "/home/celestia/.celestia-app/entrypoint.sh" ]
4 changes: 2 additions & 2 deletions docker/standalone.Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ RUN apk update && apk add --no-cache \
# Copy the celestia-appd binary from the builder into the final image.
COPY --from=builder /celestia-app/build/celestia-appd /bin/celestia-appd
# Copy the entrypoint script into the final image.
COPY --chown=${USER_NAME}:${USER_NAME} docker/entrypoint.sh /opt/entrypoint.sh
COPY --chown=${USER_NAME}:${USER_NAME} docker/entrypoint.sh ${CELESTIA_APP_HOME}/entrypoint.sh
# Set the user to celestia.
USER ${USER_NAME}
# Set the working directory to the home directory.
Expand All @@ -83,4 +83,4 @@ WORKDIR ${CELESTIA_APP_HOME}
# 26660 is the port used for Prometheus.
# 26661 is the port used for tracing.
EXPOSE 1317 9090 26656 26657 26660 26661
ENTRYPOINT [ "/bin/bash", "/opt/entrypoint.sh" ]
ENTRYPOINT [ "/bin/bash", "/home/celestia/.celestia-app/entrypoint.sh" ]
4 changes: 2 additions & 2 deletions docker/txsim/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,11 @@ RUN apk update && apk add --no-cache \
# Copy in the txsim binary
COPY --from=builder /celestia-app/build/txsim /bin/txsim

COPY --chown=${USER_NAME}:${USER_NAME} docker/txsim/entrypoint.sh /opt/entrypoint.sh
COPY --chown=${USER_NAME}:${USER_NAME} docker/txsim/entrypoint.sh ${CELESTIA_APP_HOME}/entrypoint.sh
Copy link
Collaborator

@rootulp rootulp Oct 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated the pullrequest to apply changes only to local_devnet

txsim can be used outside of local_devnet

Copy link
Collaborator

@rootulp rootulp Oct 27, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've re-read the original issue and I still don't understand the concrete problem and proposal. I think we need more clarity in #4670 before we try implementing it.


USER ${USER_NAME}

# Set the working directory to the home directory.
WORKDIR ${CELESTIA_APP_HOME}

ENTRYPOINT [ "/bin/bash", "/opt/entrypoint.sh" ]
ENTRYPOINT [ "/bin/bash", "/home/celestia/.celestia-app/entrypoint.sh" ]