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

Liveness and Readiness probes failing #1329

Closed
nandacamargo opened this issue Dec 28, 2021 · 7 comments · Fixed by #1339
Closed

Liveness and Readiness probes failing #1329

nandacamargo opened this issue Dec 28, 2021 · 7 comments · Fixed by #1339
Assignees
Labels
scope/infrastructure CI / Dev. Env scope/k8s K8s or helm stuff (really annoying) status/accepted An issue which has passed triage and has been accepted status/pending Further information is requested type/bug Something isn't working
Milestone

Comments

@nandacamargo
Copy link

Hello!

First of all, congrats to you all, kafka-ui is such a nice project!

We wrote a Helm Chart to deploy Kafka UI in a Kubernetes cluster, using version 0.3.0.
We are facing the following issue when using liveness and readiness probes in the k8s deployment:
Readiness probe failed: HTTP probe failed with statuscode: 404

Even using different paths like /actuator/health, /actuator/info and login that are declared here the probes are failing. Also, we checked the Helm Chart deployment you provided.

Set up

  • Kafka UI - Version: 0.3.0
  • Docker image: provectuslabs/kafka-ui:0.3.0
  • The Helm Chart applied in the cluster has the following liveness and readiness probes:
ports:
   - name: view
     containerPort: 8080
     protocol: TCP
livenessProbe:
  httpGet:
    path: /actuator/health
    port: view
  initialDelaySeconds: 60
  periodSeconds: 30
  timeoutSeconds: 10
readinessProbe:
  httpGet:
    path: /actuator/health
    port: view
  initialDelaySeconds: 60
  periodSeconds: 30
  timeoutSeconds: 10

Removing the probes, it is possible to use Kafka UI normally in an integration environment in Kubernetes cluster, but we really need the probes before deploying in production.

Thanks a lot!

@nandacamargo nandacamargo added the type/bug Something isn't working label Dec 28, 2021
@github-actions github-actions bot added the status/triage Issues pending maintainers triage label Dec 28, 2021
@github-actions
Copy link

Hello there nandacamargo! 👋

Thank you and congratulations 🎉 for opening your very first issue in this project! 💖

In case you want to claim this issue, please comment down below! We will try to get back to you as soon as we can. 👀

@Haarolean
Copy link
Contributor

Hi, thanks for reaching out.
Does the app work itself? Which authentication mechanism (if any) are you using?

@Haarolean Haarolean added status/accepted An issue which has passed triage and has been accepted status/pending Further information is requested and removed status/triage Issues pending maintainers triage labels Dec 28, 2021
@Haarolean Haarolean self-assigned this Dec 28, 2021
@Haarolean Haarolean added scope/k8s K8s or helm stuff (really annoying) scope/infrastructure CI / Dev. Env labels Dec 28, 2021
@nandacamargo
Copy link
Author

Hi!

Yes, the app works and an additional information that might help is that besides testing with Kafka-UI version 0.3.0, we currently have the same app working in the cluster running with version 0.2.0. It works using the old liveness/readiness probe:
path: /index.html.

Since there are some changes/fixes in Kafka-UI version 0.3.0, we decided to update our deployment to keep up to date. So initially we tried using the probes with /index.html path for version 0.3.0 as well, but they failed, so we tried with different paths mentioned previously, without success.

Regarding to the authentication mechanism, we are using environment variable AUTH_TYPE, with value LOGIN_FORM, passing an username and password.

Thanks!

@Haarolean Haarolean added this to the 0.3.1 milestone Dec 29, 2021
Haarolean added a commit that referenced this issue Dec 29, 2021
@Haarolean Haarolean linked a pull request Dec 29, 2021 that will close this issue
13 tasks
@Haarolean
Copy link
Contributor

Haarolean commented Dec 29, 2021

@nandacamargo thanks much for the detailed feedback!
This is to be fixed within #1339 and we'll release a separate build shortly after.

@nandacamargo
Copy link
Author

Great! Thank you so much!

@batazor
Copy link

batazor commented Jan 29, 2023

@Haarolean Hi. I set SERVER_SERVLET_CONTEXT_PATH: "/kafka-ui"

My logs:

2023-01-29 18:22:41,591 INFO  [background-preinit] o.h.v.i.u.Version: HV000001: Hibernate Validator 6.2.5.Final
2023-01-29 18:22:41,905 INFO  [main] c.p.k.u.KafkaUiApplication: Starting KafkaUiApplication using Java 17.0.5 on kafka-ui-5d458c5d-v2pfq with PID 1 (/kafka-ui-api.jar started by kafkaui in /)
2023-01-29 18:22:41,907 DEBUG [main] c.p.k.u.KafkaUiApplication: Running with Spring Boot v2.7.5, Spring v5.3.23
2023-01-29 18:22:43,028 INFO  [main] c.p.k.u.KafkaUiApplication: No active profile set, falling back to 1 default profile: "default"
2023-01-29 18:24:12,400 DEBUG [main] c.p.k.u.s.SerdesInitializer: Configuring serdes for cluster shortlink

I get:

Readiness probe failed: Get "http://10.233.102.248:8080/kafka-ui/actuator/health"
Liveness probe failed: Get "http://10.233.102.248:8080/kafka-ui/actuator/health":

What can we do about this problem?

source: https://github.com/shortlink-org/shortlink/blob/main/ops/Helm/addons/mq/kafka/values.yaml

@Haarolean
Copy link
Contributor

@batazor hey, please raise a new discussion (or feel free to chat with me on our discord), and provide the info regarding what you see/get when you visit the probe URLs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/infrastructure CI / Dev. Env scope/k8s K8s or helm stuff (really annoying) status/accepted An issue which has passed triage and has been accepted status/pending Further information is requested type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants