/juju

mongo/mongometrics: one package to rule them all #7551

Merged
merged 1 commit into from Jun 23, 2017

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@axw @jujubot @wallyworld
@axw
Member

axw commented Jun 23, 2017

Description of change

Move the existing MongoDB-related Prometheus collectors into a
single new package, and introduce a new collector that tracks
MongoDB server dials. Add unit tests for the mgo stats collector,
and change the "sent ops", "received ops" and "received docs"
metrics to counters.

QA steps

  1. juju bootstrap localhost
  2. juju ssh -m controller, edit agent.conf setting MGO_STATS_ENABLED=true
  3. juju ssh -m controller juju-introspect metrics
    (check mongo-related metrics in output)

Documentation changes

None.

Bug reference

None.
@axw
mongo/mongometrics: one package to rule them all 
Move the existing MongoDB-related Prometheus collectors into a
single new package, and introduce a new collector that tracks
MongoDB server dials. Add unit tests for the mgo stats collector,
and change the "sent ops", "received ops" and "received docs"
metrics to counters.
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
ec3095a
@axw
Member

axw commented Jun 23, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Jun 23, 2017

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

@jujubot jujubot merged commit 07b656c into juju:2.2 Jun 23, 2017
1 check failed

1 check failed

github-check-merge-juju Ran tests against PR. 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