shifu
shifu copied to clipboard
Added design for TDEngine integration
What this PR does / why we need it:
Design for integrate TDEngine
Will this PR make the community happier?
Yes
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes #
How is this PR tested
- [ ] unit test
- [ ] e2e test
- [x] other (please specify) Design doc
Special notes for your reviewer:
Release note:
Codecov Report
Base: 44.98% // Head: 49.61% // Increases project coverage by +4.63%
:tada:
Coverage data is based on head (
e5c8f7f
) compared to base (1571264
). Patch has no changes to coverable lines.
:exclamation: Current head e5c8f7f differs from pull request most recent head 1bc4967. Consider uploading reports for the commit 1bc4967 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## main #339 +/- ##
==========================================
+ Coverage 44.98% 49.61% +4.63%
==========================================
Files 14 16 +2
Lines 1436 1566 +130
==========================================
+ Hits 646 777 +131
+ Misses 719 712 -7
- Partials 71 77 +6
Impacted Files | Coverage Δ | |
---|---|---|
pkg/deviceshifu/utils/utils.go | 55.17% <0.00%> (-11.50%) |
:arrow_down: |
pkg/deviceshifu/deviceshifuhttp/deviceshifuhttp.go | 58.35% <0.00%> (-0.35%) |
:arrow_down: |
pkg/deviceshifu/deviceshifubase/push_telemetry.go | ||
pkg/telemetryservice/telemetryservice.go | 80.76% <0.00%> (ø) |
|
pkg/telemetryservice/mqtt/telemetryservicemqtt.go | 82.35% <0.00%> (ø) |
|
pkg/deviceshifu/deviceshifubase/pushtelemetry.go | 78.86% <0.00%> (ø) |
|
...iceshifu/deviceshifubase/deviceshifubase_config.go | 54.43% <0.00%> (+5.06%) |
:arrow_up: |
pkg/deviceshifu/deviceshifubase/deviceshifubase.go | 60.00% <0.00%> (+18.88%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.