Merge ~brad-marshall/charm-prometheus-ceph-exporter/+git/prometheus-ceph-exporter-charm:fix-prometheus-target-relation into ~prometheus-charmers/charm-prometheus-ceph-exporter/+git/prometheus-ceph-exporter-charm:master

Proposed by Brad Marshall
Status: Merged
Approved by: James Hebden
Approved revision: 1f03753be0bdb2ef3ef6e58b457bd157be8e774a
Merged at revision: 1f03753be0bdb2ef3ef6e58b457bd157be8e774a
Proposed branch: ~brad-marshall/charm-prometheus-ceph-exporter/+git/prometheus-ceph-exporter-charm:fix-prometheus-target-relation
Merge into: ~prometheus-charmers/charm-prometheus-ceph-exporter/+git/prometheus-ceph-exporter-charm:master
Diff against target: 24 lines (+3/-3)
1 file modified
reactive/prometheus-ceph-exporter.py (+3/-3)
Reviewer Review Type Date Requested Status
James Hebden (community) Approve
Review via email: mp+328290@code.launchpad.net

Description of the change

Fixed prometheus target relation

To post a comment you must log in.
Revision history for this message
James Hebden (ec0) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/reactive/prometheus-ceph-exporter.py b/reactive/prometheus-ceph-exporter.py
index 41e9e69..9d6fb73 100644
--- a/reactive/prometheus-ceph-exporter.py
+++ b/reactive/prometheus-ceph-exporter.py
@@ -18,7 +18,7 @@ import os
18from charmhelpers.core import host, hookenv18from charmhelpers.core import host, hookenv
19from charmhelpers.core.templating import render19from charmhelpers.core.templating import render
20from charms.reactive import (20from charms.reactive import (
21 when, when_not, set_state, remove_state21 when, when_any, set_state, remove_state
22)22)
23from charms.reactive.helpers import any_file_changed, data_changed23from charms.reactive.helpers import any_file_changed, data_changed
24# from charms.layer import snap24# from charms.layer import snap
@@ -41,8 +41,8 @@ def validate_config(filename):
41 return yaml.safe_load(open(filename))41 return yaml.safe_load(open(filename))
4242
4343
44@when('snap.installed.prometheus-ceph-exporter')44@when_any('snap.installed.prometheus-ceph-exporter',
45@when('ceph-exporter.do-reconfig-yaml')45 'ceph-exporter.do-reconfig-yaml')
46def write_ceph_exporter_config_yaml():46def write_ceph_exporter_config_yaml():
47 # config = hookenv.config()47 # config = hookenv.config()
48 hookenv.open_port(PORT_DEF)48 hookenv.open_port(PORT_DEF)

Subscribers

People subscribed via source and target branches