feat: Include rootfs in the Exec
event
#291
Closed
+23
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
image
field of theExec
event is relative to the root filesystem of the process. That makes it hard to determine the location of the binary if the process is containerized (or, in general, runs inside a mount namespace).That's where the new
rootfs
comes with help. It contains the full path to the root filesystem, which in case of containers has format like:Using that information, we can still inspect that container layer even when the process (or even container) are not running anymore.
Pull Request Title
Short introduction explaining the motivation and reasoning behind the pull request.
Implementation (Optional)
Feel free to include design and implementation for external review outside of the code changes.
I have
cargo fmt
;cargo clippy
;cargo test
and all tests pass;