Skip to content

add logging demoapp#290

Open
ShivamPatidar24 wants to merge 1 commit into
eclipse-score:mainfrom
ShivamPatidar24:add_logging_demoapp
Open

add logging demoapp#290
ShivamPatidar24 wants to merge 1 commit into
eclipse-score:mainfrom
ShivamPatidar24:add_logging_demoapp

Conversation

@ShivamPatidar24

Copy link
Copy Markdown
Contributor

Summary

This PR adds a Logging example showcase demonstrating local and remote logging.

The example generates logs for all supported severity levels.

Related

Issue: #229

Verification
DEBUG, INFO, WARN, ERROR and FATAL logs generated
Logs visible in console output
Logs visible in DLT Viewer and Chipmunk

@github-actions

Copy link
Copy Markdown

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

Comment thread feature_integration_tests/itf/test_remote_logging.py Outdated
@ShivamPatidar24
ShivamPatidar24 force-pushed the add_logging_demoapp branch 2 times, most recently from 8fd5d59 to 03363b7 Compare July 16, 2026 08:54

# OCI / Docker image rules for integration tests
bazel_dep(name = "rules_oci", version = "2.2.7", dev_dependency = True)
-bazel_dep(name = "rules_pkg", version = "1.2.0", dev_dependency = True)

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.

You can make the binary target public in score_logging. Then create a pkg_tar here with the config.file. E.g.: logging.json - Then you don't need this patch!

Every patch is an hidden tech debt making debugging hard. Also the config files should not be shared. A changed config. could affect tests repo wide which is not ideal.

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

Comment thread patches/logging/001-logging-example-visibility.patch Outdated

@rmaddikery rmaddikery 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.

@ShivamPatidar24
ShivamPatidar24 force-pushed the add_logging_demoapp branch 6 times, most recently from d8a9d61 to ec753ca Compare July 24, 2026 14:48
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.

3 participants