Skip to content

Fix in shutdown of PGM - #537

Merged
NicolasFussberger merged 2 commits into
eclipse-score:mainfrom
etas-contrib:bugfix/crash_on_lcm_shutdown
Aug 28, 2026
Merged

NicolasFussberger merged 2 commits into
eclipse-score:mainfrom
etas-contrib:bugfix/crash_on_lcm_shutdown

Conversation

@TimoSteuerwaldETAS

Copy link
Copy Markdown
Contributor

Race condition in pipeline runs during lcm shutdown, when process_monitor is still used in worker_jobs / thread_pool, which leads to failing integration tests.

For example here: https://github.com/eclipse-score/lifecycle/actions/runs/32954787452/job/98133959929?pr=530

Solution: Move process_monitor_.reset() after the reset of thread_pool / worker_jobs.

@github-actions

github-actions Bot commented Aug 26, 2026 •

Copy link
Copy Markdown
Contributor

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: 6949ff7b-51ff-4a60-9014-3d8fed997799
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

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

Analyzing: target //:license-check (155 packages loaded, 3389 targets configured)

Analyzing: target //:license-check (160 packages loaded, 6259 targets configured)

Analyzing: target //:license-check (161 packages loaded, 6379 targets configured)

Analyzing: target //:license-check (167 packages loaded, 11377 targets configured)

Analyzing: target //:license-check (167 packages loaded, 11377 targets configured)

INFO: Analyzed target //:license-check (173 packages loaded, 11555 targets configured).
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: 23.674s, Critical Path: 0.34s
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>]

process_monitor_ could be still used in a worker_jobs / thread_pool.
@NicolasFussberger
NicolasFussberger deployed to workflow-approval August 28, 2026 07:33 — with GitHub Actions Active
@NicolasFussberger
NicolasFussberger deployed to workflow-approval August 28, 2026 07:33 — with GitHub Actions Active
@NicolasFussberger
NicolasFussberger merged commit e9a71c6 into eclipse-score:main Aug 28, 2026
30 of 32 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in LCM - Lifecycle & Health FT Aug 28, 2026

This branch was successfully deployed

1 active deployment
workflow-approval — 5f1307f0 Deployed Aug 28, 2026 by NicolasFussberger via Build and test unit-tests-arm64-qnx / approval #405
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants