Skip to content

chore(argo-cd): Align haproxy connections default timeout with Argo CD redis client#3244

Open
OpenGuidou wants to merge 1 commit intoargoproj:mainfrom
OpenGuidou:haproxy
Open

chore(argo-cd): Align haproxy connections default timeout with Argo CD redis client#3244
OpenGuidou wants to merge 1 commit intoargoproj:mainfrom
OpenGuidou:haproxy

Conversation

@OpenGuidou
Copy link
Copy Markdown

@OpenGuidou OpenGuidou commented Apr 11, 2025

Related to argoproj/argo-cd#22489.
Pending the possibility to customize the client timeout on Argo CD side, this change aligns the connection timeout settings with HA proxy so that clients are not disconnected when it happens.

Checklist:

  • I have bumped the chart version according to versioning
  • I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

@OpenGuidou OpenGuidou changed the title chore: Align haproxy connections default timeout with Argo CD redis client chore(argo-cd): Align haproxy connections default timeout with Argo CD redis client Apr 11, 2025
@tico24
Copy link
Copy Markdown
Member

tico24 commented Apr 11, 2025

Thanks for this, but this won't get released until it has been released upstream.

@tico24 tico24 marked this pull request as draft April 11, 2025 07:46
@github-actions github-actions bot added size/S and removed size/XS labels Apr 11, 2025
@OpenGuidou
Copy link
Copy Markdown
Author

Thanks for this, but this won't get released until it has been released upstream.

This is in fact a fix to make it work as long as we don't have the upstream settings.
Today the default in go redis client is 30min and we can't change that.

…D redis client

Signed-off-by: OpenGuidou <guillaume.doussin@gmail.com>
@OpenGuidou OpenGuidou marked this pull request as ready for review April 11, 2025 08:05
@yu-croco yu-croco added the awaiting-upstream Is waiting for a change upstream to be completed before it can be merged. label May 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

argo-cd awaiting-upstream Is waiting for a change upstream to be completed before it can be merged. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants