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

packages: add procps-ng-bin package#41

Merged
jodh-intel merged 1 commit intoclearcontainers:masterfrom
devimc:packages/procps-ng-bin
Dec 5, 2017
Merged

packages: add procps-ng-bin package#41
jodh-intel merged 1 commit intoclearcontainers:masterfrom
devimc:packages/procps-ng-bin

Conversation

@devimc
Copy link
Copy Markdown

@devimc devimc commented Nov 28, 2017

procps-ng-bin includes the ps command that is needed to list the processes
inside the container and used by cc-runtime ps and docker top

fixes #40

Signed-off-by: Julio Montes julio.montes@intel.com

Copy link
Copy Markdown
Contributor

@jcvenegas jcvenegas left a comment

Choose a reason for hiding this comment

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

lgtm, just one question now is working ps command for james or it is need something else?

@jcvenegas
Copy link
Copy Markdown
Contributor

@devimc could you rebase master ? to have the PR passing again , @jodh-intel can you approve this PR if now ps command is working for you?

procps-ng-bin includes the ps command that is needed to list the processes
inside the container and used by ```cc-runtime ps``` and ```docker top```

fixes clearcontainers#40

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the packages/procps-ng-bin branch from af4afc9 to 0f6cf40 Compare December 4, 2017 19:54
@devimc
Copy link
Copy Markdown
Author

devimc commented Dec 4, 2017

@jcvenegas done

@jodh-intel
Copy link
Copy Markdown

Yep - clearcontainers/runtime#444 is now working for me \o/

lgtm

@jodh-intel jodh-intel merged commit 374da32 into clearcontainers:master Dec 5, 2017
jodh-intel pushed a commit to jodh-intel/osbuilder that referenced this pull request Aug 28, 2020
Require an additional approval from a `documentation` team member for
PRs containing documentation changes.

Fixes clearcontainers#41.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add procps-ng-bin package

3 participants