Skip to content

Rename HealthMonitor component to AliveMonitor#206

Merged
NicolasFussberger merged 2 commits into
eclipse-score:mainfrom
PetarResetarRTRK:issue-138-rename-healthmonitor-to-alivemonitor
May 20, 2026
Merged

Rename HealthMonitor component to AliveMonitor#206
NicolasFussberger merged 2 commits into
eclipse-score:mainfrom
PetarResetarRTRK:issue-138-rename-healthmonitor-to-alivemonitor

Conversation

@PetarResetarRTRK
Copy link
Copy Markdown
Contributor

@PetarResetarRTRK PetarResetarRTRK commented May 19, 2026

Summary

Rename HealthMonitor component to AliveMonitor

Changes

Renamed Components:

Interface:
IHealthMonitorThread → IAliveMonitorThread
Implementation Class:
HealthMonitorThread → AliveMonitorThread
Files:
ihealth_monitor_thread.hpp → ialive_monitor_thread.hpp
health_monitor_thread.hpp → alive_monitor_thread.hpp
health_monitor_thread.cpp → alive_monitor_thread.cpp

Updated Variable Names:

Constructor parameter:
health_monitor_thread → alive_monitor_thread (ProcessGroupManager)
Member variable:
health_monitor_thread_ → alive_monitor_thread_ (ProcessGroupManager)
Thread object:
alive_monitor_thread_ (AliveMonitorThread implementation)
Member IHealthMonitor pointer:
m_health_monitor (preserved from original naming convention)

Internal callers using the renamed components will need to:

  • Update includes from health_monitor_thread.hpp to alive_monitor_thread.hpp
  • Update type references: HealthMonitorThread → AliveMonitorThread, IHealthMonitorThread → IAliveMonitorThread

Closes #138

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

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.4.2) and connecting to it...
INFO: Invocation ID: bd2cd125-318f-411b-875b-b2b28463a00a
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 (29 packages loaded, 10 targets configured)

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

Analyzing: target //:license-check (131 packages loaded, 855 targets configured)

Analyzing: target //:license-check (145 packages loaded, 3876 targets configured)

Analyzing: target //:license-check (147 packages loaded, 8030 targets configured)

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

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

INFO: Analyzed target //:license-check (162 packages loaded, 10232 targets configured).
[1 / 1] no actions running
[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: 22.221s, Critical Path: 2.64s
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
Copy link
Copy Markdown

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

@PetarResetarRTRK PetarResetarRTRK force-pushed the issue-138-rename-healthmonitor-to-alivemonitor branch from 064552e to c012104 Compare May 20, 2026 07:59
@NicolasFussberger NicolasFussberger merged commit 0566998 into eclipse-score:main May 20, 2026
19 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename LaunchManager-internal component "HealthMonitor"

2 participants