Skip to content

added a way to retry backups - #11

Open
ClaytonNorthey92 wants to merge 3 commits into
mainfrom
clayton/upload-retries
Open

added a way to retry backups#11
ClaytonNorthey92 wants to merge 3 commits into
mainfrom
clayton/upload-retries

Conversation

@ClaytonNorthey92

@ClaytonNorthey92 ClaytonNorthey92 commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

added a way to retry backups

  • added flags --max-retries, --backoff-milliseconds for retrying with an exponential backoff
  • the flags apply to metadata and snapshots
  • changed the --keep-archives flag to default to true, so that a "null" value is true

fixes #8

@ClaytonNorthey92 ClaytonNorthey92 changed the title squashme draft: retry with exponential backoff Aug 14, 2025
@ClaytonNorthey92 ClaytonNorthey92 changed the title draft: retry with exponential backoff draft: added a way to retry backups Aug 27, 2025
@ClaytonNorthey92 ClaytonNorthey92 changed the title draft: added a way to retry backups added a way to retry backups Aug 27, 2025
@ClaytonNorthey92
ClaytonNorthey92 marked this pull request as ready for review August 27, 2025 19:35
Comment thread cmd/dave/backup.go Outdated
Comment thread internal/dave/dave_test.go Outdated
@ClaytonNorthey92

Copy link
Copy Markdown
Contributor Author

hey @joshuasing I believe I addressed all of your feedback, do you mind taking another look?

@AL-CT AL-CT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

* added flags --max-retries, --backoff-milliseconds for retrying with an exponential backoff
* the flags apply to metadata and snapshots
* changed the --keep-archives flag to default to true, so that a "null" value is true

fixes #8
* using a duration variable instead of an int (ms) for backoff
* removed extra newline
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.

internal/dave/repo_s3: handle upload failures

3 participants