Skip to content

Commit 6bffda4

Browse files
RomneyDaRomneyDa
authored
ci: temporarily disable QA smoke (#101185)
1 parent 19491fe commit 6bffda4

3 files changed

Lines changed: 14 additions & 75 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -315,12 +315,6 @@ jobs:
315315
if (runNodeFull) {
316316
checksFastCoreTasks.push(
317317
{ check_name: "checks-fast-bundled-protocol", runtime: "node", task: "bundled-protocol" },
318-
{
319-
check_name: "QA Smoke CI",
320-
runtime: "node",
321-
task: "qa-smoke-ci",
322-
runner: "blacksmith-16vcpu-ubuntu-2404",
323-
},
324318
{ check_name: "checks-fast-bun-launcher", runtime: "bun", task: "bun-launcher" },
325319
);
326320
} else {
@@ -906,31 +900,6 @@ jobs:
906900
pnpm test:bundled
907901
pnpm protocol:check
908902
;;
909-
qa-smoke-ci)
910-
output_dir=".artifacts/qa-e2e/smoke-ci-profile"
911-
package_dir=".artifacts/qa-e2e/smoke-ci-package"
912-
export OPENCLAW_BUILD_PRIVATE_QA=1
913-
export OPENCLAW_ENABLE_PRIVATE_QA_CLI=1
914-
export OPENCLAW_DISABLE_BUNDLED_PLUGINS=0
915-
export OPENCLAW_QA_REDACT_PUBLIC_METADATA=1
916-
export OPENCLAW_QA_TRANSPORT_READY_TIMEOUT_MS=180000
917-
export NODE_OPTIONS=--max-old-space-size=16384
918-
node scripts/package-openclaw-for-docker.mjs \
919-
--output-dir "$package_dir" \
920-
--output-name openclaw-current.tgz
921-
export OPENCLAW_CURRENT_PACKAGE_TGZ="$PWD/$package_dir/openclaw-current.tgz"
922-
qa_exit_code=0
923-
pnpm openclaw qa run \
924-
--repo-root . \
925-
--qa-profile smoke-ci \
926-
--concurrency 8 \
927-
--output-dir "$output_dir" || qa_exit_code=$?
928-
echo "QA smoke profile evidence: \`${output_dir}\`" >> "$GITHUB_STEP_SUMMARY"
929-
if [ "$qa_exit_code" -ne 0 ]; then
930-
echo "::error title=QA smoke profile failed::smoke-ci exited ${qa_exit_code}; evidence upload will still run"
931-
exit "$qa_exit_code"
932-
fi
933-
;;
934903
contracts-plugins-ci-routing)
935904
pnpm test:contracts:plugins
936905
pnpm test src/commands/status.scan-result.test.ts src/scripts/ci-changed-scope.test.ts test/scripts/changed-lanes.test.ts test/scripts/ci-workflow-guards.test.ts test/scripts/run-vitest.test.ts test/scripts/test-projects.test.ts
@@ -947,15 +916,6 @@ jobs:
947916
;;
948917
esac
949918
950-
- name: Upload QA smoke profile evidence
951-
if: always() && matrix.task == 'qa-smoke-ci'
952-
uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
953-
with:
954-
name: qa-smoke-profile-${{ github.run_id }}-${{ github.run_attempt }}
955-
path: .artifacts/qa-e2e/smoke-ci-profile/
956-
if-no-files-found: warn
957-
retention-days: 7
958-
959919
checks-fast-plugin-contracts-shard:
960920
permissions:
961921
contents: read

‎docs/ci.md‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ dispatch.
3232
| `security-fast` | Private key detection, changed-workflow audit via `zizmor`, and production lockfile audit | Always on non-draft pushes and PRs |
3333
| `pnpm-store-warmup` | Warm the lockfile-pinned pnpm store cache without blocking Linux Node shards | Node or docs-check lanes selected |
3434
| `build-artifacts` | Build `dist/`, Control UI, built-CLI smoke checks, startup memory, and embedded built-artifact checks | Node-relevant changes |
35-
| `checks-fast-core` | Fast Linux correctness lanes: bundled + protocol, QA Smoke CI, Bun launcher, and the CI-routing fast task | Node-relevant changes |
35+
| `checks-fast-core` | Fast Linux correctness lanes: bundled + protocol, Bun launcher, and the CI-routing fast task | Node-relevant changes |
3636
| `checks-fast-contracts-plugins-*` | Two weighted plugin contract shards | Node-relevant changes |
3737
| `checks-fast-contracts-channels-*` | Two weighted channel contract shards | Node-relevant changes |
3838
| `checks-node-*` | Core Node test shards, excluding channel, bundled, contract, and extension lanes | Node-relevant changes |
@@ -151,15 +151,15 @@ Release, private dist-tag, or other platform publication.
151151

152152
## Runners
153153

