Skip to content

Commit

Permalink
fix: rename gcsfuse script (GoogleCloudPlatform#3369)
Browse files Browse the repository at this point in the history
* run node index.js in bg; rename gcsfuse sh script

* update name in dockerfile too
  • Loading branch information
rogerthatdev authored Jul 28, 2023
1 parent e3bbf60 commit e2bbb0c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions run/filesystem/gcsfuse.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -48,12 +48,12 @@ RUN npm install --only=production
COPY . ./

# Ensure the script is executable
RUN chmod +x /app/gcsfuse.run.sh
RUN chmod +x /app/gcsfuse_run.sh

# Use tini to manage zombie processes and signal forwarding
# https://github.com/krallin/tini
ENTRYPOINT ["/usr/bin/tini", "--"]

# Pass the wrapper script as arguments to tini
CMD ["/app/gcsfuse.run.sh"]
CMD ["/app/gcsfuse_run.sh"]
# [END cloudrun_fuse_dockerfile]
File renamed without changes.
2 changes: 1 addition & 1 deletion run/filesystem/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ mount -o nolock $FILESTORE_IP_ADDRESS:/$FILE_SHARE_NAME $MNT_DIR
echo "Mounting completed."

# Start the application
node index.js
node index.js &

# Exit immediately when one of the background processes terminate.
wait -n
Expand Down

0 comments on commit e2bbb0c

Please sign in to comment.