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

Readiness probe improvements #8

Merged
merged 5 commits into from
Oct 28, 2019
Merged

Readiness probe improvements #8

merged 5 commits into from
Oct 28, 2019

Conversation

mpereira
Copy link
Contributor

@mpereira mpereira commented Oct 28, 2019

  • Move "readiness probe"-related params to the operator settings block.
  • Prepend NODE_ to "readiness probe"-related params to be consistent with per-container naming.
  • s/cassandra-readiness-probe/node-readiness-probe/ to be consistent with per-container naming.

Also, prepend "NODE" since they're for the "node" container. It's likely that
we'll add readiness probes for other containers.
To be consistent with per-container naming.
Copy link
Contributor

@phrynchanka phrynchanka left a comment

Choose a reason for hiding this comment

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

LGTM!

@mpereira mpereira merged commit 2eea33e into DCOS-58972 Oct 28, 2019
@mpereira mpereira deleted the node-readiness-probe branch October 28, 2019 08:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants