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

fix: Increase HAProxy check interval to prevent intermittent failures #3356

Merged
merged 2 commits into from
Apr 4, 2020

Conversation

jannfis
Copy link
Member

@jannfis jannfis commented Apr 3, 2020

Increases HAProxy check interval from 1s to 3s, see slack thread https://argoproj.slack.com/archives/CASHNF6MS/p1585935567397100

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
  • The title of the PR states what changed and the related issues number (used for the release note).
  • I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
  • Optional. My organization is added to USERS.md.
  • I've signed the CLA and my build is green (troubleshooting builds).

Closes #3358

@jannfis jannfis requested review from jessesuen and alexmt April 3, 2020 20:23
@codecov
Copy link

codecov bot commented Apr 3, 2020

Codecov Report

Merging #3356 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #3356      +/-   ##
=========================================
- Coverage   40.54%   40.5%   -0.04%     
=========================================
  Files         175     175              
  Lines       19019   19079      +60     
  Branches      237     237              
=========================================
+ Hits         7711    7728      +17     
- Misses      10377   10415      +38     
- Partials      931     936       +5
Impacted Files Coverage Δ
server/application/application.go 28.42% <0%> (-0.15%) ⬇️
controller/sync_hooks.go 0% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c428e09...3009531. Read the comment docs.

>> ./chart/upstream.yaml
>> ./chart/upstream_orig.yaml

sed -e 's/check inter 1s/check inter 3s/' ./chart/upstream_orig.yaml > ./chart/upstream.yaml && rm ./chart/upstream_orig.yaml
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍 that looks good enough to me

@alexmt alexmt merged commit aa4fb9a into argoproj:master Apr 4, 2020
alexmt pushed a commit that referenced this pull request Apr 6, 2020
…#3356)

* Increase HAProxy check interval to prevent intermittent failures/state flapping

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

Successfully merging this pull request may close these issues.

HAProxy mode causes Redis intermittent failures
2 participants