Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Conversation

@alban
Copy link
Member

@alban alban commented Dec 14, 2015

Fixes #1868

@alban alban self-assigned this Dec 14, 2015
@alban alban added this to the v0.14.0 milestone Dec 14, 2015
@alban
Copy link
Member Author

alban commented Dec 14, 2015

/cc @jonboulle @yifan-gu @iaguis

Copy link
Member

Choose a reason for hiding this comment

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

What do you mean with (pod). Should the cgroups below also have (pod)?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll put (pod) on the etcd.service above.

But is this section necessary at all? Maybe I could move this in the sections below.

@alban alban force-pushed the alban/doc-cgroup branch 2 times, most recently from 21ed692 to 169c4ba Compare December 14, 2015 18:10
@alban
Copy link
Member Author

alban commented Dec 15, 2015

Branch updated.

@iaguis
Copy link
Member

iaguis commented Dec 15, 2015

LGTM

iaguis added a commit that referenced this pull request Dec 15, 2015
Documentation: cgroup doc for developers
@iaguis iaguis merged commit 28b303d into rkt:master Dec 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants