/juju

Fix region handling in model-defaults --reset #6442

Merged
merged 1 commit into from Oct 13, 2016

Conversation

Reviewers

@mjs mjs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@wallyworld @jujubot @mjs
@wallyworld
Owner

wallyworld commented Oct 13, 2016

A few fixes here:

  1. juju model-defaults region --reset foo
    Fixed to work properly with region specified. The fix was to revert a change made elsewhere in sysCmdWrapper
  2. Fix case of table headers in model-defaults command
  3. Improve error message for juju grant

Also added feature tests for model-defaults command to test the issue fixed.
@mjs

mjs approved these changes Oct 13, 2016
View changes

Nice

@wallyworld
Owner

wallyworld commented Oct 13, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Oct 13, 2016

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

@jujubot jujubot merged commit 12e978b into juju:master Oct 13, 2016

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