~pjdc/prometheus-mongodb-exporter-charm/+git/prometheus-mongodb-exporter-charm:no-needless-blocked

Branch merges

Propose for merging

Rejected for merging into prometheus-mongodb-exporter-charm:master

Stuart Bishop (community): Needs Fixing on 2019-09-04

Canonical IS Reviewers: Pending requested 2019-09-04

Diff: 57 lines (+21/-4)

1 file modified

reactive/prometheus_mongodb_exporter.py (+21/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

681810e... by Paul Collins on 2019-09-04

ensure installation happens immediately after setting mongodb_uri to juju-db://

Previously, install_prometheus_mongodb_exporter would bail when
the config item was "juju-db://" but the leader setting was empty,
because create_juju_db_user had not yet been invoked.

b02f55c... by Paul Collins on 2019-09-02

support mongodb_uri=juju-db:// for Juju controllers

Reviewed-on: https://code.launchpad.net/~pjdc/prometheus-mongodb-exporter-charm/+git/prometheus-mongodb-exporter-charm/+merge/371776
Reviewed-by: Stuart Bishop <email address hidden>

c082062... by Paul Collins on 2019-08-26

maybe_monitor_juju_db: config.changed not config.set

This handler will now no longer run on every hook.

The downside is that an existing deployment with mongodb_uri
already set to juju-db:// will not get the state set, but
after I destroy my test models there will be zero of those.

5f5f360... by Paul Collins on 2019-08-26

add maybe_monitor_juju_db, use the new state to gate create_juju_db_user

c14b12e... by Paul Collins on 2019-08-26

mark blocked if we can't create the user due to NotMasterError

The cluster is probably broken or in transition.

ce454d9... by Paul Collins on 2019-08-26

remove pointless except+raise

7372b07... by Paul Collins on 2019-08-26

disable ssl_match_hostname since Juju's server.pem no longer includes member IPs

7fea7f8... by Paul Collins on 2019-08-26

[r=hloeung] add bionic to supported series

6a82b49... by Paul Collins on 2019-08-26

use juju CA and cert when mongodb_uri=juju-db://

a48cb68... by Paul Collins on 2019-08-25

support mongodb_uri=juju-db:// for Juju controllers