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

Reduce amount of calls to git config #62649

Closed
eseliger opened this issue May 14, 2024 · 1 comment · Fixed by #62776
Closed

Reduce amount of calls to git config #62649

eseliger opened this issue May 14, 2024 · 1 comment · Fixed by #62776
Assignees
Labels
perf team/source Tickets under the purview of Source - the one Source to graph it all

Comments

@eseliger
Copy link
Member

We have a lot of mechanisms revolving around git config, to store sourcegraph internal state.

This is by far the most frequently run exec call on sourcegraph.com so we should investigate how we can cause less load from spawning all these millions of processes in a 3h window.

https://ui.honeycomb.io/sourcegraph/board/DrqVyyy4nWq/Gitserver-DotCom?endTime=0&startTime=-1800

@eseliger eseliger added perf team/source Tickets under the purview of Source - the one Source to graph it all labels May 14, 2024
@eseliger
Copy link
Member Author

Made some progress here: #62615

The number of processes spawned for git config went from ~110 to ~96 on S2, and from 1150 to 750 on dotcom.

Still lots of room for improvement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
perf team/source Tickets under the purview of Source - the one Source to graph it all
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant