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

Throttle GitHub API calls #152

Merged
merged 7 commits into from
Jul 5, 2021
Merged

Throttle GitHub API calls #152

merged 7 commits into from
Jul 5, 2021

Conversation

EnricoMi
Copy link
Owner

@EnricoMi EnricoMi commented Jul 5, 2021

No description provided.

@EnricoMi EnricoMi force-pushed the branch-throttle-gh-api-calls branch 3 times, most recently from 3ce3d75 to e6de96f Compare July 5, 2021 10:19
@github-actions

This comment has been minimized.

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (Linux)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (macOS)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (Dockerfile)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (Docker Image)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (Windows)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the branch-throttle-gh-api-calls branch from e6de96f to bd2aee6 Compare July 5, 2021 10:25
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Replaces backoff configuration parameter with seconds between reads/writes.
@EnricoMi EnricoMi force-pushed the branch-throttle-gh-api-calls branch from aea9177 to db29e64 Compare July 5, 2021 12:42
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@EnricoMi
Copy link
Owner Author

EnricoMi commented Jul 5, 2021

This should avoid abuse rate limit errors as reported in #150 if configured with proper throttle rates.

@EnricoMi EnricoMi merged commit 4b6277f into master Jul 5, 2021
@EnricoMi EnricoMi deleted the branch-throttle-gh-api-calls branch July 5, 2021 14:06
@github-actions
Copy link

github-actions bot commented Jul 5, 2021

Unit Test Results (setup-python)

     11 files  ±0       11 suites  ±0   27s ⏱️ ±0s
   232 tests ±0     232 ✔️ ±0    0 💤 ±0  0 ❌ ±0 
2 552 runs  ±0  2 540 ✔️ ±0  12 💤 ±0  0 ❌ ±0 

Results for commit 4b6277f. ± Comparison against base commit 4b6277f.

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.

1 participant