Controllers cmd help change. #6457

Merged
merged 3 commits into from Oct 17, 2016

Conversation

Projects
None yet
3 participants
Member

anastasiamac commented Oct 17, 2016

There is potential for user confusion about how to update controllers information when listing controllers.
We do guide users to "Use --refresh to see the latest information."
However, sometimes users can interprete it as a direction to run "juju --refresh" (see https://bugs.launchpad.net/bugs/1632699)

This PR attempts to make obvious that --refresh is a flag on "juju controllers" command not "juju" itself.

Member

anastasiamac commented Oct 17, 2016

$$merge$$

Contributor

jujubot commented Oct 17, 2016

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

Contributor

jujubot commented Oct 17, 2016

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

Member

anastasiamac commented Oct 17, 2016

$$merge$$

Contributor

jujubot commented Oct 17, 2016

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

@jujubot jujubot merged commit 632a66a into juju:develop Oct 17, 2016

@anastasiamac anastasiamac deleted the anastasiamac:controllers-refresh-clarification branch Oct 17, 2016

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