Skip to content

fix docker healthcheck unhealthy issue #672

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

Jump to bottom
Merged
overtrue merged 1 commit into from
Oct 21, 2025
Merged

fix docker healthcheck unhealthy issue #672

overtrue merged 1 commit into from
Oct 21, 2025

Conversation

@majinghe
Copy link
Contributor

@majinghe majinghe commented Oct 21, 2025

Type of Change

  • New Feature
  • Bug Fix
  • Documentation
  • Performance Improvement
  • Test/CI
  • Refactor
  • Other:

Related Issues

Summary of Changes

Due to he docker container has no curl command, which leads to the failure for healthcheck

curl -f localhost:9000/health && curl -f localhost:9001/health

Checklist

  • I have read and followed the CONTRIBUTING.md guidelines
  • Passed make pre-commit
  • Added/updated necessary tests
  • Documentation updated (if needed)
  • CI/CD passed (if applicable)

Impact

  • Breaking change (compatibility)
  • Requires doc/config/deployment update
  • Other impact:

Additional Notes

Thank you for your contribution! Please ensure your PR follows the community standards (CODE_OF_CONDUCT.md) and sign the CLA if this is your first contribution.

@majinghe majinghe requested review from loverustfs and overtrue and removed request for overtrue October 21, 2025 01:16
@majinghe
Copy link
Contributor Author

majinghe commented Oct 21, 2025

After add curl command in dockerfile and build the docker image, the healthy status comes to healthy

docker ps
CONTAINER ID   IMAGE                                                                                      COMMAND                  CREATED          STATUS                    PORTS     NAMES
1da30ae98915   crpi-oofhqnuk3aidukbe.cn-hangzhou.personal.cr.aliyuncs.com/xiaoamgex-devops/rustfs:1.0.0   "/entrypoint.sh sh -…"   25 minutes ago   Up 25 minutes (healthy)             rustfs

@overtrue overtrue merged commit 14bc554 into rustfs:main Oct 21, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@loverustfs loverustfs Awaiting requested review from loverustfs

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@majinghe @overtrue