154-
| Runner | Jobs |
155-
| ------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
156-
| `ubuntu-24.04` | Manual CI dispatch and non-canonical repository fallbacks, CodeQL security and quality scans, workflow-sanity, labeler, auto-response, the standalone Docs workflow, and the whole Install Smoke workflow |
157-
| `blacksmith-4vcpu-ubuntu-2404` | `preflight`, `security-fast`, `pnpm-store-warmup`, `native-i18n`, `checks-fast-core` except QA Smoke CI, plugin/channel contract shards, most bundled/lower-weight Linux Node shards, `check-*` lanes except `check-lint`, selected `check-additional-*` shards, `check-docs`, and `skills-python` |
158-
| `blacksmith-8vcpu-ubuntu-2404` | Retained heavy Linux Node suites, boundary/extension-heavy `check-additional-*` shards, and `android` |
159-
| `blacksmith-16vcpu-ubuntu-2404` | QA Smoke CI, `build-artifacts` in CI and Testbox, and `check-lint` (CPU-sensitive enough that 8 vCPU cost more than they saved) |
160-
| `blacksmith-8vcpu-windows-2025` | `checks-windows` |
161-
| `blacksmith-6vcpu-macos-15` | `macos-node` on `openclaw/openclaw`; forks fall back to `macos-15` |
162-
| `blacksmith-12vcpu-macos-26` | `macos-swift` and `ios-build` on `openclaw/openclaw`; forks fall back to `macos-26` |
154+
| Runner | Jobs |
155+
| ------------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
156+
| `ubuntu-24.04` | Manual CI dispatch and non-canonical repository fallbacks, CodeQL security and quality scans, workflow-sanity, labeler, auto-response, the standalone Docs workflow, and the whole Install Smoke workflow |
157+
| `blacksmith-4vcpu-ubuntu-2404` | `preflight`, `security-fast`, `pnpm-store-warmup`, `native-i18n`, `checks-fast-core`, plugin/channel contract shards, most bundled/lower-weight Linux Node shards, `check-*` lanes except `check-lint`, selected `check-additional-*` shards, `check-docs`, and `skills-python` |
158+
| `blacksmith-8vcpu-ubuntu-2404` | Retained heavy Linux Node suites, boundary/extension-heavy `check-additional-*` shards, and `android` |
159+
| `blacksmith-16vcpu-ubuntu-2404` | `build-artifacts` in CI and Testbox, and `check-lint` (CPU-sensitive enough that 8 vCPU cost more than they saved) |
160+
| `blacksmith-8vcpu-windows-2025` | `checks-windows` |
161+
| `blacksmith-6vcpu-macos-15` | `macos-node` on `openclaw/openclaw`; forks fall back to `macos-15` |
162+
| `blacksmith-12vcpu-macos-26` | `macos-swift` and `ios-build` on `openclaw/openclaw`; forks fall back to `macos-26` |
163163

164164
## Runner registration budget
165165

‎test/scripts/ci-workflow-guards.test.ts‎

Lines changed: 4 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -958,10 +958,6 @@ describe("ci workflow guards", () => {
958958
const runStep = fastCoreJob.steps.find(
959959
(step) => step.name === "Run ${{ matrix.task }} (${{ matrix.runtime }})",
960960
);
961-
const uploadStep = fastCoreJob.steps.find(
962-
(step) => step.name === "Upload QA smoke profile evidence",
963-
);
964-
965961
const ciWorkflowText = readFileSync(".github/workflows/ci.yml", "utf8");
966962

967963
expect(preflightStep.run).not.toContain("qa-smoke-profile");
@@ -971,30 +967,13 @@ describe("ci workflow guards", () => {
971967
expect(ciWorkflowText).not.toContain(`"${categoryId}"`);
972968
}
973969
expect(runStep.run).toContain("bundled-protocol)");
974-
expect(runStep.run).toContain("qa-smoke-ci)");
970+
expect(runStep.run).not.toContain("qa-smoke-ci)");
975971
expect(runStep.run).toContain("contracts-plugins-ci-routing)");
976972
expect(runStep.run).toContain("ci-routing)");
977-
expect(ciWorkflowText).toContain('runner: "blacksmith-16vcpu-ubuntu-2404"');
978973
expect(fastCoreJob["runs-on"]).toContain("matrix.runner");
979-
expect(runStep.run).toContain("--qa-profile smoke-ci");
980-
expect(runStep.run).toContain("--concurrency 8");
981-
expect(runStep.run).not.toContain("--category");
982-
expect(runStep.run).not.toContain("--allow-failures");
983-
expect(runStep.run).toContain("qa_exit_code=0");
984-
expect(runStep.run).toContain('exit "$qa_exit_code"');
985-
expect(runStep.run).toContain("scripts/package-openclaw-for-docker.mjs");
986-
expect(runStep.run).toContain("OPENCLAW_CURRENT_PACKAGE_TGZ");
987-
expect(runStep.run).toContain("--max-old-space-size=16384");
988-
expect(runStep.run).not.toContain("scripts/build-all.mjs qaRuntime");
989-
expect(runStep.run).not.toContain("OPENAI_API_KEY");
990-
expect(runStep.run).toMatch(
991-
/bundled-protocol\)\s+pnpm test:bundled\s+pnpm protocol:check\s+;;\s+qa-smoke-ci\)/,
992-
);
993-
expect(uploadStep.if).toBe("always() && matrix.task == 'qa-smoke-ci'");
994-
expect(uploadStep.with).toMatchObject({
995-
path: ".artifacts/qa-e2e/smoke-ci-profile/",
996-
"if-no-files-found": "warn",
997-
});
974+
expect(ciWorkflowText).not.toContain('check_name: "QA Smoke CI"');
975+
expect(ciWorkflowText).not.toContain("--qa-profile smoke-ci");
976+
expect(ciWorkflowText).not.toContain("Upload QA smoke profile evidence");
998977
expect(runStep.run.match(/test\/scripts\/ci-workflow-guards\.test\.ts/g)?.length).toBe(2);
999978
});
1000979

0 commit comments

Comments
 (0)