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

[7.0] Increase dns-app update status checks (#2324) #2326

Merged
merged 3 commits into from
Nov 18, 2020

Conversation

knisbet
Copy link
Contributor

@knisbet knisbet commented Nov 15, 2020

Description

The number of retry attempts when checking the status of rigging changes may be too short for clusters experiencing high scheduler load. We have a customer app that appears to crash during the app update, creating lots of kubernetes scheduler events and the dns pods can take more than 2 minutes to start.

Update the retry attempts to allow for ~10 minutes.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Linked tickets and other PRs

Updates #2282
Backport #2324

TODOs

  • Self-review the change
  • Address review feedback

(cherry picked from commit 2fd474a)

The number of retry attempts when checking the status of rigging changes may be too short for clusters experiencing high scheduler load. We have a customer app that appears to crash during the app update, creating lots of kubernetes scheduler events and the dns pods can take more than 2 minutes to start.

(cherry picked from commit 2fd474a)
@knisbet knisbet requested review from a team, r0mant and bernardjkim November 15, 2020 05:19
@knisbet knisbet changed the title Increase dns-app update status checks (#2324) [7.0] Increase dns-app update status checks (#2324) Nov 15, 2020
@knisbet knisbet merged commit a8c934b into version/7.0.x Nov 18, 2020
@knisbet knisbet deleted the kevin/7.0.x/dns-app-retry-attempts branch November 18, 2020 18:32
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.

3 participants