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

add retries request #232

Closed
wants to merge 7 commits into from
Closed

add retries request #232

wants to merge 7 commits into from

Conversation

emilevauge
Copy link
Member

This PR adds retry request in case of network error.
You can enable retry in your global config file adding [retry].
The default config set a maximum storage in memory by request of 2 Mo and a maximum attempts of (number servers in backend) -1.
Your can override this:

# Enable retry sending request if network error
#
# Optional
#
[retry]

# Number of attempts
#
# Optional
# Default: (number servers in backend) -1
#
attempts = 3

# Sets the maximum request body to be stored in memory in Mo
#
# Optional
# Default: 2
#
maxMem = 3

Fixes #210

Signed-off-by: Emile Vauge <emile@vauge.com>
@vdemeester
Copy link
Contributor

@emilevauge this is general to all providers, you cannot set different values by providers. Is this intended ? 👼

@emilevauge
Copy link
Member Author

If you think I should add the possibility to override by provider, I'm OK to add it ;)

emilevauge and others added 4 commits March 4, 2016 15:55
Signed-off-by: Emile Vauge <emile@vauge.com>
Signed-off-by: Emile Vauge <emile@vauge.com>
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.

bad gateway during rolling service deployment using the Marathon backend
4 participants