Skip to content

Implementation completion datarouter - #379

Merged
rmaddikery merged 5 commits into
eclipse-score:mainfrom
qorix-group:a.b/implementation_completion_datarouter
Oct 28, 2026
Merged

Implementation completion datarouter#379
rmaddikery merged 5 commits into
eclipse-score:mainfrom
qorix-group:a.b/implementation_completion_datarouter

Conversation

@AdamBienkunski

@AdamBienkunski AdamBienkunski commented Oct 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Notes for Reviewer

Pre-Review Checklist for the PR Author

  • PR title is short, expressive and meaningful
  • Commits are properly organized
  • Relevant issues are linked in the References section
  • Tests are conducted
  • Unit tests are added

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes ##329

Merge PR #363 first

@github-actions

github-actions Bot commented Oct 26, 2026 •
edited
Loading

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: 8199d934-5b2a-4db4-a695-0cdf73056572
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'rules_cc', the root module requires module version rules_cc@0.2.17, but got rules_cc@0.2.25 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 4 packages loaded
DEBUG: /home/runner/.bazel/external/score_docs_as_code+/docs.bzl:716:14: INFO: ⚠️ DEPRECATED: Passing a Needs inventory through docs(data = [...]) is deprecated!!! Move it to external_needs = [...] instead.
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/logging/logging/BUILD
INFO: Elapsed time: 19.904s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

rmaddikery
rmaddikery reviewed Oct 27, 2026
View reviewed changes
Comment thread scripts/dependency_reports/datarouter.json Outdated
@@ -0,0 +1,47 @@
{

@rmaddikery rmaddikery Oct 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed lets exclude this from this PR.

@AdamBienkunski AdamBienkunski Oct 27, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

rmaddikery
rmaddikery reviewed Oct 27, 2026
View reviewed changes
Comment thread docs/components/datarouter/detailed_design/dependencies.rst Outdated
@@ -0,0 +1,666 @@
..

@rmaddikery rmaddikery Oct 27, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please exclude this (manual maintained list) since we intend to validate this through process tooling

@AdamBienkunski AdamBienkunski Oct 27, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@AdamBienkunski
AdamBienkunski force-pushed the a.b/implementation_completion_datarouter branch from f50e165 to f756545 Compare October 27, 2026 14:29
@AdamBienkunski
AdamBienkunski deployed to workflow-approval October 27, 2026 14:29 — with GitHub Actions Active
@AdamBienkunski
AdamBienkunski force-pushed the a.b/implementation_completion_datarouter branch from f756545 to 7c117af Compare October 27, 2026 14:42
@AdamBienkunski
AdamBienkunski deployed to workflow-approval October 27, 2026 14:43 — with GitHub Actions Active
rmaddikery
rmaddikery reviewed Oct 27, 2026
View reviewed changes
Comment thread score/datarouter/doc/design/logging_architecture.md Outdated
@AdamBienkunski
AdamBienkunski force-pushed the a.b/implementation_completion_datarouter branch from 7c117af to c2db0b8 Compare October 28, 2026 08:28
@github-project-automation github-project-automation Bot moved this from Backlog to On Hold in LOG - Logging FT Oct 28, 2026
@github-actions

github-actions Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-379: https://eclipse-score.github.io/logging/pr-379/

@AdamBienkunski
AdamBienkunski force-pushed the a.b/implementation_completion_datarouter branch from c2db0b8 to 364ed08 Compare October 28, 2026 10:13
@AdamBienkunski
docs(datarouter): use local diagrams in the design
d9f2207 
- Replace the diagrams loaded by URL from plantuml.com and
  raw.githubusercontent.com by uml directives
- Link the diagram files in the design folder through symbolic links in
  docs/components/datarouter/detailed_design/_assets
- Add the unit, IPC and file transfer diagrams that no document used
- Add a section about the file transfer

Signed-off-by: Adam Bienkunski <Adam.Bienkunski.Ext@qorix.ai>
@AdamBienkunski
docs(datarouter): restructure detailed design after template
49d2055 
- Add description, design decisions, constraints and rationale. Decisions
  and constraints refer to the diagrams that show them.
- Add the static and dynamic views, the units table and the interfaces
  table.
- Refer to the checklist for the external dependencies and their
  findings.

Signed-off-by: Adam Bienkunski <Adam.Bienkunski.Ext@qorix.ai>
@AdamBienkunski
build(datarouter): add architectural design target
51a1795 
- Add the architectural_design target for the datarouter design
  diagrams, validated with the rules_score validator
- Add the unit diagram that the validator requires (units and
  interfaces of the sequence diagrams)
- Rename participant aliases that clashed between sequence diagrams
- Replace syntax that the diagram parser does not support (multi-line
  members and notes, default and mock method declarations)

Signed-off-by: Adam Bienkunski <Adam.Bienkunski.Ext@qorix.ai>
@AdamBienkunski
test(datarouter): link tests to component requirements
a39f4d0 
Link existing tests with RecordProperty("PartiallyVerifies", ...) to
the requirements they verify. No test logic changed.

- comp_req__data_router__dlt_server: UDP send test
- comp_req__data_router__dlt_multiple_channels,
  comp_req__data_router__dlt_message_filtering: channel configuration
  file tests
- comp_req__data_router__dlt_bw_quota_config: quota configuration test
- comp_req__data_router__dlt_quota_bw_reduction: quota enforcement
  attribute tests (the dropping of messages is not tested)

Signed-off-by: Adam Bienkunski <Adam.Bienkunski.Ext@qorix.ai>
@AdamBienkunski
docs(datarouter): add implementation inspection checklist results
9feb96a 
Fill in the implementation inspection checklist of the datarouter.

- Add the passed value and the remarks for each checklist item.
- Record the findings with their tickets (eclipse-score#318 coverage, eclipse-score#319
  requirement traceability, eclipse-score#336 CodeQL, eclipse-score#327 mw::log context) and
  mark the findings that still need a ticket.
- Refer to the generated verification report and the dependency page
  instead of repeating analysis results.

Signed-off-by: Adam Bienkunski <Adam.Bienkunski.Ext@qorix.ai>
@AdamBienkunski
AdamBienkunski force-pushed the a.b/implementation_completion_datarouter branch from 364ed08 to 9feb96a Compare October 28, 2026 10:42
@rmaddikery
rmaddikery merged commit 0dba677 into eclipse-score:main Oct 28, 2026
18 checks passed
@github-project-automation github-project-automation Bot moved this from On Hold to Done in LOG - Logging FT Oct 28, 2026

This branch was previously deployed

1 inactive deployment
workflow-approval — 9feb96ad Deployed Oct 28, 2026 by AdamBienkunski via approval #1005
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rmaddikery rmaddikery rmaddikery approved these changes
@antonkri antonkri Awaiting requested review from antonkri antonkri is a code owner
@4og 4og Awaiting requested review from 4og 4og is a code owner
@hoppe-and-dreams hoppe-and-dreams Awaiting requested review from hoppe-and-dreams hoppe-and-dreams is a code owner
@pawelrutkaq pawelrutkaq Awaiting requested review from pawelrutkaq pawelrutkaq is a code owner
@arsibo arsibo Awaiting requested review from arsibo arsibo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

LOG - Logging FT
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@AdamBienkunski @rmaddikery