-
Notifications
You must be signed in to change notification settings - Fork 236
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
Added ebps support(based on runc implementation) #116
Conversation
@AkihiroSuda Do we need allowed devices list here? |
The current eBPF code already supports allowing specific devices |
Codecov Report
@@ Coverage Diff @@
## master #116 +/- ##
=========================================
+ Coverage 35.46% 36.56% +1.1%
=========================================
Files 32 34 +2
Lines 2154 2297 +143
=========================================
+ Hits 764 840 +76
- Misses 1262 1324 +62
- Partials 128 133 +5
Continue to review full report at Codecov.
|
I'm still wondering devices should be under this structure: https://godoc.org/github.com/containerd/cgroups/v2#Resources |
Can you rebase this one as well? |
383624f
to
1820009
Compare
@crosbymichael Done |
rebase:) |
Signed-off-by: bpopovschi <zyqsempai@mail.ru>
Signed-off-by: bpopovschi <zyqsempai@mail.ru>
Signed-off-by: bpopovschi <zyqsempai@mail.ru>
Signed-off-by: bpopovschi <zyqsempai@mail.ru>
Signed-off-by: bpopovschi <zyqsempai@mail.ru>
4df6a7b
to
dea9e11
Compare
@crosbymichael Done |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Partially Fixes #104
Added eBPS support(based on @AkihiroSuda implementation for runc)
Signed-off-by: bpopovschi zyqsempai@mail.ru