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.

Sign up for GitHub

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

Jump to bottom

Cache Octokit for better workflow performance #9250

Closed
jimchamp opened this issue May 9, 2024 · 0 comments · Fixed by #9330
Closed

Cache Octokit for better workflow performance #9250

jimchamp opened this issue May 9, 2024 · 0 comments · Fixed by #9330
Assignees
@jimchamp
Labels
Affects: Developers Affects: Operations Affects the IA DevOps folks Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]
Milestone
Sprint 2024-05

Comments

@jimchamp
Copy link
Collaborator

jimchamp commented May 9, 2024

Describe the problem that you'd like solved

Our two GitHub workflows that use Octokit take a very long time to run. Much of this time is spent installing the Octokit action.

Proposal & Constraints

We can use the cache action to avoid having to download Octokit every time these workflows run. We already do something similar today with our javascript_tests workflow.

Additional context

Workflows to update:
https://github.com/internetarchive/openlibrary/blob/master/.github/workflows/new_pr_labeler.yml
https://github.com/internetarchive/openlibrary/blob/master/.github/workflows/stale_assignee_digest.yml

javascript_tests workflow:
https://github.com/internetarchive/openlibrary/blob/master/.github/workflows/javascript_tests.yml

Stakeholders
@jimchamp jimchamp added Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed] Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels May 9, 2024
@mekarpeles mekarpeles added Priority: 3 Issues that we can consider at our leisure. [managed] Affects: Operations Affects the IA DevOps folks Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Affects: Developers and removed Needs: Triage This issue needs triage. The team needs to decide who should own it, what to do, by when. [managed] Needs: Lead labels May 13, 2024
@mekarpeles mekarpeles added this to the Sprint 2024-05 milestone May 13, 2024
@jimchamp jimchamp self-assigned this May 13, 2024
@jimchamp jimchamp mentioned this issue May 13, 2024
Closed
This was referenced May 22, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jimchamp jimchamp

Labels
Affects: Developers Affects: Operations Affects the IA DevOps folks Lead: @jimchamp Issues overseen by Jim (Front-end Lead, BookNotes) [managed] Priority: 3 Issues that we can consider at our leisure. [managed] Type: Feature Request Issue describes a feature or enhancement we'd like to implement. [managed]
Projects
None yet
Milestone
Sprint 2024-05
Development

Successfully merging a pull request may close this issue.

Cache octokit in GitHub workflows jimchamp/openlibrary
2 participants
@mekarpeles @jimchamp