Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Include ps command #81

Closed
devimc opened this issue Apr 13, 2018 · 4 comments
Closed

Include ps command #81

devimc opened this issue Apr 13, 2018 · 4 comments
Assignees
@devimc

Comments

@devimc
Copy link

devimc commented Apr 13, 2018 •
edited
Loading

kata-runtime should have an implementation similar to runc to list the processes inside the containers.

  • ps command should be included in all the images generated by osbuilder
  • document why ps command is needed
@devimc devimc self-assigned this Apr 13, 2018
@grahamwhaley
Copy link
Contributor

or presumably we could parse /proc ?

@devimc devimc mentioned this issue Apr 13, 2018
Merged
@devimc
Copy link
Author

devimc commented Apr 13, 2018

@grahamwhaley no we can't, user can specify ps options using docker top, if we read /proc then we have to support ALL ps options

$ docker top --help
Usage:	docker top CONTAINER [ps OPTIONS]

@devimc
Copy link
Author

devimc commented Apr 13, 2018

[ps OPTIONS]

$ docker top --help
Usage:	docker top CONTAINER [ps OPTIONS]

@grahamwhaley
Copy link
Contributor

Agreed. Thanks for explaining - I'd forgotten the user can supply options through docker top.

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

@devimc devimc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@grahamwhaley @devimc @jodh-intel