8000 Add functionality for retrieving all tasks of a cgroup by gpanouts · Pull Request #67 · containerd/cgroups · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add functionality for retrieving all tasks of a cgroup #67

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

Merged
merged 1 commit into from
Nov 21, 2018

Conversation

gpanouts
Copy link
Contributor

So that we can get all tasks of a certain cgroup when we need to. Until now it has been possible to get only processes of that cgoup.

@crosbymichael
@estesp

Signed-off-by: Georgia Panoutsakopoulou <gpanoutsak@gmail.com>
@codecov-io
Copy link

Codecov Report

Merging #67 into master will increase coverage by 0.59%.
The diff coverage is 69.38%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #67      +/-   ##
=========================================
+ Coverage    13.7%   14.3%   +0.59%     
=========================================
  Files          23      23              
  Lines        4530    4579      +49     
=========================================
+ Hits          621     655      +34     
- Misses       3795    3803       +8     
- Partials      114     121       +7
Impacted Files Coverage Δ
cgroup.go 51.36% <62.06%> (+1.17%) ⬆️
utils.go 68.91% <80%> (+1.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82cb49f...a31a0ff. Read the comment docs.

@crosbymichael
Copy link
Member

LGTM

Thanks @gpanouts , code looks great!

Copy link
Member
@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

4 participants
0