juju run: accept multiple args for command #6930

Merged
merged 1 commit into from Feb 7, 2017

Conversation

Projects
None yet
3 participants
Member

axw commented Feb 7, 2017

Description of change

Update the "juju run" command so that it accepts
multiple positional arguments, and combines them
into a command for the remote targets to run.
Update the help text to explain how to run a
command that takes flags.

QA steps

  1. juju bootstrap localhost
  2. juju switch controller
  3. juju run --all 'hostname -f'
  4. juju run --all -- hostname -f
    Both commands should run successfully, and print out the FQDN of the controller instance.

Documentation changes

No change to existing usage.

Bug reference

Fixes https://bugs.launchpad.net/juju/+bug/1637819

juju run: accept multiple args for command
Update the "juju run" command so that it accepts
multiple positional arguments, and combines them
into a command for the remote targets to run.
Update the help text to explain how to run a
command that takes flags.

Fixes https://bugs.launchpad.net/juju/+bug/1637819

Looks kind of awesome \o/

Member

axw commented Feb 7, 2017

$$merge$$

Contributor

jujubot commented Feb 7, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Feb 7, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10221

Member

anastasiamac commented Feb 7, 2017

$$merge$$ since "sockets in a dirty state" was last failure

Contributor

jujubot commented Feb 7, 2017

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit aacad05 into juju:2.1 Feb 7, 2017

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