Skip to content

Add configurable timeouts for get/put to remote storage - #47

Merged
xakep666 merged 2 commits into
mainfrom
remote-get-put-timeouts
Jul 23, 2026
Merged

Add configurable timeouts for get/put to remote storage#47
xakep666 merged 2 commits into
mainfrom
remote-get-put-timeouts

Conversation

@xakep666

Copy link
Copy Markdown
Collaborator

Description

Added new flags/envs CACHEPROG_REMOTE_GET_TIMEOUT and CACHEPROG_REMOTE_PUT_TIMEOUT to restrict amount of time spent in "Get" and "Put" to remote storage because sometimes remote storages are failing without returning any errors but simply didn't responding to requests. Circuit breaker will also trip if too many timeouts were registered.

TODOs

Read the Contribution guidelines.

  • Generate the docs.
  • Run the relevant tests successfully.
  • Include release notes. If this PR is backward incompatible, include a migration guide.
  • Update the CHANGELOG.md file: add a feature / bug fix entry for this PR under Unreleased section.

Release Notes

  • Add configurable timeouts for get and put operations to remote storage
  • Upgrade dependencies

@xakep666
xakep666 force-pushed the remote-get-put-timeouts branch from bde8077 to 35d7e08 Compare July 23, 2026 07:46
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

go-test-coverage report:

Total test coverage: 78.0% (921/1181)

@xakep666
xakep666 force-pushed the remote-get-put-timeouts branch from 0612e20 to 5e9b6af Compare July 23, 2026 07:59
@xakep666
xakep666 force-pushed the remote-get-put-timeouts branch 3 times, most recently from 1eddc34 to 36d6781 Compare July 23, 2026 08:45
@xakep666
xakep666 force-pushed the remote-get-put-timeouts branch 2 times, most recently from 37d4262 to 1eddc34 Compare July 23, 2026 08:54
@xakep666
xakep666 merged commit 9264c95 into main Jul 23, 2026
4 checks passed
@xakep666
xakep666 deleted the remote-get-put-timeouts branch July 23, 2026 11:29
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.

2 participants