/juju

Show model SLA owner #7202

Merged
merged 1 commit into from Apr 6, 2017

Conversation

Reviewers

@mjs mjs

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

cmars commented Apr 5, 2017 •
Edited 1 time

Description of change

Why is this change needed?

A user needs to be able to view the owner of the SLA on a model.

QA steps

How do we verify that the change works?

As user "bob":

juju add-model testmodel
juju sla essential
juju show-model testmodel

...

sla: essential
sla-owner: bob

Documentation changes

Does it affect current user workflow? CLI? API?

Yes, it makes the SLA information on a model more observable.

Bug reference

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

N/A

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

@cmars cmars referenced this pull request in juju/description Apr 5, 2017

Merged

Add SLA owner. #11

@cmars
Store the SLA owner when SLA is set. 
Display SLA and SLA owner in `juju show-model` output.
f778110

@cmars cmars changed the title from WIP: Show model sla owner to Show model SLA owner Apr 6, 2017

@mjs

mjs approved these changes Apr 6, 2017
View changes

Pretty straightforward.

@cmars
Owner

cmars commented Apr 6, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Apr 6, 2017

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

@jujubot jujubot merged commit 521accf into juju:develop Apr 6, 2017

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