netobserv / flowlogs-pipeline

Transform flow logs into metrics
Apache License 2.0
75 stars 23 forks source link

NETOBSERV-1522 Add otel to pipeline builder #672

Closed jpinsonneau closed 3 months ago

jpinsonneau commented 3 months ago

Description

This is just background code not requiring testing:

Dependencies

n/a

Checklist

If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.

Project coverage is 66.28%. Comparing base (6f7ae99) to head (4ccec33). Report is 2 commits behind head on main.

:exclamation: Current head 4ccec33 differs from pull request most recent head 56a8aa7

Please upload reports for the commit 56a8aa7 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #672 +/- ## ========================================== - Coverage 66.49% 66.28% -0.21% ========================================== Files 104 104 Lines 6658 6679 +21 ========================================== Hits 4427 4427 - Misses 1916 1937 +21 Partials 315 315 ``` | [Flag](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/672/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/672/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv) | `66.28% <0.00%> (-0.21%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/672?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv) | Coverage Δ | | |---|---|---| | [pkg/config/stage\_params.go](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/672?src=pr&el=tree&filepath=pkg%2Fconfig%2Fstage_params.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv#diff-cGtnL2NvbmZpZy9zdGFnZV9wYXJhbXMuZ28=) | `72.22% <0.00%> (-14.45%)` | :arrow_down: | | [pkg/config/pipeline\_builder.go](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/672?src=pr&el=tree&filepath=pkg%2Fconfig%2Fpipeline_builder.go&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv#diff-cGtnL2NvbmZpZy9waXBlbGluZV9idWlsZGVyLmdv) | `58.76% <0.00%> (-10.75%)` | :arrow_down: |
openshift-ci[bot] commented 3 months ago

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/netobserv/flowlogs-pipeline/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment