Closed jotak closed 4 weeks ago
New image: quay.io/netobserv/flowlogs-pipeline:dfe4fdf
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=dfe4fdf make set-flp-image
Attention: Patch coverage is 64.33566%
with 51 lines
in your changes missing coverage. Please review.
Project coverage is 65.34%. Comparing base (
5034e92
) to head (ff6e56a
). Report is 2 commits behind head on main.
https://github.com/netobserv/flowlogs-pipeline/pull/674/commits/f39388d718f3a6714b8103160e2e3d0f7d6cf7de ^ checking on mac addresses when possible
thanks @jpinsonneau ! /lgtm
@jotak: you cannot LGTM your own PR.
@jotak: This pull request references NETOBSERV-1790 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.
@jotak: This pull request references NETOBSERV-1790 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.
/ok-to-test
New image: quay.io/netobserv/flowlogs-pipeline:f457772
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=f457772 make set-flp-image
/ok-to-test
New image: quay.io/netobserv/flowlogs-pipeline:6e9178a
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=6e9178a make set-flp-image
it's working well to enrich traffic for secondary interfaces.
/label qe-approved
@jotak: This pull request references NETOBSERV-1790 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.
@jotak: This pull request references NETOBSERV-1790 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jotak
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Description
Manage enrichment by extracting pod IPs from the annotation "k8s.v1.cni.cncf.io/network-status", which is used (at least) by multus
This allows to correlate Pods with their Mac / IPs on secondary interfaces
Breaking change
The API for
add_kubernetes_infra
transform stage has been modified to directly use enriched name & namespace to figure out the layer, instead of doing an informer lookup by IP. The reason is informers can now index by MAC.To migrate, if
add_kubernetes_infra
rule was used such as:it must be changed such as:
where SrcK8S_Namespace, SrcK8S_Name, DstK8S_Namespace and DstK8S_Name match the names of the corresponding enriched fields.
Dependencies
https://github.com/netobserv/network-observability-operator/pull/732 for Mac enrichment
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.