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

Enable eBPF tracking by default #2535

Merged
merged 3 commits into from Jun 4, 2017
Merged

Enable eBPF tracking by default #2535

merged 3 commits into from Jun 4, 2017

Conversation

@2opremio
Copy link
Contributor

@2opremio 2opremio commented May 23, 2017

Fixes #2441

We already had an integration test for the proc fallback (313), so I just adjusted the ebpf flag according to its default value and changed tests explicitly testing ebpf-tracking to explicitly test tracking without ebpf.

@2opremio 2opremio force-pushed the 2441-enable-ebpf-by-default branch 2 times, most recently from 1b56042 to 71e0356 May 31, 2017
@2opremio 2opremio changed the title [WIP] Enable ebpf tracking by default Enable ebpf tracking by default May 31, 2017
@2opremio 2opremio changed the title Enable ebpf tracking by default Enable eBPF tracking by default May 31, 2017
@2opremio 2opremio requested a review from alban May 31, 2017
@2opremio
Copy link
Contributor Author

@2opremio 2opremio commented May 31, 2017

@alban PTAL (ignore the netlify error)

@2opremio 2opremio requested a review from ekimekim Jun 4, 2017
@2opremio 2opremio merged commit e4f5537 into master Jun 4, 2017
2 of 3 checks passed
2 of 3 checks passed
deploy/netlify Deploy preview failed.
Details
ci/circleci Your tests passed on CircleCI!
Details
coverage/coveralls Coverage increased (+0.02%) to 49.299%
Details
@2opremio 2opremio deleted the 2441-enable-ebpf-by-default branch Jun 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants