Skip to content

impl: logging support #206

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
arkjedrz wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

impl: logging support #206

arkjedrz wants to merge 1 commit into from

Conversation

@arkjedrz
Copy link
Contributor

@arkjedrz arkjedrz commented Jan 7, 2026 •
edited
Loading

  • Use stdout_logger backend for Cargo builds.
  • Use score_log_bridge backend for Bazel builds.
  • Additional logs from Rust implementation.
  • Use proxy module for logging with correct context across the lib.
  • Fix tests.
  • Bump toolchain version.

@arkjedrz arkjedrz marked this pull request as draft January 7, 2026 15:01
@arkjedrz arkjedrz requested a review from pawelrutkaq January 7, 2026 15:01
@github-actions
Copy link

github-actions bot commented Jan 7, 2026 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 38488dbb-1afc-46ac-a43e-3f683355d28e
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'rust_qnx8_toolchain+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-oEubHgeZDdT0svMmBKJx7c3/2TdSI/vfwRUyDn+TPGA="
DEBUG: Repository rust_qnx8_toolchain+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'rules_python', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_rust_policies', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_cc', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'googletest', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_bazel_platforms', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_process', the root module requires module version [email protected], but got [email protected] in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Loading: 
Loading: 4 packages loaded
Loading: 4 packages loaded
    currently loading: 
Analyzing: target //:license-check (5 packages loaded, 0 targets configured)
Analyzing: target //:license-check (5 packages loaded, 0 targets configured)

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

Analyzing: target //:license-check (158 packages loaded, 2628 targets configured)

Analyzing: target //:license-check (166 packages loaded, 7391 targets configured)

Analyzing: target //:license-check (166 packages loaded, 7391 targets configured)

INFO: Analyzed target //:license-check (169 packages loaded, 9407 targets configured).
[9 / 14] [Prepa] Executing genrule //:filtered_cargo_lock ... (2 actions, 0 running)
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 66 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
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.954s, Critical Path: 0.83s
INFO: 14 processes: 3 disk cache hit, 9 internal, 2 processwrapper-sandbox.
INFO: Build completed successfully, 14 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>]

@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch from 77bc368 to d9faad5 Compare January 7, 2026 15:06
pawelrutkaq
pawelrutkaq reviewed Jan 7, 2026
View reviewed changes
src/rust/rust_kvs_tool/BUILD Outdated
"@score_baselibs_rust//src/log/mw_log",
"@score_crates//:pico_args",
"@score_crates//:tinyjson",
"@score_logging//src/rust/mw_logger",
Copy link
Contributor

@pawelrutkaq pawelrutkaq Jan 7, 2026

Choose a reason for hiding this comment

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

can't rustc_flags comes from mw_logger ?

Copy link
Contributor Author

@arkjedrz arkjedrz Jan 7, 2026

Choose a reason for hiding this comment

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

Won't work without them.

MODULE.bazel
commit = "5a70133dd8bd632f5c07f200a5ee4bc9f507c23b",
remote = "https://github.com/eclipse-score/communication.git",
)

Copy link
Contributor

@pawelrutkaq pawelrutkaq Jan 7, 2026

Choose a reason for hiding this comment

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

we wait with PR until logging is fixed.

@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch 5 times, most recently from b065c66 to ba914d3 Compare January 8, 2026 09:55
@arkjedrz arkjedrz requested a review from vinodreddy-g January 8, 2026 13:53
@arkjedrz arkjedrz self-assigned this Jan 12, 2026
@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch 2 times, most recently from 3594ed5 to 18dc8b8 Compare January 15, 2026 08:19
@arkjedrz arkjedrz mentioned this pull request Jan 15, 2026
Closed
vinodreddy-g
vinodreddy-g reviewed Jan 16, 2026
View reviewed changes
src/rust/rust_kvs/Cargo.toml Outdated
[dependencies]
adler32.workspace = true
tinyjson.workspace = true
mw_log.workspace = true
Copy link
Contributor

@vinodreddy-g vinodreddy-g Jan 16, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

the build with cargo only works with stdlogger right?

Copy link
Contributor Author

@arkjedrz arkjedrz Jan 19, 2026

Choose a reason for hiding this comment

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

Yes, but to be clear - this is not due to some issues or limitations. stdout_logger is just the only implementation that exists and interoperates with Cargo.
rust_kvs does not specify backend, because it can work with any.

@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch from 18dc8b8 to c2eabd7 Compare January 19, 2026 13:48
@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch 3 times, most recently from 3394468 to 00dcfad Compare January 20, 2026 07:28
@arkjedrz
impl: use score_log for logs
ac5219a 
- Use `stdout_logger` for Cargo builds.
- Use `score_log_bridge` for Bazel builds.
- Additional logs from Rust implementation.
- Use proxy module for logging with correct context across the lib.
- Fix tests.
- Bump toolchain version.
@arkjedrz arkjedrz force-pushed the arkjedrz_enable-logging-2 branch from 00dcfad to ac5219a Compare January 20, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vinodreddy-g vinodreddy-g Awaiting requested review from vinodreddy-g

@pawelrutkaq pawelrutkaq Awaiting requested review from pawelrutkaq

At least 1 approving review is required to merge this pull request.

Assignees

@arkjedrz arkjedrz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@arkjedrz @vinodreddy-g @pawelrutkaq