Closed roivaz closed 1 year ago
/lgtm
LGTM label has been added.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: roivaz
The full list of commands accepted by this bot can be found here.
The pull request process is described here
With the addition of health checks, the rollout of new versions of the discovery service is slower because the new pod takees some seconds to pass the health checks. This makes that both the old controllers and the new ones run in parallel for some seconds, trying to write to the same EnvoyConfigRevisions and causing problems. To fix this:
I have tested an upgrade from v0.11.1 to a version with the changes in this PR and it fixes the problems.
/kind bug /priority critical-urgent /assign /ok-to-test