/juju

apiserver/metricsmanager: Send juju-machines metrics if sla is set #7110

Merged
merged 1 commit into from Mar 17, 2017

Conversation

Reviewers

@perrito666 perrito666

@alesstimec alesstimec

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@mattyw @jujubot @perrito666 @alesstimec
@mattyw
Member

mattyw commented Mar 16, 2017 •
Edited 1 time

Description of change

Adds the sending of "juju-machines" metrics if an sla has been set on the model

QA steps

# bootstrap a new model
juju sla essential
juju deploy ubuntu
# wait for the unit to come up
juju metrics --all
UNIT               TIMESTAMP                   METRIC                VALUE
                       2017-03-16T12:50:00Z    juju-machines       1

Documentation changes

There are no ui changes as a result of this change. But the sla scheme in general needs documenting

Bug reference

n/a
@mattyw
Member

mattyw commented Mar 17, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Mar 17, 2017

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

jujubot commented Mar 17, 2017

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10488
@mattyw
Member

mattyw commented Mar 17, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Mar 17, 2017

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

@jujubot jujubot merged commit e46ae8a into juju:rising-sun Mar 17, 2017

@mattyw mattyw deleted the mattyw:53-send-juju-machine-metrics branch Mar 17, 2017

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