Skip to content

Revert "Add health check on API server"#530

Closed
merenbach wants to merge 1 commit intomasterfrom
revert-522-520-add-health-probes
Closed

Revert "Add health check on API server"#530
merenbach wants to merge 1 commit intomasterfrom
revert-522-520-add-health-probes

Conversation

@merenbach
Copy link
Contributor

@merenbach merenbach commented Aug 23, 2018

Reverts #522

There's some sort of issue in further testing on my end. @jessesuen After blowing away my whole environment and redoing all my tests, I'm getting one out of two argocd-server pods not coming up with this error in describe:

Warning Unhealthy 1m (x8 over 4m) kubelet, minikube Readiness probe failed: HTTP probe failed with statuscode: 404

I'm not sure why this HTTP endpoint wouldn't be working on just one of two replicas, since they should be the same product, and /healthz is being served from the latest codebase:

$ curl 127.0.0.1:8080/healthz
ok

Proposing a rollback of the liveness probes for now unless this can be resolved or is determined to be a non-issue (i.e., my further tests are flawed).

@merenbach merenbach requested a review from jessesuen August 23, 2018 18:44
@merenbach merenbach closed this Aug 23, 2018
@jessesuen jessesuen deleted the revert-522-520-add-health-probes branch September 5, 2018 01:13
leoluz pushed a commit to leoluz/argo-cd that referenced this pull request Mar 13, 2025
Signed-off-by: fengshunli <1171313930@qq.com>
Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Co-authored-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant