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

Lxc list display number of processes inside of container #4142

Merged
merged 3 commits into from Jan 8, 2018

Conversation

3 participants
@dnegreira
Collaborator

dnegreira commented Jan 6, 2018

Title is self-explanatory :)

Reason to have this done is that I saw this post: https://discuss.linuxcontainers.org/t/lxc-list-column-for-process-number/1017/4 and seemed pretty easy to implement.

ubuntu@lxd-dev:~/go/bin$ ./lxc info c1 | grep Proc
  Processes: 26
ubuntu@lxd-dev:~/go/bin$ ./lxc list -c n,N
+------+-----------+
| NAME | PROCESSES |
+------+-----------+
| c1   | 26        |
+------+-----------+
ubuntu@lxd-dev:~/go/bin$ 

dnegreira added some commits Jan 6, 2018

lxc_list: Option to display number of processes
Signed-off-by: David Negreira <David@otherreality.net>
lxc_list_test: added N option to tests
Signed-off-by: David Negreira <David@otherreality.net>
@lxc-jenkins

This comment has been minimized.

Show comment
Hide comment
@lxc-jenkins

lxc-jenkins Jan 6, 2018

This pull request didn't trigger Jenkins as its author isn't in the whitelist.

An organization member must perform one of the following:

  • To have this branch tested by Jenkins, use the "ok to test" command.
  • To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

lxc-jenkins commented Jan 6, 2018

This pull request didn't trigger Jenkins as its author isn't in the whitelist.

An organization member must perform one of the following:

  • To have this branch tested by Jenkins, use the "ok to test" command.
  • To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

@stgraber

This comment has been minimized.

Show comment
Hide comment
@stgraber

stgraber Jan 8, 2018

Member

This needs an update of the i18n templates, I'll push it to your branch directly.

Member

stgraber commented Jan 8, 2018

This needs an update of the i18n templates, I'll push it to your branch directly.

i18n: Update translation templates
Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
@stgraber

This comment has been minimized.

Show comment
Hide comment
@stgraber

stgraber Jan 8, 2018

Member

jenkins: test this please

Member

stgraber commented Jan 8, 2018

jenkins: test this please

@stgraber stgraber merged commit fc2de94 into lxc:master Jan 8, 2018

4 of 5 checks passed

Testsuite Testsuite failed
Details
Branch target Branch target is correct
Details
Signed-off-by All commits signed-off
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@dnegreira dnegreira deleted the dnegreira:lxc_list_display_process branch Jan 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment