Closed jpinsonneau closed 3 months ago
Attention: Patch coverage is 38.57143%
with 172 lines
in your changes are missing coverage. Please review.
Project coverage is 34.04%. Comparing base (
6e00a92
) to head (df212a7
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/retest
Rebased + removed tcp packet capture https://github.com/netobserv/netobserv-ebpf-agent/pull/291/commits/6e65e79a0050842f22a82466d71bba97e02550ed
I am good with the changes few nit left other than that LGTM up to u fixing them /LGTM
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: msherif1234
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Description
Add gRPC export capability for packet capture. Since I moved flow grpc to
flowgrpc
package, we will need to update FLP at the same timeI also took the opportunity to reuse the target host / port from flow config since it doesn't make sense to have both here.
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.