Skip to content
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

Healthcheck doesn't work (Cannot find module '/app/dist/healthcheck.js') #699

Closed
SystemParadox opened this issue Jan 23, 2021 · 2 comments · Fixed by #700
Closed

Healthcheck doesn't work (Cannot find module '/app/dist/healthcheck.js') #699

SystemParadox opened this issue Jan 23, 2021 · 2 comments · Fixed by #700
Labels
released

Comments

@SystemParadox
Copy link

The healthcheck isn't working with the latest image on dockerhub.

Error: Cannot find module '/app/dist/healthcheck.js'

The Dockerfile has WORKDIR /app and then HEALTHCHECK --interval=5s CMD node healthcheck.js, but later on there is WORKDIR /app/dist which seems to be affecting the path used for the healthcheck.
cblanc added a commit that referenced this issue Jan 24, 2021
@cblanc
fix(Healthcheck): Fixues docker healthcheck
d301472 
Points dockerfile to location of new healthcheck script

Fixes #699
@cblanc cblanc mentioned this issue Jan 24, 2021
Merged
cblanc added a commit that referenced this issue Jan 24, 2021
@cblanc
fix(Healthcheck): Fixues docker healthcheck
12ec633 
Points dockerfile to location of new healthcheck script

Fixes #699
idealbot pushed a commit that referenced this issue Jan 24, 2021
@semantic-release-bot
chore(release): 13.0.4 [skip ci]
ff3a192 
## [13.0.4](13.0.3...13.0.4) (2021-01-24)

### Bug Fixes

* **Healthcheck:** Fixues docker healthcheck ([12ec633](12ec633)), closes [#699](#699)
@idealbot
Copy link
Member

🎉 This issue has been resolved in version 13.0.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@cblanc
Copy link
Member

cblanc commented Jan 24, 2021

Thanks for the report. Forgot to include the script in the build output

Should be fixed in 13.0.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
released
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(Healthcheck): Fixes docker healthcheck ideal-postcodes/postcodes.io
3 participants
@SystemParadox @cblanc @idealbot