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

Scylla readiness probe does not work #132

Open
yuha0 opened this issue Jun 12, 2020 · 4 comments
Open

Scylla readiness probe does not work #132

yuha0 opened this issue Jun 12, 2020 · 4 comments

Comments

@yuha0
Copy link

yuha0 commented Jun 12, 2020

The readiness probe defined in here:

It uses exec to execute the probe script, which will always return 0.

➤ kubectl get pods scylla-2
NAME       READY   STATUS    RESTARTS   AGE
scylla-2   1/1     Running   0          15m

➤ kubectl exec scylla-2 bash /opt/ready-probe.sh
command terminated with exit code 1

➤ kubectl exec scylla-2 -- bash -c exec /opt/ready-probe.sh
(returned 0 here)

➤ kubectl get pods scylla-2 -ojsonpath='{.spec.containers[].readinessProbe}'
map[exec:map[command:[/bin/bash -c exec /opt/ready-probe.sh]] failureThreshold:3 initialDelaySeconds:15 periodSeconds:10 successThreshold:1 timeoutSeconds:5]
@tzach
Copy link
Collaborator

tzach commented Jun 14, 2020

@dahankzter is this example relevant?
If its out of date we can (should) kill it and direct users to https://github.com/scylladb/scylla-operator

@dahankzter
Copy link
Contributor

It's very much obsolete and we should either change it here or redirect them to the operator repo.

@yuha0
Copy link
Author

yuha0 commented Jun 15, 2020

Unless the plan is to make the operator the only way to deploy scylla in k8s, I think a standalone example as a statefulset is still useful.

Also, I took a look at the operator and looks like it uses a sidecar to run nodetool and exposes an http endpoint. That is definitely a nicer interface, but the idea is the same.

@dahankzter
Copy link
Contributor

The idea of the operator isn't to make it impossible the deploy scylla without it but simply to make it easier. It will be hard to keep these examples in sync though as time goes by but personally I don't mind it all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants