Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Conversation

crosbymichael
Copy link
Contributor

This also changes the functionality of the default exec in to just be an
existing func that is called than handles the implementation to exec a
user user's process inside the container. This implements this
functionallity in nsinit but is a base for how we will be handling these
types of features inside docker.

Output should look like:

0ff591c518ad241ed1d3e9a7c587505c0d6faa569d67b9301fe571d81683828b|⇒ nsinit exec --list
NAME      USAGE
exec      execute a process inside an existing container
mknod     mknod a device inside an existing container
ip        display the container's network interfaces
0ff591c518ad241ed1d3e9a7c587505c0d6faa569d67b9301fe571d81683828b|⇒ nsinit exec --func ip
NAME      MTU       MAC       FLAG          ADDRS
lo        65536               up|loopback   127.0.0.1/8,::1/128
0ff591c518ad241ed1d3e9a7c587505c0d6faa569d67b9301fe571d81683828b|⇒ nsinit exec sh
/ $ ps aux
PID   USER     COMMAND
    1 daemon   sh
   11 daemon   sh
   13 daemon   ps aux
/ $

Closes #7

Michael Crosby added 4 commits August 12, 2014 10:45
Signed-off-by: Michael Crosby <[email protected]>
Signed-off-by: Michael Crosby <[email protected]>
This also changes the functionality of the default exec in to just be an
existing func that is called than handles the implementation to exec a
user user's process inside the container.  This implements this
functionallity in nsinit but is a base for how we will be handling these
types of features inside docker.

Signed-off-by: Michael Crosby <[email protected]>
@crosbymichael
Copy link
Contributor Author

ping @calfonso @vishh @imain

@mrunalp
Copy link
Contributor

mrunalp commented Aug 12, 2014

Reviewing.

Copy link
Contributor

Choose a reason for hiding this comment

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

Neat! :)

@mrunalp
Copy link
Contributor

mrunalp commented Aug 13, 2014

+1 to reusing the exec flag.

@crosbymichael
Copy link
Contributor Author

@mrunalp done

@mrunalp
Copy link
Contributor

mrunalp commented Aug 13, 2014

LGTM.

@crosbymichael
Copy link
Contributor Author

thanks for the review

crosbymichael pushed a commit that referenced this pull request Aug 13, 2014
Implement execin by using registered functions
@crosbymichael crosbymichael merged commit 1befa2f into docker-archive:master Aug 13, 2014
@crosbymichael crosbymichael deleted the execin-func branch August 13, 2014 01:09
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.

2 participants