perf(ci): bound parallel regression execution by devantler · Pull Request #5558 · devantler-tech/platform

devantler · 2026-10-29T06:26:17Z

🤖 Generated by the Agentic Engineer

Why

Slow independent validation delays platform delivery and recovery.

What

Use bounded concurrency for existing regression checks and run independent validation in two isolated groups, preserving every check and deadline. A matched local comparison improved the slowest suite from 407s to 245s; hosted feedback and total delivery time remain to be verified.

Part of #5495

Part of #5495. Preserve the complete suite, dependency startup controls and package deadline while allowing eight already parallel test cases per package.

# Conflicts: # .github/workflows/ci.yaml

devantler · 2026-10-29T10:49:38Z

🤖 Generated by the Agentic Engineer

@codex review

chatgpt-codex-connector · 2026-10-29T10:50:01Z

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-10-29T10:50:52.792268Z	`8418cdc`	Manual request

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

chatgpt-codex-connector · 2026-10-29T10:50:50Z

Codex Review: Didn't find any major issues. Chef's kiss.

Reviewed commit: 8418cdcfd0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

devantler · 2026-10-29T12:54:16Z

🤖 Generated by the Agentic Engineer

Blocker: none

Cleared 2026-10-29: Runner repair #5559 merged at 0fd031e; the merge-group and actual merged-source browser runs, Validate Main, CodeQL and Code Quality all passed. Resume on that reviewed base and renew exact-head CI, independent review and readiness before protected queue entry.

Previous record (historical):

> 🤖 Generated by the Agentic Engineer

<!-- pr-blocker-record -->

**Blocker:** devantler-tech/platform#5559 | upstream | last-verified 2026-10-29: Merge-group 40875591717 at be94f92c01 failed browser installation: Playwright 1.56.1 does not support Ubuntu 26.04. The same workflow blob is on main and both queued branches. Removed before the production job started; require #5559 merged and fresh successful current-base CI before requeue.

devantler · 2026-10-29T15:24:49Z

🤖 Generated by the Agentic Engineer

@coderabbitai full review

coderabbitai · 2026-10-29T15:24:57Z

✅ Action performed

Full review finished.

coderabbitai · 2026-10-29T15:33:17Z

No actionable comments were generated in the recent review.

Recent review info

Run configuration

Source mode: Auto
Repository: devantler-tech/platform
Branch: codex/ci-auth-fixture-timing-5495
File: .coderabbit.yaml
Commit: dce945e
Layers: Repository YAML (base), Organization UI (inherited)
Review profile: ASSERTIVE
Plan: Advanced
Run ID: 6852de3a-0829-464a-a8c8-5691905e5ba7

Commits

Reviewing files that changed from the base of the PR and between 0fd031e and dce945e.

Files selected for processing: 7

.github/workflows/ci.yaml
AGENTS.md
docs/adr/ci-regression-partitions.md
scripts/guard-arc-recovery/ci_dependency_test.go
scripts/tests/test-policy-ci-job.sh
scripts/tests/test-private-mail-configuration.sh
scripts/validate-merge-group-heal/go_regressions_test.go

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Recent review details

Context from checks skipped due to timeout: 1

GitHub Check: CodeRabbit

Additional comments: 5

scripts/tests/test-private-mail-configuration.sh: 1

101-107: LGTM!

.github/workflows/ci.yaml: 2

108-110: LGTM!

Also applies to: 112-112

3385-3385: LGTM!

Also applies to: 3389-3395, 3417-3417, 3448-3448, 3454-3454, 3460-3460, 3468-3468, 3471-3471, 3477-3477, 3484-3484, 3491-3491, 3500-3500, 3512-3512

scripts/guard-arc-recovery/ci_dependency_test.go: 1

33-33: LGTM!

scripts/validate-merge-group-heal/go_regressions_test.go: 1

66-67: LGTM!

Also applies to: 69-69

Walkthrough

The full Go regression command now uses -parallel 8 and retains its 20-minute timeout. The manifest regression job now runs contracts and native matrix suites, with up to two jobs in parallel. Regression routing checks and documentation cover eleven checks and the matrix settings. Bootstrap image verification now checks additional assets and reports distinct failure markers.

Possibly related PRs

devantler-tech/platform#5366: Introduced the manifest-regression job and seven-regression routing contract, which this PR expands into two suites and eleven checks.
devantler-tech/platform#5313: Added the timeout command and guards that this PR updates to require -parallel 8.
devantler-tech/platform#5565: Also changes the manifest-regression workflow and bounded-concurrency checks.

Priority: Low

Merge Risk: Minimal · up to dce94

