grabana
grabana copied to clipboard
adding cloudwatch target
This will address the feature requested here
Codecov Report
Merging #227 (8e7d9ae) into master (6867a4a) will increase coverage by
0.15%
. Report is 1 commits behind head on master. The diff coverage is100.00%
.
Additional details and impacted files
@@ Coverage Diff @@
## master #227 +/- ##
==========================================
+ Coverage 90.08% 90.24% +0.15%
==========================================
Files 92 93 +1
Lines 6941 7051 +110
==========================================
+ Hits 6253 6363 +110
+ Misses 543 537 -6
- Partials 145 151 +6
Files Changed | Coverage Δ | |
---|---|---|
decoder/gauge.go | 64.12% <100.00%> (+0.84%) |
:arrow_up: |
decoder/graph.go | 64.32% <100.00%> (+0.63%) |
:arrow_up: |
decoder/heatmap.go | 82.14% <100.00%> (+0.49%) |
:arrow_up: |
decoder/singlestat.go | 80.16% <100.00%> (+0.50%) |
:arrow_up: |
decoder/stat.go | 61.76% <100.00%> (+0.68%) |
:arrow_up: |
decoder/table.go | 81.25% <100.00%> (+1.25%) |
:arrow_up: |
decoder/target.go | 99.52% <100.00%> (+0.02%) |
:arrow_up: |
decoder/timeseries.go | 73.80% <100.00%> (+0.23%) |
:arrow_up: |
gauge/gauge.go | 100.00% <100.00%> (ø) |
|
graph/graph.go | 98.42% <100.00%> (+0.05%) |
:arrow_up: |
... and 6 more |
I realized there's more work to be done to support cloudwatch target. I will this PR soon once the changes in my local are working as expected.
This PR is ready for review.
@K-Phoen Would really appreciate your feedback on this PR. Thank you.
@slimsag Just checking: Would you be able to take a look at this one? 🙏🏼
Any news?