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 retryable transport for errors #1704

Merged
merged 60 commits into from
Jan 10, 2024
Merged
Changes from 1 commit
Commits
Show all changes
60 commits
Select commit Hold shift + click to select a range
8d199e7
Add retryable transport for errors
dcfranca May 29, 2023
5ae5432
Update documentation with new parameters for retry
dcfranca May 29, 2023
df9f8d5
Change default of max_retries to 3
dcfranca May 31, 2023
e41f935
Fix typo in naming
dcfranca May 31, 2023
0da71b9
Update github/transport_test.go
kfcampbell Jun 1, 2023
0b3ba89
Merge branch 'main' into add-retries
kfcampbell Jun 1, 2023
c2c341c
Add error check for data seek
dcfranca Jun 3, 2023
e31cac0
Merge branch 'add-retries' of github.com:dcfranca/terraform-provider-…
dcfranca Jun 3, 2023
88e9ae8
Merge branch 'main' into add-retries
dcfranca Jun 6, 2023
2aa7bfd
Merge branch 'main' into add-retries
dcfranca Jun 15, 2023
bb97a82
Merge branch 'main' into add-retries
dcfranca Jun 16, 2023
61d9a17
Merge branch 'main' into add-retries
dcfranca Jun 20, 2023
27ded6c
Merge branch 'main' into add-retries
dcfranca Jun 21, 2023
218287f
Consolidate the default retriable errors on a function
dcfranca Jun 22, 2023
bd787ef
Merge branch 'add-retries' of github.com:dcfranca/terraform-provider-…
dcfranca Jun 22, 2023
9b98c08
Merge branch 'main' into add-retries
kfcampbell Jun 22, 2023
6d9b78d
Merge branch 'main' into add-retries
kfcampbell Jun 26, 2023
c2f8580
Fix typo on the comments
dcfranca Jun 27, 2023
6ad697e
Merge branch 'main' into add-retries
dcfranca Jun 27, 2023
9bb406a
Merge branch 'main' into add-retries
dcfranca Jun 28, 2023
5f93d2e
Update vendor
dcfranca Jul 4, 2023
4d51e0d
Fix merging conflicts
dcfranca Jul 11, 2023
7ad4857
Merge branch 'main' into add-retries
dcfranca Jul 12, 2023
8962435
Fix merging conflicts
dcfranca Jul 14, 2023
70a8ec2
Update documentation with new parameters for retry
dcfranca May 29, 2023
9a682ae
Change default of max_retries to 3
dcfranca May 31, 2023
1e347cc
Fix typo in naming
dcfranca May 31, 2023
0f944d2
Add error check for data seek
dcfranca Jun 3, 2023
09a1dc5
Update github/transport_test.go
kfcampbell Jun 1, 2023
e546022
Consolidate the default retriable errors on a function
dcfranca Jun 22, 2023
f3d3ce3
Fix typo on the comments
dcfranca Jun 27, 2023
5f83f00
Don't run go mod tidy on release (#1788)
kfcampbell Jul 11, 2023
f75c2de
Merge branch 'add-retries' of github.com:dcfranca/terraform-provider-…
dcfranca Jul 14, 2023
f4a3723
Update vendors
dcfranca Jul 14, 2023
9248f3d
Merge branch 'main' into add-retries
kfcampbell Jul 14, 2023
66bbc57
Merge branch 'main' into add-retries
dcfranca Jul 21, 2023
5af0b6f
Merge branch 'main' into add-retries
kfcampbell Jul 24, 2023
db8d50e
Merge branch 'main' into add-retries
dcfranca Jul 28, 2023
f402f24
Merge branch 'main' into add-retries
dcfranca Aug 3, 2023
a234ece
Merge branch 'main' into add-retries
dcfranca Aug 7, 2023
6ba0735
Merge branch 'main' into add-retries
dcfranca Aug 10, 2023
f4274eb
Merge branch 'main' into add-retries
dcfranca Aug 14, 2023
e9534c7
Merge branch 'main' into add-retries
dcfranca Aug 15, 2023
b40c2b7
Merge branch 'main' into add-retries
dcfranca Aug 21, 2023
61e196a
Merge branch 'main' into add-retries
dcfranca Aug 23, 2023
bf61de6
Merge branch 'main' into add-retries
dcfranca Aug 29, 2023
7de4a84
Merge branch 'main' into add-retries
dcfranca Sep 1, 2023
679e033
Merge branch 'main' into add-retries
dcfranca Sep 11, 2023
564ed52
Merge branch 'main' into add-retries
dcfranca Sep 13, 2023
8465c92
Merge branch 'main' into add-retries
dcfranca Sep 19, 2023
3221167
Merge branch 'main' into add-retries
dcfranca Sep 25, 2023
c1b5e49
Merge branch 'main' into add-retries
dcfranca Sep 27, 2023
ef547ad
Merge branch 'main' into add-retries
dcfranca Oct 3, 2023
4d63684
Merge branch 'main' into add-retries
dcfranca Oct 9, 2023
053fd0b
Merge branch 'main' into add-retries
dcfranca Oct 12, 2023
fff38a1
Merge branch 'main' into add-retries
dcfranca Oct 14, 2023
4f27468
Merge branch 'main' into add-retries
nickfloyd Oct 16, 2023
23b3302
Merge branch 'main' into add-retries
dcfranca Nov 8, 2023
a2c18d9
Merge branch 'main' into add-retries
kfcampbell Jan 10, 2024
23634d2
Fix lint with APIMeta deprecation
kfcampbell Jan 10, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update documentation with new parameters for retry
dcfranca committed Jul 14, 2023

Verified

This commit was signed with the committer’s verified signature.
lemeurherve Hervé Le Meur
commit 70a8ec27772d41f0c7e6e29f6e0391dc056d8f19
6 changes: 6 additions & 0 deletions website/docs/index.html.markdown
Original file line number Diff line number Diff line change
@@ -111,8 +111,14 @@ The following arguments are supported in the `provider` block:

* `write_delay_ms` - (Optional) The number of milliseconds to sleep in between write operations in order to satisfy the GitHub API rate limits. Defaults to 1000ms or 1 second if not provided.

* `retry_delay_ms` - (Optional) Amount of time in milliseconds to sleep in between requests to GitHub API after an error response. Defaults to 1000ms or 1 second if not provided, the max_retries must be set to greater than zero.

* `read_delay_ms` - (Optional) The number of milliseconds to sleep in between non-write operations in order to satisfy the GitHub API rate limits. Defaults to 0ms.

* `retryable_errors` - (Optional) "Allow the provider to retry after receiving an error status code, the max_retries should be set for this to work. Defaults to [500, 502, 503, 504]

* `max_retries` - (Optional) Number of times to retry a request after receiving an error status code. Defaults to 0

Note: If you have a PEM file on disk, you can pass it in via `pem_file = file("path/to/file.pem")`.

For backwards compatibility, if more than one of `owner`, `organization`,