Closed msherif1234 closed 3 months ago
@msherif1234: This pull request references NETOBSERV-1732 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 bug to target the "4.17.0" version, but no target version was set.
/ok-to-test
New image: quay.io/netobserv/netobserv-ebpf-agent:29b9188
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=29b9188 make set-agent-image
/ok-to-test
New image: quay.io/netobserv/netobserv-ebpf-agent:846d29a
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=846d29a make set-agent-image
Attention: Patch coverage is 0%
with 52 lines
in your changes missing coverage. Please review.
Please upload report for BASE (
main@484bc41
). Learn more about missing BASE report. Report is 3 commits behind head on main.
@msherif1234 I thought it was already managed e.g. here: https://github.com/netobserv/netobserv-ebpf-agent/blob/main/pkg/ebpf/tracer.go#L308 , what's the difference exactly with the existing deletions?
Is it because previously it would only delete for the current interfaces, but not if interfaces have changed in the meantime?
@msherif1234 I thought it was already managed e.g. here: https://github.com/netobserv/netobserv-ebpf-agent/blob/main/pkg/ebpf/tracer.go#L308 , what's the difference exactly with the existing deletions?
Is it because previously it would only delete for the current interfaces, but not if interfaces have changed in the meantime?
current logic won't work as when the pod restart the FD value won't be the same as old program so the existing code won't ever kick in https://github.com/netobserv/netobserv-ebpf-agent/blob/main/pkg/ebpf/tracer.go#L304
/ok-to-test
New image: quay.io/netobserv/netobserv-ebpf-agent:a705627
It will expire after two weeks.
To deploy this build, run from the operator repo, assuming the operator is running:
USER=netobserv VERSION=a705627 make set-agent-image
/label no-qe
/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
Today if ebpf agent pod is killed with
SIGKILL
the clean up code never get a chance to remove all TC filters and qdisc installed this PR check all previously installed TC filters and remove themDependencies
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.