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

Support for lxc 1.0.0 #14

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

Support for lxc 1.0.0 #14

wants to merge 6 commits into from

Conversation

hamann
Copy link

@hamann hamann commented Mar 1, 2014

Hi,

this should add basic support for lxc 1.0.0.
With 1.0.0 the following things has changed:

  • lxc-version was removed and lxc-info --version has to be used
  • output of lxc-info -n foo has changed (state => State, pid => PID)
  • lxc-ps and lxc-netstat where removed, instead lxc-attach should be used, see mail

I also some removed unnecessary dependency checks for binaries which aren't provided by upstream lxc and aren't used in this gem, like setcap or setuid

Still open: reimplement Container.processes method to use lxc-attach -n foo -- ps <arguments>

Implemented and tested on debian wheezy 7.4

@sosedoff
Copy link
Owner

sosedoff commented Mar 2, 2014

I havent looked into new LXC version yet, so this PR will have to wait for a while.

@hamann
Copy link
Author

hamann commented Mar 6, 2014

The lxc guys created a ruby binding for lxc, and it works. Don't know if it makes sense to investigate more work on this issue.

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.

None yet

2 participants