Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Aug 26, 2022. It is now read-only.

Conversation

@luiz-couto
Copy link
Contributor

Description

Kind: Feature

Section: Sidecar

Summary: Created metrics for sidecar

Changelog

  • Added two summary metrics for the sidecar

@luiz-couto luiz-couto requested a review from a team as a code owner September 14, 2021 14:49
@luiz-couto luiz-couto changed the title dev (kafka sc): throughput metrics Story CORE-563 | Develop sidecar metrics Sep 14, 2021
@codecov
Copy link

codecov bot commented Sep 14, 2021

Codecov Report

Merging #128 (029ed8a) into develop (964f017) will increase coverage by 0.31%.
The diff coverage is 85.71%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #128      +/-   ##
===========================================
+ Coverage    75.05%   75.36%   +0.31%     
===========================================
  Files          116      116              
  Lines         6538     6544       +6     
===========================================
+ Hits          4907     4932      +25     
+ Misses        1391     1370      -21     
- Partials       240      242       +2     
Flag Coverage Δ
unittests 75.36% <85.71%> (+0.31%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/sidecars/server/handlers.go 50.60% <66.66%> (-6.55%) ⬇️
pkg/sidecars/server/server.go 78.94% <100.00%> (+28.42%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 964f017...029ed8a. Read the comment docs.

@luiz-couto luiz-couto merged commit a59ea5a into develop Sep 14, 2021
@luiz-couto luiz-couto deleted the story/CORE-563 branch September 14, 2021 15:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants