Skip to content

feat(gitter): git commit graph and patch ID caching, affected commits walking #4728

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

Draft
Ly-Joey wants to merge 20 commits into
base: master
Choose a base branch
from
+1,624 −58
Draft

feat(gitter): git commit graph and patch ID caching, affected commits walking #4728

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
06338be
Add gitter /cache endpoint
Ly-Joey Feb 2, 2026
399aa4c
Fix affected commit graph traversal
Ly-Joey Feb 2, 2026
0eae85a
Fix typo persistance->persistence and better names
Ly-Joey Feb 2, 2026
c04f039
Parallelize patch ID process and minor refactoring
Ly-Joey Feb 2, 2026
2905015
Better logging and names
Ly-Joey Feb 2, 2026
6f4f7b6
Lint and comments mostly
Ly-Joey Feb 3, 2026
12b6e20
Use new git log command with stdout pipe instead
Ly-Joey Feb 3, 2026
9cb85c4
Split fetch and archive of repo, better logs
Ly-Joey Feb 4, 2026
ef7e2b9
Change git log to redirect to file
Ly-Joey Feb 5, 2026
4627629
Enumerate commits w/ cherrypick + Singleflight
Ly-Joey Feb 6, 2026
294400d
Remove singleflight for affected commits
Ly-Joey Feb 6, 2026
637dc75
Add limit logic for affected commits
Ly-Joey Feb 9, 2026
9c00da1
Fix patchID worker, use pointer for commit details
Ly-Joey Feb 9, 2026
6a6b3a2
Change gitter flags to kebab-case
Ly-Joey Feb 9, 2026
3d02b47
Add repo test cases, fix logic issues
Ly-Joey Feb 9, 2026
e71b763
Add more gitter test cases. Rename endpoints
Ly-Joey Feb 9, 2026
92919ad
Use forceUpdate flag in other fetches as well
Ly-Joey Feb 9, 2026
5f6a11e
Lint
Ly-Joey Feb 9, 2026
838e869
Fix some lint
Ly-Joey Feb 10, 2026
bc0e4b4
I think that's the final lints
Ly-Joey Feb 10, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions deployment/clouddeploy/gke-workers/base/gitter.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ spec:
imagePullPolicy: Always
args:
- "--port=8888"
- "--work_dir=/work/gitter"
- "--fetch_timeout=1h"
- "--work-dir=/work/gitter"
- "--fetch-timeout=1h"
volumeMounts:
- mountPath: /work
name: disk-data
Expand Down
4 changes: 2 additions & 2 deletions deployment/clouddeploy/gke-workers/environments/oss-vdb-test/gitter.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ spec:
- name: gitter
args:
- "--port=8888"
- "--work_dir=/work/gitter"
- "--fetch_timeout=1h"
- "--work-dir=/work/gitter"
- "--fetch-timeout=1h"
env:
- name: GOOGLE_CLOUD_PROJECT
value: oss-vdb-test
4 changes: 2 additions & 2 deletions deployment/clouddeploy/gke-workers/environments/oss-vdb/gitter.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ spec:
- name: gitter
args:
- "--port=8888"
- "--work_dir=/work/gitter"
- "--fetch_timeout=1h"
- "--work-dir=/work/gitter"
- "--fetch-timeout=1h"
env:
- name: GOOGLE_CLOUD_PROJECT
value: oss-vdb
Loading
Loading