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.

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

[bug] Health check for guac rest docker is broken #2002

Closed
mlieberman85 opened this issue Jul 7, 2024 · 0 comments · Fixed by #2001
Closed

[bug] Health check for guac rest docker is broken #2002

mlieberman85 opened this issue Jul 7, 2024 · 0 comments · Fixed by #2001
Labels
bug Something isn't working

Comments

@mlieberman85
Copy link
Collaborator

mlieberman85 commented Jul 7, 2024

Describe the bug
When running docker compose on guac-demo-compose.yaml it appears that GUAC REST component fails the health check. Here is the output of docker ps:

CONTAINER ID   IMAGE                         COMMAND                  CREATED      STATUS                    PORTS                    NAMES
69cf49d0f6a5   ghcr.io/guacsec/guac:v0.7.2   "/opt/guac/guacone c…"   2 days ago   Up 47 hours                                        talks-guac-depsdev-collector-1
781201da2c92   ghcr.io/guacsec/guac:v0.7.2   "/opt/guac/guacone c…"   2 days ago   Up 47 hours                                        talks-guac-osv-certifier-1
539385d2ddcb   ghcr.io/guacsec/guac:v0.7.2   "/opt/guac/guacrest …"   2 days ago   Up 47 hours (unhealthy)   0.0.0.0:8081->8081/tcp   talks-guac-guac-rest-1
552e2d8c1714   ghcr.io/guacsec/guac:v0.7.2   "/opt/guac/guacgql -…"   2 days ago   Up 2 days (healthy)       0.0.0.0:8080->8080/tcp   talks-guac-graphql-1
a86a5cd4dcba   ghcr.io/guacsec/guac:v0.7.2   "/opt/guac/guaccsub"     2 days ago   Up 2 days (healthy)       0.0.0.0:2782->2782/tcp   talks-guac-collectsub-1

Even though it fails the check though it seems to be healthy and working. Here's the output of the REST logs:

{"level":"info","ts":1720355816.7454953,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000036875}
{"level":"info","ts":1720355826.7956483,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000032542}
{"level":"info","ts":1720355836.8528605,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000036459}
{"level":"info","ts":1720355846.9241624,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000035375}
{"level":"info","ts":1720355856.9841287,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000031542}
{"level":"info","ts":1720355867.050839,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000065708}
{"level":"info","ts":1720355877.1276712,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000031834}
{"level":"info","ts":1720355887.1973948,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000087}
{"level":"info","ts":1720355897.2716732,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000071833}
{"level":"info","ts":1720355907.3278902,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000033458}
{"level":"info","ts":1720355917.3791525,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.0000255}
{"level":"info","ts":1720355927.4440491,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000037875}
{"level":"info","ts":1720355937.511789,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000081208}
{"level":"info","ts":1720355947.571566,"caller":"server/server.go:61","msg":"Request handled successfully","guac-version":"v0.7.2","method":"HEAD","path":"/healthz","latency":0.000186958}

In addition docker compose ls says:

NAME                STATUS              CONFIG FILES
talks-guac          running(5)          /Users/mlieberman/Projects/talks-guac/guac-demo-compose.yaml

According to our docs: https://docs.guac.sh/setup/ -- The running(5) should be a running(7) I don't know if we changed stuff so that 5 is now accurate and the docs are incorrect or if the docker compose is broken.

To Reproduce
Run: docker compose up --force-recreate against the most recent release of guac-demo-compose.yaml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant