Skip to content

perf(ci): remove metadata-only setup and empty build jobs - #571

Merged
neilime merged 6 commits into
mainfrom
perf/475-p2-node-ci-preparation
Nov 8, 2026
Merged

neilime merged 6 commits into
mainfrom
perf/475-p2-node-ci-preparation

Conversation

@neilime

@neilime neilime commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Callers using prebuilt images with build: "" currently start containers for metadata-only setup and an empty build before tests can run. This change prepares metadata on the host, skips both jobs, and runs lint and tests in parallel. Requested builds still gate tests and optional packaging.

  • Preserve build inputs, secrets, reports and artifact layout; resolve paths and validate workspace boundaries in the application jobs.
  • Keep workspace-package discovery conditional and independent of compilation. Failed preparation, discovery or requested builds block dependent jobs; cancellation remains effective.
  • Cover the job graph with 60 contract tests and host/container integration cases, including source packaging without a build.

CI for the current head passed. Local documentation formatting, Markdown/prose lint and link checks passed. Historical failure, disabled-check and cancellation trials are linked from ADR 0002.

Five paired benchmark runs on 2026-11-06 reduced median elapsed time from preparation from 101s to 52s, and summed job time from 135s to 90s. These results concern the controlled one-test fixture at the revisions recorded in the ADR; they do not establish Pleey end-to-end performance or billed-minute savings. The ADR retains the methodology, ranges and direct run links.

Callers requiring the removed Setup check or an empty Build check must update their required checks when adopting the release.

Relates to cloud-native-aixmarseille/pleey#475, phase P2.1. Release and Pleey adoption remain separate steps.

@github-actions github-actions Bot added the type: performance Improves performance label Nov 6, 2026
@github-actions

github-actions Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Super-linter summary

Language Validation result
BIOME_FORMAT Pass ✅
BIOME_LINT Pass ✅
CHECKOV Pass ✅
GITHUB_ACTIONS Pass ✅
GITHUB_ACTIONS_ZIZMOR Pass ✅
GITLEAKS Pass ✅
GIT_COMMITLINT Pass ✅
GIT_MERGE_CONFLICT_MARKERS Pass ✅
JSCPD Pass ✅
MARKDOWN Pass ✅
MARKDOWN_PRETTIER Pass ✅
NATURAL_LANGUAGE Pass ✅
PRE_COMMIT Pass ✅
SPELL_CODESPELL Pass ✅
TRIVY Pass ✅
YAML Pass ✅
YAML_PRETTIER Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter
Super-linter revision: 2da136927bd4a73596db63044b504547c62cb854
Super-linter version: 9.0.0

@codecov

codecov Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (bf955e5) to head (0565284).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #571   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            3         3           
=========================================
  Hits             3         3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Code Coverage Report

Coverage Results

Coverage

Metric Covered Total Percentage
Lines 2 2 100.00%
Functions 1 1 100.00%

Overall: 100.00% 🟢
🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩🟩

@neilime
neilime marked this pull request as ready for review November 6, 2026 20:52
@neilime
neilime force-pushed the perf/475-p2-node-ci-preparation branch 3 times, most recently from c19c5d8 to 3fa7d98 Compare November 7, 2026 15:53
@neilime
neilime force-pushed the perf/475-p2-node-ci-preparation branch from 3fa7d98 to cdf5653 Compare November 8, 2026 11:09
@neilime
neilime merged commit daf186b into main Nov 8, 2026
226 checks passed
@neilime
neilime deleted the perf/475-p2-node-ci-preparation branch November 8, 2026 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: performance Improves performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant