Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Configure liveness probes for pods #73

Merged
merged 5 commits into from
Apr 8, 2020
Merged

Configure liveness probes for pods #73

merged 5 commits into from
Apr 8, 2020

Conversation

nfnt
Copy link
Contributor

@nfnt nfnt commented Apr 2, 2020

Liveness probes check that the Cassandra node running in the pod is responsive by calling 'nodetool info'. It does not consider the node's status provided by 'nodetool status' for liveness. As such it's only concerned by the liveness of the pods application.

@nfnt nfnt added the wip label Apr 2, 2020
@nfnt nfnt self-assigned this Apr 2, 2020
Liveness probes check that the Cassandra node running in the pod is responsive by calling 'nodetool info'. It does not consider the node's status provided by 'nodetool status' for liveness. As such it's only concerned by the liveness of the pods application.

Signed-off-by: Jan Schlicht <[email protected]>
@nfnt nfnt force-pushed the nfnt/liveness-probes branch from 7597f9c to 0b1b95c Compare April 2, 2020 12:35
@nfnt nfnt marked this pull request as ready for review April 2, 2020 14:10
@nfnt nfnt requested review from zmalik and ANeumann82 April 2, 2020 14:11
@nfnt nfnt added ready for review and removed wip labels Apr 3, 2020
Copy link
Contributor

@zmalik zmalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
lets please open an issue to discuss a replacement of nodetool for periodic checks like liveness

Jan Schlicht added 4 commits April 6, 2020 15:59
The liveness probe will check for responsiveness of the JMX port. This will reduce the overhead compared with running 'nodetool info'.

Signed-off-by: Jan Schlicht <[email protected]>
This reverts commit 9e1a02e.

Signed-off-by: Jan Schlicht <[email protected]>
@nfnt nfnt merged commit 8baba07 into master Apr 8, 2020
@nfnt nfnt deleted the nfnt/liveness-probes branch April 8, 2020 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants