xendk/dce

Run shell commands in docker

dce's Issues

Piping into containers doesn't work anymore

It's caused by switching to docker-compose exec rather than docker exec. The -T switch apparently causes docker-compose to not wire stdin to the command.

It's a known issue: docker/compose#3352

Our only option is reverting to using docker exec, but can improve on guessing the container name by using docker-compose ps -q <service>.

Weird error messages when containers doesn't exists.

dce in a project where docker-compose up has never been run results in Error response from daemon: No such container: sh

Secondly Error response from daemon: Container 5eca8ad2063a2811e01ea4ee8d08955eeee9c1d933671d647190bc43fb45b379 is not running could be nicer.

Switch/command to copy ssh keys into container

On Linux it's quite easy to mount your ssh-agent socket into a container, to make ssh'ing easy.

On OSX it's not quite as simple. The agents default socket path lies outside the part of the filesystem mounted into the VM running docker, and it's unclear whether the socket would actually work.

A workaround is a few simple shell commands to copy the keys in. To make life easier for the OSX crowd, dce should provide a switch for it. Optionally one that ensures the key(s) are present in the container before running the supplied command.

Default to opening a shell if no command specified

Basically having a default command of if [ -e /bin/bash ]; then /bin/bash; else /bin/sh;fi.

As the current behavior is to run an empty command, which sh will do successfully, it'll probably not trip up people.

And people doing dce bash will get a new option that doesn't fail when the container doesn't have bash.

Then there's no need for the dcs alias.

-c switch for specifying command

Aliases like alias dcd="dce drush" has the issue that one cannot add any more dce switches (-c, -v, etc.) to the run command.

Adding a -c switch for specifying the "base" command would allow the argument parser to continue processing arguments after the command.

For extra points, the -c argument should support a placeholder like {}, to allow for injecting the command supplied at a particular place in the base command, allowing for alias dcl="dce -c "ls {} | sort", which could still be called as dcl -c container /tmp.

Escaping the {} would be an issue though, adding another switch (say -I to mimic xargs) to allow for specifying the string to be replaced is probably the best bet.

Fix $TERM when running commands.

Apparently the docker devs think everybody uses a xterm compatible terminal: https://github.com/moby/moby/pull/26461/files#diff-3ed197c0a6f7a9ca73a3ee424e448ac1R130

Prepending export TERM=<$TERM from the env> and most likely TERMCAP as well should fix that.

For extra niceness we can add stty rows $(tput lines) cols $(tput cols) to propagate window size.