Skip to content

Faster build and test workflow - #473

Merged
MaciejKaszynski merged 3 commits into
eclipse-score:mainfrom
etas-contrib:faster-actions
Aug 14, 2026
Merged

Faster build and test workflow#473
MaciejKaszynski merged 3 commits into
eclipse-score:mainfrom
etas-contrib:faster-actions

Conversation

@danth

@danth danth commented Aug 14, 2026

Copy link
Copy Markdown
Member

Changes extracted from #460 which was abandoned.

  • Combine build and test into one job so we can reuse the artifacts rather than building them twice.
  • Use the S-CORE bazel cache action rather than configuring it manually.
  • Merge sanitizer job into the build matrix.
  • Always upload test logs.

@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: efadfce3-eb27-43bc-91ad-842f1ad11735
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (28 packages loaded, 10 targets configured)

Analyzing: target //:license-check (87 packages loaded, 10 targets configured)

Analyzing: target //:license-check (90 packages loaded, 10 targets configured)

Analyzing: target //:license-check (149 packages loaded, 1738 targets configured)

Analyzing: target //:license-check (159 packages loaded, 6264 targets configured)

Analyzing: target //:license-check (164 packages loaded, 6285 targets configured)

Analyzing: target //:license-check (169 packages loaded, 6332 targets configured)

Analyzing: target //:license-check (171 packages loaded, 8060 targets configured)

Analyzing: target //:license-check (176 packages loaded, 11463 targets configured)

INFO: Analyzed target //:license-check (177 packages loaded, 11589 targets configured).
[2 / 5] checking cached actions
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 26.290s, Critical Path: 0.33s
INFO: 16 processes: 4 disk cache hit, 12 internal.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@github-actions

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@danth

danth commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

Remaining issues caused by building the whole codebase under sanitizers rather than only the parts covered by tests:

  • Maybe-uninit error in TSAN build
  • Some tests have hardcoded TSAN flags from before there was a config for it, this conflicts with ASAN

Worked around by skipping the build when sanitizers are on, as we did before. These can be fixed later.

Comment thread .github/workflows/on-pr.yml
Comment thread .github/workflows/on-pr-target.yml
danth added 3 commits August 14, 2026 14:46
As with all other Rust tests.
There is a build flag to run all tests with TSAN now, so these are
not needed.
@MaciejKaszynski
MaciejKaszynski merged commit 32fd776 into eclipse-score:main Aug 14, 2026
17 of 19 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in LCM - Lifecycle & Health FT Aug 14, 2026
@danth
danth deleted the faster-actions branch August 14, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants