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

feat(kubernetes): add opt-in livenessProbe with configurable initialD… #1367

Merged
merged 2 commits into from
Jul 10, 2019

Conversation

maggieneterval
Copy link
Contributor

…elaySeconds

Closes spinnaker/spinnaker#4023

  • Adds two new sub-commands to the hal config deploy edit command, modeled off the code for the vault sub-commands : --liveness-probe-enabled and --liveness-probe-initial-delay-seconds.
  • Adds liveness probe configuration to Kubernetes v1 and v2 providers, abstracting logic shared with readiness probes into its own method in each of the three affected classes.
  • Adds initialDelaySeconds to manifest probe templates, and removes "readiness" from the name of each since these are now reused by liveness probes.

@maggieneterval
Copy link
Contributor Author

@plumpy if we could hold the next halyard release until this is merged I would be forever grateful 🙏

Copy link
Contributor

@ezimanyi ezimanyi left a comment

Choose a reason for hiding this comment

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

📦 🐢 🕐

@maggieneterval maggieneterval merged commit 2e4abee into spinnaker:master Jul 10, 2019
@maggieneterval maggieneterval deleted the liveness-probe-take-2 branch July 10, 2019 14:39
@davidxia
Copy link
Contributor

@maggieneterval is there a way to tell what Spinnaker version here this will be released as? https://www.spinnaker.io/community/releases/versions/

@maggieneterval
Copy link
Contributor Author

@davidxia Halyard is actually released separately from Spinnaker, but I expect @plumpy will release Halyard 1.22.0 with this change next week? 🙏

@davidxia
Copy link
Contributor

Ah, that's right. Thanks for explaining. Keep me posted on the release. Is there a page I can see the releases or changelog for halyard?

@maggieneterval
Copy link
Contributor Author

I will definitely let you know and also comment on the original issue so anyone following knows as well! As far as I know we are not as good about documenting the halyard releases on spinnaker.io, but at least the last commit in each release is documented here: https://github.com/spinnaker/halyard/releases
Generally, I think the #halyard channel in slack is currently the best place to be notified of halyard releases specifically since we don't post about them in #spinnaker-releases.

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

Successfully merging this pull request may close these issues.

Support enabling livenessProbes for Spinnaker components deployed to K8s
4 participants