Skip to content

ci: reduce PR workflow latency by relocating coverage#76

Merged
PLeVasseur merged 17 commits intoeclipse-uprotocol:mainfrom
PLeVasseur:perf/ci-pr-latency-reduction
Feb 10, 2026
Merged

ci: reduce PR workflow latency by relocating coverage#76
PLeVasseur merged 17 commits intoeclipse-uprotocol:mainfrom
PLeVasseur:perf/ci-pr-latency-reduction

Conversation

@PLeVasseur
Copy link
Contributor

@PLeVasseur PLeVasseur commented Feb 7, 2026

Summary

  • Move tarpaulin coverage out of PR Test jobs in bundled/unbundled workflows and run standard functional test paths in PR CI.
  • Add a dedicated Nightly Coverage workflow (schedule, workflow_dispatch, and push to main) that runs tarpaulin once and uploads coverage artifacts.
  • Remove duplicated no-feature lint/fmt checks from unbundled lint and add Rust dependency/build caching to bundled and unbundled compile-heavy jobs.
  • Preserve existing required PR job names (Lint, Test, Build documentation) and keep obtain_and_build_vsomeip intact.

CI timing comparison

Comparison method

Initial-run comparison (PR #75 baseline -> PR #76 first successful run)

Metric Baseline (PR #75) Initial (PR #76) Delta
Bundled workflow total 12m30s 13m16s +6.1%
Bundled Lint 12m21s 12m50s +3.9%
Bundled Test 9m18s 4m54s -47.3%
Unbundled workflow total 9m28s 7m28s -21.1%
Unbundled Lint 9m01s 6m45s -25.1%
Unbundled Test 9m05s 4m48s -47.2%

Latest-run improvement on this PR after iterative branch updates (initial -> latest)

Metric Initial (PR #76) Latest (PR #76) Delta
Bundled workflow total 13m16s 4m21s -67.2%
Bundled lint critical path 12m50s 2m53s* -77.5%
Bundled Test 4m54s 4m10s -15.0%
Unbundled workflow total 7m28s 4m27s -40.4%
Unbundled Lint 6m45s 1m30s -77.8%
Unbundled Test 4m48s 4m04s -15.3%
  • Latest bundled lint is split into shards and aggregated (Lint (feature build+clippy) 2m49s + Lint aggregator 4s; Lint (base build+clippy) runs in parallel).

Notes

  • This is not a forced cold-cache benchmark; numbers reflect normal CI behavior with cache reuse.
  • obtain_and_build_vsomeip remained a cache-hit in all compared runs (10s baseline, 15s initial, 11s latest), so the main gains are from CI workflow/test/lint path changes.

Run links

@PLeVasseur PLeVasseur force-pushed the perf/ci-pr-latency-reduction branch 3 times, most recently from 0e63b9d to 71fc302 Compare February 7, 2026 13:46
@sophokles73
Copy link

@PLeVasseur Am I right in assuming that this should be rebased after #75 has been merged?

@PLeVasseur
Copy link
Contributor Author

@PLeVasseur Am I right in assuming that this should be rebased after #75 has been merged?

Yep, exactly, will do that now

@PLeVasseur PLeVasseur force-pushed the perf/ci-pr-latency-reduction branch from 5f7e6b6 to fc68e02 Compare February 9, 2026 14:57
@PLeVasseur
Copy link
Contributor Author

FYI -- rebased this now.

Copy link

@sophokles73 sophokles73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sophokles73 sophokles73 added the enhancement New feature or request label Feb 10, 2026
@PLeVasseur PLeVasseur merged commit ae8afd9 into eclipse-uprotocol:main Feb 10, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants