-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ig panics when it fails to create container-collection #2812
Comments
I tried to synthetise the error with: --- a/pkg/container-collection/options.go
+++ b/pkg/container-collection/options.go
@@ -672,7 +672,7 @@ func WithContainerFanotifyEbpf() ContainerCollectionOption {
}
return true
})
- return nil
+ return fmt.Errorf("lets fail on WithContainerFanotifyEbpf")
}
}
And then run:
So I get the warnings:
But I am not sure of the differences between LocalManager, localManagerTrace and localManagerTraceWrapper. |
Fixed by #2874 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
ig panics when it fails to create container-collection
Impact
Environment and steps to reproduce
I don't have the steps or the environment, I'll try to find more details.
Start ig in a Kubernetes pod:
Expected behavior
ig should not panic.
Additional information
https://github.com/inspektor-gadget/inspektor-gadget/blob/v0.27.0/pkg/operators/localmanager/localmanager.go#L532
Presumably,
l.manager.igManager.ContainerCollection
isnil
. The functionPreStart
should check this.cc @flyth
The text was updated successfully, but these errors were encountered: