Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: Fix postrepoactions for repos with many refs#62800

Merged
eseliger merged 1 commit into
mainfrom
es/05-20-gitserverfixpostrepoactionsforreposwithmanyrefs
May 21, 2024
Merged

gitserver: Fix postrepoactions for repos with many refs#62800
eseliger merged 1 commit into
mainfrom
es/05-20-gitserverfixpostrepoactionsforreposwithmanyrefs

Conversation

@eseliger

Copy link
Copy Markdown
Member

Running RefHash on repos with a few hundred thousand refs (nixpkgs) can run into a deadline exeeded issue, so we add a separate timeout for the postRepoFetchActions.

Test plan:

The repo clones correctly after this change.

@cla-bot cla-bot Bot added the cla-signed label May 20, 2024

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @eseliger and the rest of your teammates on Graphite Graphite

@github-actions github-actions Bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels May 20, 2024
@eseliger eseliger marked this pull request as ready for review May 21, 2024 13:18
@eseliger eseliger requested a review from a team May 21, 2024 13:18
@eseliger eseliger force-pushed the es/05-20-gitserverfixpostrepoactionsforreposwithmanyrefs branch from b521176 to 482afee Compare May 21, 2024 15:14
Running RefHash on repos with a few hundred thousand refs (nixpkgs) can run into a deadline exeeded issue, so we add a separate timeout for the `postRepoFetchActions`.

Test plan:

The repo clones correctly after this change.
@eseliger eseliger force-pushed the es/05-20-gitserverfixpostrepoactionsforreposwithmanyrefs branch from 482afee to 1d36c93 Compare May 21, 2024 17:35
@eseliger eseliger merged commit bea8317 into main May 21, 2024
@eseliger eseliger deleted the es/05-20-gitserverfixpostrepoactionsforreposwithmanyrefs branch May 21, 2024 18:03

Copy link
Copy Markdown
Member Author

Merge activity

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants