Skip to content
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

Support container event to denote container starts #550

Merged
merged 3 commits into from
Mar 8, 2019
Merged

Conversation

mstemm
Copy link
Contributor

@mstemm mstemm commented Mar 7, 2019

Add support for a "container" event as an alternate way to note that a container was created. Related to sysdig changes draios/sysdig#1326 that create that event.

@mstemm mstemm changed the title Use TBB_INCLUDE_DIR for consistency w sysdig,agent Support container event to denote container starts Mar 7, 2019
@mstemm mstemm requested a review from mattpag March 7, 2019 04:51
mstemm and others added 3 commits March 7, 2019 20:24
Previously it was a mix of TBB_INCLUDE and TBB_INCLUDE_DIR.
Create a new macro "container_started" to check both the old and
the new check.
Also, only look for execve exit events with vpid=1.
@mstemm mstemm force-pushed the async-container-info branch from 9e7d061 to 90657b4 Compare March 8, 2019 04:24
@mstemm mstemm merged commit 5740186 into dev Mar 8, 2019
@mstemm mstemm deleted the async-container-info branch March 8, 2019 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants