Skip to content

Fixes issue 284#287

Merged
NicolasFussberger merged 1 commit into
eclipse-score:mainfrom
etas-contrib:feature/fixes-for-284_tmp
Jul 6, 2026
Merged

Fixes issue 284#287
NicolasFussberger merged 1 commit into
eclipse-score:mainfrom
etas-contrib:feature/fixes-for-284_tmp

Conversation

@paulquiring

Copy link
Copy Markdown
Contributor
  1. added lock guard in

    • ipc_dropin::RingBuffer::empty
    • ipc_dropin::RingBuffer::full
  2. added static mutex with accessor in IdentifierHash which is to be called before `get_registry``

  3. added else if for kTerminating in ProcessInfoNode::unexpectedTermination to prevent fall-through

fixes #284

    * ipc_dropin::RingBuffer::empty
    * ipc_dropin::RingBuffer::full

2. added static mutex with accessor in `IdentifierHash`
   which is to be called before `get_registry``

3. added else if for `kTerminating` in `ProcessInfoNode::unexpectedTermination`
   to prevent fall-through

fixes eclipse-score#284
@github-actions

github-actions Bot commented Jul 6, 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: fe85be11-aac7-420e-8a65-8559147bc301
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: 
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

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

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

Analyzing: target //:license-check (142 packages loaded, 1259 targets configured)

Analyzing: target //:license-check (152 packages loaded, 7044 targets configured)

Analyzing: target //:license-check (157 packages loaded, 8216 targets configured)

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

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

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

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

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

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

Analyzing: target //:license-check (168 packages loaded, 10283 targets configured)

Analyzing: target //:license-check (168 packages loaded, 10283 targets configured)

INFO: Analyzed target //:license-check (169 packages loaded, 10409 targets configured).
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[15 / 16] Building license.check.license_check.jar (); 0s disk-cache, multiplex-worker
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.831s, Critical Path: 2.63s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
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

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

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

@NicolasFussberger NicolasFussberger self-requested a review July 6, 2026 13:08
@NicolasFussberger NicolasFussberger merged commit 5b1ab69 into eclipse-score:main Jul 6, 2026
21 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in LCM - Lifecycle & Health FT Jul 6, 2026
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.

[launch_manager] Read-Write races and a logical timing Bug in ProcessInfoNode::unexpectedTermination()

2 participants