Skip to content

feat: repo-based allowlist for git enumeration flags - #5797

Merged
Ly-Joey merged 8 commits into
google:masterfrom
Ly-Joey:feat-repo-based-cab
Aug 11, 2026
Merged

feat: repo-based allowlist for git enumeration flags#5797
Ly-Joey merged 8 commits into
google:masterfrom
Ly-Joey:feat-repo-based-cab

Conversation

@Ly-Joey

@Ly-Joey Ly-Joey commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Add repo-based git analysis flags logic in worker before we send the request to gitter to enumerate affected commits.

List of configurable bools:

  • consider_all_branches
  • detect_cherrypicks_intro
  • detect_cherrypicks_fixed
  • detect_cherrypicks_limit

The allowlist is in datastore.
#5800 is the tool to sync the allowlist (in yaml) into datastore entities.

@Ly-Joey
Ly-Joey marked this pull request as ready for review August 6, 2026 01:46
Comment thread go/internal/database/datastore/repo_cab.go Outdated
Comment thread go/internal/database/datastore/repo_cab.go Outdated
Comment thread go/internal/worker/affectedcommits.go Outdated
@Ly-Joey
Ly-Joey force-pushed the feat-repo-based-cab branch from 5f30f10 to 6e3fafa Compare August 7, 2026 01:55
@Ly-Joey Ly-Joey changed the title feat: repo-based consider all branches feat: repo-based allowlist for git enumeration flags Aug 7, 2026
Comment thread go/internal/database/datastore/models.go Outdated
Comment thread go/internal/worker/affectedcommits.go Outdated
Comment thread go/internal/database/datastore/repo_allowlist.go Outdated
Comment thread go/internal/database/datastore/repo_allowlist.go Outdated

@another-rex another-rex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Ly-Joey
Ly-Joey merged commit c516b15 into google:master Aug 11, 2026
26 checks passed
Ly-Joey added a commit that referenced this pull request Aug 12, 2026
Add a tool to sync repo-based git analysis flags allowlist entries from
yaml to datastore.

Related: #5797
(Stack PR makes me sad)
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.

3 participants