thin-edge / thin-edge.io

The open edge framework for lightweight IoT devices
https://thin-edge.io
Apache License 2.0
211 stars 55 forks source link

fix(#2370): use bridge topic id in service_monitor #2837

Open Bravo555 opened 2 months ago

Bravo555 commented 2 months ago

Proposed changes

Use the correct bridge topic id when converting health status message.

Types of changes

Paste Link to the issue

Checklist

Further comments

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 76.3%. Comparing base (6c5f45c) to head (98ccfe7). Report is 36 commits behind head on main.

Additional details and impacted files | [Files](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge) | Coverage Δ | | |---|---|---| | [crates/core/tedge\_api/src/mqtt\_topics.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837?src=pr&el=tree&filepath=crates%2Fcore%2Ftedge_api%2Fsrc%2Fmqtt_topics.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2NvcmUvdGVkZ2VfYXBpL3NyYy9tcXR0X3RvcGljcy5ycw==) | `87.4% <ø> (-0.2%)` | :arrow_down: | | [crates/extensions/c8y\_mapper\_ext/src/converter.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837?src=pr&el=tree&filepath=crates%2Fextensions%2Fc8y_mapper_ext%2Fsrc%2Fconverter.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvYzh5X21hcHBlcl9leHQvc3JjL2NvbnZlcnRlci5ycw==) | `83.3% <100.0%> (ø)` | | | [crates/extensions/c8y\_mapper\_ext/src/lib.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837?src=pr&el=tree&filepath=crates%2Fextensions%2Fc8y_mapper_ext%2Fsrc%2Flib.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvYzh5X21hcHBlcl9leHQvc3JjL2xpYi5ycw==) | `88.8% <ø> (ø)` | | | [...s/extensions/c8y\_mapper\_ext/src/service\_monitor.rs](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837?src=pr&el=tree&filepath=crates%2Fextensions%2Fc8y_mapper_ext%2Fsrc%2Fservice_monitor.rs&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge#diff-Y3JhdGVzL2V4dGVuc2lvbnMvYzh5X21hcHBlcl9leHQvc3JjL3NlcnZpY2VfbW9uaXRvci5ycw==) | `96.4% <100.0%> (+<0.1%)` | :arrow_up: | ... and [26 files with indirect coverage changes](https://app.codecov.io/gh/thin-edge/thin-edge.io/pull/2837/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=thin-edge)
github-actions[bot] commented 2 months ago

Robot Results

:white_check_mark: Passed :x: Failed :next_track_button: Skipped Total Pass % :stopwatch: Duration
429 0 3 429 100 0s