netobserv / flowlogs-pipeline

Transform flow logs into metrics
Apache License 2.0
77 stars 24 forks source link

NETOBSERV-1466 Update eBPF dependency for FLP #601

Closed jpinsonneau closed 8 months ago

jpinsonneau commented 9 months ago

Description

See eBPF update https://github.com/netobserv/netobserv-ebpf-agent/pull/275 and followup to remove Duplicate field https://github.com/netobserv/netobserv-ebpf-agent/pull/276

Dependencies

Update go mod once eBPF PR is merged

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.

jotak commented 8 months ago

/lgtm

github-actions[bot] commented 8 months ago

New image: quay.io/netobserv/flowlogs-pipeline:013a597

It will expire after two weeks.

To deploy this build, run from the operator repo, assuming the operator is running:

USER=netobserv VERSION=013a597 make set-flp-image
openshift-ci[bot] commented 8 months ago

New changes are detected. LGTM label has been removed.

openshift-ci[bot] commented 8 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
openshift-ci[bot] commented 8 months ago

New changes are detected. LGTM label has been removed.

codecov[bot] commented 8 months ago

Codecov Report

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

Project coverage is 67.20%. Comparing base (df307e0) to head (8f10ff5).

Files Patch % Lines
pkg/pipeline/write/write_ipfix.go 0.00% 6 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #601 +/- ## ======================================= Coverage 67.19% 67.20% ======================================= Files 104 104 Lines 7459 7461 +2 ======================================= + Hits 5012 5014 +2 - Misses 2144 2145 +1 + Partials 303 302 -1 ``` | [Flag](https://app.codecov.io/gh/netobserv/flowlogs-pipeline/pull/601/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/601/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=netobserv) | `67.20% <0.00%> (+<0.01%)` | :arrow_up: | 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.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.