The regressions remain selected and are split across two runners. No actionable merge blocker was identified.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title "perf(ci): bound parallel regression execution" matches the main change. The diff caps parallelism for the Go regression suite with -parallel 8 and splits manifest regression validation into…
Description check		The description states the purpose (slow independent validation delays delivery and recovery) and the change (bounded concurrency for existing regression checks, run in two isolated groups, with every…
Docstring Coverage		Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title "perf(ci): bound parallel regression execution" matches the main change. The diff caps parallelism for the Go regression suite with -parallel 8 and splits manifest regression validation into a two-suite matrix (contracts, native) with max-parallel of two. The title is short, specific, and follows Conventional Commits.

Full details: Description check

Explanation

The description states the purpose (slow independent validation delays delivery and recovery) and the change (bounded concurrency for existing regression checks, run in two isolated groups, with every check and deadline preserved). It also gives a local timing comparison and notes that hosted feedback is unverified. This relates directly to the changeset, so the check passes.

Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 2 functions across 4 files. (3 skipped: 3 unsupported.)

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

devantler · 2026-10-29T16:27:54Z

🤖 Generated by the Agentic Engineer

Readiness at dce945e71289ae9fc56e46312e6e2695ccba6642: the complete current-head CI exercised the changed regression partitioning and passed; the reported checks are settled at 45 success/9 skipped. The current-head CodeRabbit review is finding-free and the canonical verdict helper returns GREEN. Fresh native reads show zero review threads, MERGEABLE/CLEAN, no outstanding maintainer instruction and this session's own worktree holder.

As the pipeline's user, I observed the complete hosted workflow and retained regression execution, plus the earlier 582-case/race validation of the unchanged authorization test inputs. The measured 407s-to-245s package result is scoped to that package; repository guards were the longest current CI job at 663s. Overall pipeline acceptance remains open on #5495.

Live main is now 3fe270060fe86bae178f29eda1e3e4a061202b5f: its actual merged-source deployment passed all 17 public checks, post-merge Validate Main/CodeQL/Code Quality passed, and live health is OK. A conflict-free merge preview retains the new main mail-recovery selection. Actionlint passes with the repository's reviewed configuration and its separate concurrency enum validator also passes; the standalone temporary-file lint lacked that configuration. The protected merge queue will validate the full combination again.

perf(ci): bound parallel regression execution

b464a03

Part of #5495. Preserve the complete suite, dependency startup controls and package deadline while allowing eight already parallel test cases per package.

github-project-automation Bot added this to 🌊 Project Board Oct 29, 2026

github-project-automation Bot moved this to 🫴 Ready in 🌊 Project Board Oct 29, 2026

devantler added 2 commits October 29, 2026 07:35

fix(ci): preserve Kubescape placement validation with jq 1.8

59b6e2d

test(ci): prove publication worker limits deterministically

4bd7ee3

devantler mentioned this pull request Oct 29, 2026

Reduce CI critical-path time while preserving validation #5495

Open

devantler added 11 commits October 29, 2026 08:12

test(ci): identify image asset proof failure stages

1bcb6d4

fix(mail): restore proxy license directory traversal

277f3c5

fix(ci): integrate publication probe repair for validation

c92f018

fix(ci): integrate jq placement repair for validation

a55c8c5

test(coroot): retain bounded fixture failure evidence

43dc224

test(ci): expose Coroot request lifetimes before native proofs

f0fc80b

fix(ci): observe Coroot query capacity with one owner

65fc418

fix(test): retain Coroot query failure evidence

f806dc5

chore: integrate validated platform CI recovery

e16f789

perf(ci): partition independent manifest validation

f1f6c87

chore: integrate protected platform CI recovery

8418cdc

# Conflicts: # .github/workflows/ci.yaml

devantler marked this pull request as ready for review October 29, 2026 10:59

devantler added this pull request to the merge queue Oct 29, 2026

devantler removed this pull request from the merge queue due to a manual request Oct 29, 2026

devantler added the blocked label Oct 29, 2026

devantler marked this pull request as draft October 29, 2026 14:58

devantler removed the blocked label Oct 29, 2026

chore(ci): integrate reviewed platform main

dce945e

devantler marked this pull request as ready for review October 29, 2026 16:28

devantler added this pull request to the merge queue Oct 29, 2026

Merged via the queue into main with commit 608f331 Oct 29, 2026
54 checks passed

devantler deleted the codex/ci-auth-fixture-timing-5495 branch October 29, 2026 19:28

github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board Oct 29, 2026

Navigation Menu

Repository navigation

perf(ci): bound parallel regression execution - #5558

perf(ci): bound parallel regression execution#5558
devantler merged 15 commits into
mainfrom
codex/ci-auth-fixture-timing-5495

devantler commented Oct 29, 2026 •

edited

Loading

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

chatgpt-codex-connector Bot commented Oct 29, 2026 •

edited

Loading

Uh oh!

chatgpt-codex-connector Bot commented Oct 29, 2026

Uh oh!

Uh oh!

devantler commented Oct 29, 2026 •

edited

Loading

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

coderabbitai Bot commented Oct 29, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Oct 29, 2026

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

devantler commented Oct 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Why

What

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

chatgpt-codex-connector Bot commented Oct 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codex Review Summary

Uh oh!

chatgpt-codex-connector Bot commented Oct 29, 2026

Uh oh!

Uh oh!

devantler commented Oct 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

coderabbitai Bot commented Oct 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coderabbitai Bot commented Oct 29, 2026

Uh oh!

devantler commented Oct 29, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

devantler commented Oct 29, 2026 •

edited

Loading

chatgpt-codex-connector Bot commented Oct 29, 2026 •

edited

Loading

devantler commented Oct 29, 2026 •

edited

Loading

coderabbitai Bot commented Oct 29, 2026 •

edited

Loading