cloudfoundry / cf-for-k8s

The open source deployment manifest for Cloud Foundry on Kubernetes
Apache License 2.0
301 stars 115 forks source link

Remove cf api server update strategy #648

Closed davewalter closed 3 years ago

davewalter commented 3 years ago

WHAT is this change about?

This PR removes the update strategy from the CF API pods as it can cause API downtime during upgrades. See #177444748 for details.

Does this PR introduce a change to config/values.yml?

No

Acceptance Steps

Deploy v2.1.1, then deploy this change while monitoring the cf-api-server pod. You should see that the old pod is not terminated until the new pod is ready.

Tag your pair, your PM, and/or team

linux-foundation-easycla[bot] commented 3 years ago

CLA Signed

The committers are authorized under a signed CLA.

cf-gitbot commented 3 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/177553123

The labels on this github issue will be updated when the story is started.