-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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.
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
Add HEALTHCHECK
to all services
#296
Conversation
README.md
Outdated
@@ -41,6 +41,12 @@ cd opentelemetry-demo/ | |||
|
|||
#### Run Docker Compose | |||
|
|||
- Build an image that downloads required files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How will this integrate with the CI / CD? Do I need to add something to a GitHub workflow?
HEALTHCHECK
to all services
Are the docker health checks re-used within a Kubernetes deployment? Do we need to update our Helm chart to reflect this? |
I do a
|
In the colored output in my terminal, this is the only red error output line from the above command
|
I conflicted the merge conflict, need to add instruction to |
Volume mapping? Map something under the host like |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
Fixes #89
Changes
grpc_health_probe
binary.HEALTHCHECK
command to all the servicesDockerfile
s and the command to copy it over from theincluder
docker imageemailservice
andfrontend
do theirHEALTHCHECK
with an HTTP endpoint/_healthz
. Other services usegRPC
.I will make a follow-up PR where I add tests to verify the services are reporting their health correctly, which will rely on #284 being merged :)
Screenshots:
Ignoring the feature-flag-service which is dead and the services that we do not build all show starting here
and after a bit they all show healthy
For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes