Add environment variable configuration for Excon timeouts and retry limit#14906
Open
yeikel wants to merge 1 commit intodependabot:mainfrom
Open
Add environment variable configuration for Excon timeouts and retry limit#14906yeikel wants to merge 1 commit intodependabot:mainfrom
yeikel wants to merge 1 commit intodependabot:mainfrom
Conversation
Contributor
Author
|
@kbukum1 Please review if you get a chance. Thanks in advance |
…imit Allow operators to override Excon default timeout values and retry limits using environment variables. This provides flexibility for environments with slower networks or registries that require longer timeout periods. Supported environment variables: - DEPENDABOT_EXCON_CONNECT_TIMEOUT_IN_SECONDS (default: 5) - DEPENDABOT_EXCON_WRITE_TIMEOUT_IN_SECONDS (default: 5) - DEPENDABOT_EXCON_READ_TIMEOUT_IN_SECONDS (default: 20) - DEPENDABOT_EXCON_RETRY_LIMIT (default: 4) Follows the pattern from PR dependabot#10359 for Docker timeout configuration.
8a29894 to
52923e5
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What are you trying to accomplish?
Allow to override Excon default timeout values and retry limits using environment variables. This provides flexibility for environments with slower networks or registries that require longer timeout periods.
Supported environment variables:
DEPENDABOT_EXCON_CONNECT_TIMEOUT_IN_SECONDS(default: 5)DEPENDABOT_EXCON_WRITE_TIMEOUT_IN_SECONDS(default: 5)DEPENDABOT_EXCON_READ_TIMEOUT_IN_SECONDS(default: 20)DEPENDABOT_EXCON_RETRY_LIMIT(default: 4)Anything you want to highlight for special attention from reviewers?
This is particularly useful when working on private, slower networks
Follows the pattern from PR #10359 for Docker timeout configuration.
How will you know you've accomplished your goal?
The new spec pass; it is now possible to overwrite the excon configuration using environment variables
Checklist