Skip to content

Commit

Permalink
Fix docker file monitor service (#295)
Browse files Browse the repository at this point in the history
* [release] 20240605 eth

* Fix dockerfile for default monitor directory

* tidy up
  • Loading branch information
jiqiang90 authored Jun 5, 2024
1 parent 15c7adb commit 78aa675
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 0 deletions.
3 changes: 3 additions & 0 deletions packages/node/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Fixed
- Update dockerfile to fix monitor default directory permission issue

## [4.4.0] - 2024-06-05
### Added
- Add monitor service to record block indexing actions in order to improve POI accuracy, and provide debug info for Admin api
Expand Down
4 changes: 4 additions & 0 deletions packages/node/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@ RUN rm /app.tgz && \
yarn cache clean && \
rm -rf /root/.npm /root/.cache

# Create ./.monitor directory and set permissions
RUN mkdir -p .monitor && \
chown 1000:1000 .monitor

# Make the user not ROOT
USER 1000

Expand Down
7 changes: 7 additions & 0 deletions packages/node/Dockerfile-flare
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,13 @@ RUN rm /app.tgz && \
yarn cache clean && \
rm -rf /root/.npm /root/.cache

# Create ./.monitor directory and set permissions
RUN mkdir -p .monitor && \
chown 1000:1000 .monitor

# Make the user not ROOT
USER 1000

# Set Entry point and command
ENTRYPOINT ["/sbin/tini", "--", "/usr/local/lib/node_modules/@subql/node-flare/bin/run"]
CMD ["-f","/app"]

0 comments on commit 78aa675

Please sign in to comment.