Skip to content

Set terminationGracePeriodSeconds to 2 hours to avoid kas pod kills, set upgrade strategy to support

Mikhail Mazurskiy requested to merge ash2k/bump-terminationGracePeriodSeconds into master

What does this MR do?

Set terminationGracePeriodSeconds to 2 hours to avoid kas pod kills. See issue for more context.

Set upgrade strategy to maxSurge ("This allows Chart to finish deploying even if old Pods are still around" prod MR, analysis)

Related KAS shutdown documentation

Related issues

Relates to Investigate tunnel registration GC spike w.r.t ... (gitlab-org/cluster-integration/gitlab-agent#589).

Similar change to GitLab.com deployment: kas: set terminationGracePeriodSeconds to 2 hours (gitlab-com/gl-infra/k8s-workloads/gitlab-com!3760 - merged).

Author checklist

For general guidance, please follow our Contributing guide.

Required

For anything in this list which will not be completed, please provide a reason in the MR discussion.

  • Merge Request Title and Description are up to date, accurate, and descriptive.
  • MR targeting the appropriate branch.
  • MR has a green pipeline.
  • Documentation created/updated.
  • Tests added/updated, and test plan for scenarios not covered by automated tests.
  • Equivalent MR/issue for omnibus-gitlab opened.

Reviewers checklist

Edited by Jason Plum

Merge request reports