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

Fix host argument #103

Merged
merged 1 commit into from
Mar 29, 2014
Merged

Fix host argument #103

merged 1 commit into from
Mar 29, 2014

Conversation

janstenpickle
Copy link
Contributor

Quick fix, the host argument on container start should be 'hostname'.

@bflad
Copy link
Contributor

bflad commented Mar 28, 2014

Are you sure? Docs say otherwise: http://docs.docker.io/en/latest/reference/commandline/cli/#run

 -h, --host="": Container host name

@janstenpickle
Copy link
Contributor Author

Yep, looks like their docs are out of date! I'll raise an issue with docker too.

#docker run

Usage: docker run [OPTIONS] IMAGE [COMMAND] [ARG...]

Run a command in a new container

  -P, --publish-all=false: Publish all exposed ports to the host interfaces
  -a, --attach=[]: Attach to stdin, stdout or stderr.
  -c, --cpu-shares=0: CPU shares (relative weight)
  --cidfile="": Write the container ID to the file
  -d, --detach=false: Detached mode: Run container in the background, print new container id
  --dns=[]: Set custom dns servers
  -e, --env=[]: Set environment variables
  --entrypoint="": Overwrite the default entrypoint of the image
  --expose=[]: Expose a port from the container without publishing it to your host
  -h, --hostname="": Container host name
  -i, --interactive=false: Keep stdin open even if not attached
  --link=[]: Add link to another container (name:alias)
  --lxc-conf=[]: Add custom lxc options -lxc-conf="lxc.cgroup.cpuset.cpus = 0,1"
  -m, --memory="": Memory limit (format: <number><optional unit>, where unit = b, k, m or g)
  -n, --networking=true: Enable networking for this container
  --name="": Assign a name to the container
  -p, --publish=[]: Publish a container's port to the host (format: ip:hostPort:containerPort | ip::containerPort | hostPort:containerPort) (use 'docker port' to see the actual mapping)
  --privileged=false: Give extended privileges to this container
  --rm=false: Automatically remove the container when it exits (incompatible with -d)
  --sig-proxy=true: Proxify all received signal to the process (even in non-tty mode)
  -t, --tty=false: Allocate a pseudo-tty
  -u, --user="": Username or UID
  -v, --volume=[]: Bind mount a volume (e.g. from the host: -v /host:/container, from docker: -v /container)
  --volumes-from=[]: Mount volumes from the specified container(s)
  -w, --workdir="": Working directory inside the container

@bflad
Copy link
Contributor

bflad commented Mar 29, 2014

Okay, will get that in the next bugfix release.

bflad added a commit that referenced this pull request Mar 29, 2014
@bflad bflad merged commit 025c622 into sous-chefs:master Mar 29, 2014
@janstenpickle
Copy link
Contributor Author

Awesome, thanks very much!

@janstenpickle janstenpickle deleted the container_hostname branch March 30, 2014 16:10
@janstenpickle janstenpickle restored the container_hostname branch March 30, 2014 16:10
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