Skip to content

Fix top-level documentation - #259

Open
rmaddikery wants to merge 4 commits into
mainfrom
rama_fix_docu
Open

Fix top-level documentation#259
rmaddikery wants to merge 4 commits into
mainfrom
rama_fix_docu

Conversation

@rmaddikery

@rmaddikery rmaddikery commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

See rendered documentation here:
#259 (comment)

Fix top-level documentation

  • Adds high-level overview explaining the mw::log structure
  • Remove outdated documentation
  • Restructure documentation to where it fits

@github-actions

github-actions Bot commented Aug 12, 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: b183ac87-a334-4f24-b534-7062713a0abc
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'score_process', the root module requires module version score_process@1.5.4, but got score_process@1.6.0 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: 
Loading: 
Loading: 4 packages loaded
Loading: 4 packages loaded
    currently loading: 
Loading: 4 packages loaded
    currently loading: 
Loading: 4 packages loaded
    currently loading: 
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: 11.889s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@github-actions

Copy link
Copy Markdown

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

@rmaddikery
rmaddikery marked this pull request as draft August 12, 2026 18:00
@rmaddikery
rmaddikery deployed to workflow-approval August 12, 2026 18:27 — with GitHub Actions Active
@rmaddikery
rmaddikery deployed to workflow-approval August 12, 2026 19:09 — with GitHub Actions Active
@rmaddikery
rmaddikery deployed to workflow-approval August 13, 2026 07:16 — with GitHub Actions Active
@rmaddikery
rmaddikery deployed to workflow-approval August 13, 2026 10:15 — with GitHub Actions Active
@rmaddikery rmaddikery moved this from Backlog to In Progress in LOG - Logging FT Aug 13, 2026
@rmaddikery
rmaddikery deployed to workflow-approval August 13, 2026 10:27 — with GitHub Actions Active
@rmaddikery
rmaddikery marked this pull request as ready for review August 13, 2026 10:50
@rmaddikery
rmaddikery deployed to workflow-approval August 13, 2026 10:55 — with GitHub Actions Active
- Adds high-level overview of the mw::log decomposition
- Remove outdated documentation
- Restructure documentation to where it fits
- Replace duplicated content with redirection to
single source that resides next to source code rendered
by Sphinx

@arsibo arsibo left a comment

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.

first formal findings

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.

A component's architecture describes how it is constructed from subcomponents. Since this component contains no subcomponents, this content should be moved to the detailed design section.

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.

file_output_backend and datarouter_backend are not shown in this diagram

=====================

.. comp:: Logging Component
:id: comp__mw_logging

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.

Should mw_looging be the final component name? I thought we wanted to use log_backend in the logging repo and 'log' in the baselibs repo.

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.

Not confirmed until we get the first poc from Aleks

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.

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.

file shall be moved to ./mw_log/requirements/
See folder structure under <component_name>/ https://eclipse-score.github.io/score/main/contribute/general/folder.html

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.

Shall we add a folder _assets for that ?

Comment thread docs/guides/index.rst

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.

content shall be moved to docs/manuals/


.. comp:: Data Router
.. comp:: Datarouter
:id: comp__data_router

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.

component id shall be 🆔 comp__datarouter due to folder name datarouter

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.

Runtime plugin loading remains experimental and is not a supported
production capability.

.. uml:: mw_log_repository_boundary.puml

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.

this belongs to the feature architecture . Feature architecture has to be modeled in sphinx needs

written is less then requested.

![Action Diagram](./score/mw/log/design/slot_drainer_action_diagram_design.puml)
<img alt="SlotDrainerActionDiagram" src="https://www.plantuml.com/plantuml/proxy?src=https://raw.githubusercontent.com/eclipse-score/baselibs/refs/heads/main/score/mw/log/design/slot_drainer_action_diagram_design.puml">

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.

puml file was not rendered

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.

The file shall be included as reference to detailed design or there should be folder on the rendered website design decisions

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.

file_output_backend and datarouter_backend are not shown in this diagram

### Process structure

[context-highlevel](uml/context-highlevel.puml)
<img alt="CONTEXT_HIGHLEVEL" src="https://www.plantuml.com/plantuml/proxy?src=https://raw.githubusercontent.com/eclipse-score/logging/refs/heads/main/score/datarouter/doc/design/uml/context-highlevel.puml">

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.

Diagramm shows an Adptive Autosar ECU

@@ -50,9 +50,9 @@ The following constraints influenced the logging infrastructure design:
## Context

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.

chapter could be copied/moved to feature description

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

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants