Skip to content

Conversation

@eranra
Copy link
Collaborator

@eranra eranra commented Apr 3, 2022

No description provided.

@eranra eranra self-assigned this Apr 3, 2022
@eranra eranra linked an issue Apr 3, 2022 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

Merging #169 (278efdb) into main (7af7877) will decrease coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #169      +/-   ##
==========================================
- Coverage   58.61%   58.32%   -0.30%     
==========================================
  Files          53       54       +1     
  Lines        3064     3105      +41     
==========================================
+ Hits         1796     1811      +15     
- Misses       1149     1173      +24     
- Partials      119      121       +2     
Flag Coverage Δ
unittests 58.32% <ø> (-0.30%) ⬇️

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

Impacted Files Coverage Δ
cmd/flowlogs-pipeline/main.go 0.00% <ø> (ø)
pkg/test/utils.go 66.66% <0.00%> (-0.30%) ⬇️
pkg/pipeline/utils/convert.go 18.18% <0.00%> (ø)
pkg/pipeline/extract/aggregate/aggregate.go 95.08% <0.00%> (+0.21%) ⬆️
pkg/pipeline/encode/encode_prom.go 81.85% <0.00%> (+0.77%) ⬆️

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 7af7877...278efdb. Read the comment docs.

@eranra eranra requested a review from KalmanMeth April 3, 2022 08:22
@eranra eranra merged commit 0bc0398 into netobserv:main Apr 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix the CLI title text

3 participants