ewilliams28/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-08-28 05:16:23 -06:00
Code Issues Packages Projects Releases Wiki Activity

improve(ci): cut hybrid compact runner overhead (#124622)

Browse Source 
* perf(ci): refit compact planner hints for Blacksmith

* test(ci): type default runner backend fixture
This commit is contained in:
Peter Steinberger
2026-08-16 07:12:32 -07:00
committed by GitHub
parent 1b34939f82
commit 3e2edc7f22
4 changed files with 166 additions and 88 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/scripts/ci-node-test-plan.test.ts
+75 -13
View File
@@ -313,20 +313,82 @@ describe("scripts/lib/ci-node-test-plan.mts", () => {
"core-tooling-isolated",
]);
// Pushes retain three lanes of headroom under the workflow's 28-worker cap.
expect(compact).toHaveLength(25);
expect(pullRequestCompact).toHaveLength(34);
expect(githubCompact).toHaveLength(70);
expect(githubPullRequestCompact).toHaveLength(79);
expect(hybridCompact).toEqual(githubCompact);
expect(hybridPullRequestCompact).toEqual(githubPullRequestCompact);
for (const profile of [
{
name: "Blacksmith",
pullRequest: pullRequestCompact,
pullRequestJobs: 34,
pullRequestMax: 204,
push: compact,
pushJobs: 25,
pushMax: 204,
},
{
name: "GitHub-hosted",
pullRequest: githubPullRequestCompact,
pullRequestJobs: 79,
pullRequestMax: 186,
push: githubCompact,
pushJobs: 70,
pushMax: 149,
},
{
name: "hybrid",
pullRequest: hybridPullRequestCompact,
pullRequestJobs: 55,
pullRequestMax: 140,
push: hybridCompact,
pushJobs: 47,
pushMax: 140,
},
]) {
expect(profile.push, `${profile.name} push jobs`).toHaveLength(profile.pushJobs);
expect(profile.pullRequest, `${profile.name} pull-request jobs`).toHaveLength(
profile.pullRequestJobs,
);
expect(
Math.max(...profile.push.map((shard) => shard.predictedSeconds ?? Infinity)),
`${profile.name} push max`,
).toBe(profile.pushMax);
expect(
Math.max(...profile.pullRequest.map((shard) => shard.predictedSeconds ?? Infinity)),
`${profile.name} pull-request max`,
).toBe(profile.pullRequestMax);
}
expect(hybridCompact.filter((shard) => !shard.requiresDist)).toHaveLength(46);
expect(githubCompact.length - hybridCompact.length).toBeGreaterThanOrEqual(20);
expect(githubPullRequestCompact.length).toBeLessThanOrEqual(96);
// Nondist hosted lanes stay under the 150-second body ceiling; the serial
// TUI PTY dist descriptor keeps its indivisible measured wall.
expect(Math.max(...githubCompact.map((shard) => shard.predictedSeconds ?? Infinity))).toBe(149);
expect(
Math.max(...githubPullRequestCompact.map((shard) => shard.predictedSeconds ?? Infinity)),
).toBe(186);
// Nondist expanded-profile lanes stay under the 150-second body ceiling;
// the hosted PR's serial TUI PTY descriptor remains indivisible.
for (const plan of [
githubCompact,
githubPullRequestCompact,
hybridCompact,
hybridPullRequestCompact,
]) {
expect(
plan
.filter((shard) => !shard.requiresDist)
.every((shard) => (shard.predictedSeconds ?? Infinity) <= 150),
).toBe(true);
}
// Historical checks-node-compact-large-2 was this gateway-core group. Its
// 139.5s Blacksmith spike keeps a dedicated floor and singleton bin even
// though compact check numbers change when the matrix shrinks.
const hybridLargeTail = hybridCompact.find((shard) =>
shard.groups.some((group) => group.shard_name === "agentic-gateway-core-3"),
);
expect(hybridLargeTail?.groups.map((group) => group.shard_name)).toEqual([
"agentic-gateway-core-3",
]);
expect(hybridLargeTail?.predictedSeconds).toBe(140);
const hybridJobOf = (name: string) =>
hybridCompact.findIndex((shard) => shard.groups.some((group) => group.shard_name === name));
// Synthesized hosted stripes retain divided admission weights, while
// native hybrid groups use Blacksmith stripe hints during rebalance.
expect(hybridJobOf("agentic-agents-core-runtime-hosted-1")).not.toBe(
hybridJobOf("agentic-agents-core-tools"),
);
expect(compact.every((shard) => Array.isArray(shard.groups))).toBe(true);
expect(compact.every((shard) => shard.groups.length <= 10)).toBe(true);
expect(compact.some((shard) => shard.requiresDist)).toBe(true);
test/scripts/ci-workflow-guards.test.ts
+24 -44
View File
@@ -5920,50 +5920,30 @@ printf '%s\n' "\${CURL_SUCCESS_IP:-203.0.113.7}"
}),
);
const push = runCiManifestFixture({
bundledPlanner: true,
eventName: "push",
});
expect(push.status, push.output).toBe(0);
expect(
JSON.parse(
expectDefined(
push.outputs.checks_node_core_nondist_matrix,
"push node core nondist matrix output",
),
).include,
).toContainEqual(
expect.objectContaining({
check_name: "bundled-node-plan",
env: {
OPENCLAW_CI_TEST_COMPACT_MODE: "push",
OPENCLAW_CI_TEST_RUNNER_BACKEND: "",
},
}),
);
const githubPush = runCiManifestFixture({
bundledPlanner: true,
eventName: "push",
runnerBackend: "github",
});
expect(githubPush.status, githubPush.output).toBe(0);
expect(
JSON.parse(
expectDefined(
githubPush.outputs.checks_node_core_nondist_matrix,
"GitHub-hosted push node core nondist matrix output",
),
).include,
).toContainEqual(
expect.objectContaining({
check_name: "bundled-node-plan",
env: {
OPENCLAW_CI_TEST_COMPACT_MODE: "push",
OPENCLAW_CI_TEST_RUNNER_BACKEND: "github",
},
}),
);
for (const runnerBackend of [undefined, "github", "hybrid"] as const) {
const push = runCiManifestFixture({
bundledPlanner: true,
eventName: "push",
runnerBackend,
});
expect(push.status, push.output).toBe(0);
expect(
JSON.parse(
expectDefined(
push.outputs.checks_node_core_nondist_matrix,
`${runnerBackend ?? "default"} push node core nondist matrix output`,
),
).include,
).toContainEqual(
expect.objectContaining({
check_name: "bundled-node-plan",
env: {
OPENCLAW_CI_TEST_COMPACT_MODE: "push",
OPENCLAW_CI_TEST_RUNNER_BACKEND: runnerBackend ?? "",
},
}),
);
}
const changedPullRequest = runCiManifestFixture({
bundledPlanner: true,