Removing Config Adapter by MaciejKaszynski · Pull Request #455 · eclipse-score/lifecycle

MaciejKaszynski · 2026-08-12T08:17:24Z

Fixes #420

Currently we have a lot of code to bridge the old config and new. This PR removes the bridge and changes. Process Group Manager, Graph and Alive Monitor to use the new config.

github-actions · 2026-08-13T07:14:54Z

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: cd4e52b3-a386-4310-b44c-fbe75b7968a0
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 (35 packages loaded, 10 targets configured)

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

Analyzing: target //:license-check (144 packages loaded, 313 targets configured)

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

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

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

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

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

INFO: Analyzed target //:license-check (177 packages loaded, 11589 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.027s, Critical Path: 0.30s
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 · 2026-08-14T14:32:22Z

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

danth

The graph creation function is a bit long, but since it will be improved anyway after #463, I don't think it is worth making any major changes in this pull request.

danth · 2026-08-14T15:00:53Z

 {
    score::concurrency::InterruptibleFuture<void> retVal_{};

+    std::cout << "Setting state " << pg_name << pg_state << std::endl;


Use LM_LOG_DEBUG, or remove this.

danth · 2026-08-14T15:20:27Z

+    ISupervisionEventPublisher& supervision_event_publisher,
+    osal::IProcess* process_interface,
+    std::shared_ptr<SafeProcessMapInserter> process_map,
+    std::unordered_map<std::size_t, GraphIndex>& run_target_map)


run_target_map is not documented

danth · 2026-08-14T15:21:22Z


-    LM_LOG_DEBUG() << "Process group index" << index << "(with name" << pg_name << ") has" << num_processes
-                   << "processes";
+    DependencyGraph<Graph::Component> graph(components.size() + run_targets.size() + 1);


Should we add a comment explaining why the + 1 is necessary?

danth · 2026-08-14T15:22:33Z

+        component_name_to_index[component_name] = index;
+
+        // all of this relies on graphindex having sequential indexes
+        // TODO use IdHash later...


Maybe add a link to #463 and explain that this method will be simplified after that change.

danth · 2026-08-14T15:23:16Z

        {
-            return static_cast<int32_t>(index);
+            auto it = component_name_to_index.find(dep_name);
+            LM_LOG_ERROR() << run_targets[i].name << "HAS DEP TO " << dep_name;


Suggested change

LM_LOG_ERROR() << run_targets[i].name << "HAS DEP TO " << dep_name;

LM_LOG_DEBUG() << run_targets[i].name << "HAS DEP TO " << dep_name;

danth · 2026-08-14T15:27:47Z

+#include <ios>
+#include <iostream>


Not needed.

danth · 2026-08-14T15:29:18Z

-    // Wait for process group states to change while actively draining shutdown events.
+    if (!graph_)
+    {
+        return;


Should we log a warning here?

danth · 2026-08-14T15:30:23Z

+        LM_LOG_DEBUG() << "HERER++++++++";
        // Now process the request
        switch (scc->request().request_or_response_)
        {
            case ControlClientCode::kSetStateRequest:
+                LM_LOG_DEBUG() << "HERER++++++++";


Remove print debugging.

danth · 2026-08-14T15:31:52Z

+            // TODO: Determine recovery state from configuration
+            recovery_state.pg_state_name_ = IdentifierHash("fallback");  // TODO: Get from configuration


Does this need to be done?

danth · 2026-08-14T15:36:31Z

+            SCORE_LANGUAGE_FUTURECPP_PRECONDITION_MESSAGE(
+                it != component_name_to_index.end(), "RunTarget dependency not found in component list");
+
+            graph.addDependency(num_of_components + i, it->second);


I think this would be clearer if we had something like

for (std::size_t run_target_i = 0; run_target_i < run_targets.size(); ++run_target_i) { std::size_t node_i = num_of_components + run_target_i;

github-project-automation Bot added this to LCM - Lifecycle & Health FT Aug 12, 2026

github-project-automation Bot moved this to Backlog in LCM - Lifecycle & Health FT Aug 12, 2026

MaciejKaszynski requested a deployment to workflow-approval August 12, 2026 08:17 — with GitHub Actions Waiting

MaciejKaszynski requested a deployment to workflow-approval August 13, 2026 07:13 — with GitHub Actions Waiting

MaciejKaszynski added 8 commits August 13, 2026 09:38

Inital changes

8118405

Rest is working

69ede3d

Fixing rebase

aa2482c

Graph converted

1b738e3

Fixing namespace

8420e6b

lm building

f2ea525

Fixing some tests

50ec974

Fix rebase

84721b9

MaciejKaszynski force-pushed the new-config branch from 48210c0 to 84721b9 Compare August 13, 2026 09:15

MaciejKaszynski requested a deployment to workflow-approval August 13, 2026 09:15 — with GitHub Actions Waiting

Fixing dep graph

7fff290

MaciejKaszynski requested a deployment to workflow-approval August 13, 2026 14:26 — with GitHub Actions Waiting

Some fixes

f961f8a

MaciejKaszynski requested a deployment to workflow-approval August 14, 2026 08:24 — with GitHub Actions Waiting

Adding bad fix

4bbbfed

MaciejKaszynski requested a deployment to workflow-approval August 14, 2026 08:28 — with GitHub Actions Waiting

Fixing last test

ee01564

MaciejKaszynski requested a deployment to workflow-approval August 14, 2026 10:51 — with GitHub Actions Waiting

MaciejKaszynski added 2 commits August 14, 2026 14:22

Moving watchdog

d87d3db

Fixing UTs

7a58b19

MaciejKaszynski deployed to workflow-approval August 14, 2026 13:45 — with GitHub Actions Active

Merge branch 'main' into new-config

98b3963

MaciejKaszynski requested a deployment to workflow-approval August 14, 2026 14:24 — with GitHub Actions Waiting

MaciejKaszynski marked this pull request as ready for review August 14, 2026 14:24

MaciejKaszynski requested review from FScholPer, NicolasFussberger, anmittag, antonkri, pawelrutkaq and ramceb as code owners August 14, 2026 14:24

danth reviewed Aug 14, 2026

View reviewed changes

Navigation Menu

Uh oh!

Removing Config Adapter - #455

Removing Config Adapter#455
MaciejKaszynski wants to merge 15 commits into
eclipse-score:mainfrom
etas-contrib:new-config

MaciejKaszynski commented Aug 12, 2026

Uh oh!

github-actions Bot commented Aug 13, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Aug 14, 2026

Uh oh!

danth left a comment

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

danth Aug 14, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	LM_LOG_ERROR() << run_targets[i].name << "HAS DEP TO " << dep_name;
	LM_LOG_DEBUG() << run_targets[i].name << "HAS DEP TO " << dep_name;

		// TODO: Determine recovery state from configuration
		recovery_state.pg_state_name_ = IdentifierHash("fallback"); // TODO: Get from configuration

		#include <ios>
		#include <iostream>

Uh oh!

Conversation

MaciejKaszynski commented Aug 12, 2026

Uh oh!

github-actions Bot commented Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

License Check Results

Uh oh!

github-actions Bot commented Aug 14, 2026

Uh oh!

danth left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented Aug 13, 2026 •

edited

Loading