/juju

Asterisk default budget #7195

Merged
merged 1 commit into from Apr 5, 2017

Conversation

Reviewers

@mattyw mattyw

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@cmars @jujubot @mattyw
@cmars
Owner

cmars commented Apr 3, 2017 •
Edited 1 time

Description of change

Why is this change needed?

So that the user can see which budget is the default when not specified.

QA steps

How do we verify that the change works?

In the tabular output from juju budgets, the default budget will have an asterisk next to it. In the JSON output, it will have a default: true attribute.

Documentation changes

Does it affect current user workflow? CLI? API?

This is a minor UX improvement to make the output more intuitive. It does not change the user workflow.

Bug reference

Does this change fix a bug? Please add a link to it.

N/A

@cmars cmars changed the title from Asterisk default budget to WIP: Asterisk default budget Apr 3, 2017

@cmars cmars changed the base branch from staging to develop Apr 5, 2017

@cmars cmars changed the title from WIP: Asterisk default budget to Asterisk default budget Apr 5, 2017

@cmars
Owner

cmars commented Apr 5, 2017

!!build!!
@cmars
Owner

cmars commented Apr 5, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Apr 5, 2017

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

@jujubot jujubot merged commit c6a9e79 into juju:develop Apr 5, 2017
1 check failed

1 check failed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment