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

Configurable wait time for GitHub query retries #74

Merged
merged 2 commits into from
Sep 7, 2023

Conversation

LRWeber
Copy link
Member

@LRWeber LRWeber commented Sep 6, 2023

Further changes for optimization may be made in the future (e.g. removal of real-time countdown printout).

@LRWeber LRWeber requested a review from IanLee1521 September 6, 2023 19:48
@LRWeber LRWeber self-assigned this Sep 6, 2023
@LRWeber
Copy link
Member Author

LRWeber commented Sep 6, 2023

@IanLee1521 This is the primary cause for LLNL/llnl.github.io updates failing due to exceeding GitHub's allowed runtime, so fast-tracking a fix and release would be appreciated!

(As a side note, it looks like all versions past v0.9.0 have been tagged, but not formally added as a "release" on GitHub.)

@LRWeber LRWeber changed the title Reduced wait time for GitHub query retries Configurable wait time for GitHub query retries Sep 7, 2023
scraper/github/queryManager.py Show resolved Hide resolved
@IanLee1521 IanLee1521 merged commit 95d721b into LLNL:main Sep 7, 2023
@LRWeber LRWeber deleted the github-fix branch September 7, 2023 17:35
@IanLee1521
Copy link
Member

As a side note, it looks like all versions past v0.9.0 have been tagged, but not formally added as a "release" on GitHub.

That's likely due to the type of tag being made... I can take a look at that.

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