Skip to content

Add rt_running_when_process_exits integration test - #466

Open
TimoSteuerwaldETAS wants to merge 15 commits into
eclipse-score:mainfrom
etas-contrib:feature/rt_running_when_process_exits
Open

Add rt_running_when_process_exits integration test#466
TimoSteuerwaldETAS wants to merge 15 commits into
eclipse-score:mainfrom
etas-contrib:feature/rt_running_when_process_exits

Conversation

@TimoSteuerwaldETAS

Copy link
Copy Markdown
Contributor

Part of #317

Verifies that a Run Target becomes ready only once a self-terminating component's process has actually exited, both when the terminated-ready component has a dependent and when it has none.

This test does not cover one requirement alone, but partially covers several requirements.

@github-actions

github-actions Bot commented Aug 13, 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: 8fde4f62-3795-41a4-bef1-c650ce852963
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 (41 packages loaded, 10 targets configured)

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

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

Analyzing: target //:license-check (162 packages loaded, 4615 targets configured)

Analyzing: target //:license-check (162 packages loaded, 9393 targets configured)

Analyzing: target //:license-check (172 packages loaded, 9451 targets configured)

Analyzing: target //:license-check (173 packages loaded, 9575 targets configured)

INFO: Analyzed target //:license-check (177 packages loaded, 11589 targets configured).
[15 / 16] [Prepa] Generating Dash formatted dependency file ...
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: 22.610s, Critical Path: 0.38s
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

run_target_slow_setup depends directly on slow_setup_sh — a Native, self-terminating component with ready condition Terminated and no dependents. At commit before the Terminated ready-condition is only enforced on dependency edges, so a component with no dependents is treated as ready-when-running: ActivateRunTarget("run_target_slow_setup").Get() returns before slow_setup.sh (which sleep 1 then writes its marker) has finished. The control client then finds the marker file missing → failure. The run_target_reader activation (a Terminated-ready component with a dependent) is the passing contrast case, and it stays green at both revisions.
Instead read out ready_on_termination flag.
TODO: Reword commit message
@anmittag

Copy link
Copy Markdown
Member

@TimoSteuerwaldETAS a) Outdated branch, b) some checks failed

Please adapt and Update the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants