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

Multiple small improvements #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Multiple small improvements #1

wants to merge 2 commits into from

Conversation

ztmr
Copy link

@ztmr ztmr commented Sep 8, 2012

(1) cgroup path update (/sys/fs/cgroups);
(2) clear screen before each iteration;
(3) drop all the ancient records like temporary cgroup
data and stopped/destroyed containers;
(4) signal handling to avoid the ugly exception
when exitting via ^C interrupt;

Tomas Morstein added 2 commits September 9, 2012 00:20
(1) cgroup path update (/sys/fs/cgroups);
(2) clear screen before each iteration;
(3) drop all the ancient records like temporary cgroup
    data and stopped/destroyed containers;
(4) signal handling to avoid the ugly exception
    when exitting via ^C interrupt;
@astro
Copy link
Owner

astro commented Sep 9, 2012

Great that this little hack actually receives pull requests. Tell me if it's urgent, otherwise I'll merge it after my holiday.

@ztmr
Copy link
Author

ztmr commented Sep 9, 2012

No problem, I'll stay with my own fork ;-)

BTW, since LXC still has not so much good tools yet, I think it's valuable to share almost anything, even if it is only hack or seed for something more complex!

@ztmr
Copy link
Author

ztmr commented Sep 9, 2012

Hmm, I've just tested it on 3.2.x kernel and it seems the /sys/fs/cgroup tree structure has significantly changed :-(
On 2.6.38, it worked very well for me...

@astro
Copy link
Owner

astro commented Sep 11, 2012

Despite that the kernel side is being developed in mainline, I found lxc to be a very much moving target. My target is at least 3.x, with focus on the latest version (currently 3.5).

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