Skip to content

Serialize gqlgen codegen to fix build cache race#1076

Merged
SachaProbo merged 1 commit intomainfrom
SachaProbo/fix-e2e-test
Apr 20, 2026
Merged

Serialize gqlgen codegen to fix build cache race#1076
SachaProbo merged 1 commit intomainfrom
SachaProbo/fix-e2e-test

Conversation

@SachaProbo
Copy link
Copy Markdown
Contributor

@SachaProbo SachaProbo commented Apr 20, 2026

Parallel gqlgen instances race on the Go build cache, causing Rewriter.getSource() to panic with "slice bounds out of range" when go/packages returns empty source. Chain gqlgen targets with order-only prerequisites in the Makefile and switch the CI build job to sequential generation.


Summary by cubic

Serialize gqlgen code generation to stop Go build cache races that caused Rewriter.getSource() panics from go/packages. CI and local builds now generate schemas sequentially for stability.

  • Bug Fixes
    • Makefile: chain codegen targets with order-only prerequisites to enforce connectconsoletrust.
    • CI: replace parallel go generate with sequential runs for connect, console, trust, and mcp.

Written for commit f7070c4. Summary will update on new commits.

@SachaProbo SachaProbo marked this pull request as ready for review April 20, 2026 14:33
@SachaProbo SachaProbo requested a review from a team April 20, 2026 14:33
Parallel gqlgen instances race on the Go build cache, causing
Rewriter.getSource() to panic with "slice bounds out of range"
when go/packages returns empty source. Chain gqlgen targets with
order-only prerequisites in the Makefile and switch the CI build
job to sequential generation.

Signed-off-by: Sacha Al Himdani <sacha@getprobo.com>
@SachaProbo SachaProbo force-pushed the SachaProbo/fix-e2e-test branch from d48d61c to f7070c4 Compare April 20, 2026 14:34
@SachaProbo SachaProbo merged commit f7070c4 into main Apr 20, 2026
15 checks passed
@SachaProbo SachaProbo deleted the SachaProbo/fix-e2e-test branch April 20, 2026 14:42
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.

2 participants