2.0 metrics breaks 1.25

Bug #1639855 reported by Chris MacNaughton
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
Critical
Casey Marshall

Bug Description

Trying to deploy a charm that defines the metrics.yaml file for metrics in 2.0+ fails due to "ERROR no metric registration url is specified"[1]

[1]: https://review.openstack.org/#/c/392585/

Tags: uosci
Revision history for this message
Casey Marshall (cmars) wrote :

Need to first land https://github.com/juju/charm/pull/225, then update dependencies.tsv in https://github.com/juju/juju/pull/6549 accordingly.

Changed in juju-core:
assignee: nobody → Casey Marshall (cmars)
status: New → In Progress
Revision history for this message
Casey Marshall (cmars) wrote :

@Icey
I'm able to deploy your ceph-mon charm at 8c1ef1d on 1.25 with my above patch.

However, you'll get a hook error in collect-metrics. This patch to your charm should fix it:

ubuntu@juju-125-dev:~/trusty/charm-ceph-mon$ git diff
diff --git a/hooks/ceph_hooks.py b/hooks/ceph_hooks.py
index 59b1c9c..8f49ccc 100755
--- a/hooks/ceph_hooks.py
+++ b/hooks/ceph_hooks.py
@@ -140,7 +140,7 @@ def collect_metrics():
             cluster = rados.Rados(conffile='/etc/ceph/ceph.conf')
             cluster.connect(timeout=60) # 1 minute timeout
             log("Gathering pool stats")
- version = rados.version()
+ version = cluster.version()
             pools = cluster.list_pools()
             kb_used = 0
             for pool in pools:

Revision history for this message
Chris MacNaughton (chris.macnaughton) wrote :

@cmars thanks for this. Is there an eta on your patch landing in a released version?

Casey Marshall (cmars)
Changed in juju-core:
status: In Progress → Fix Committed
Ryan Beisner (1chb1n)
tags: added: uosci
Curtis Hovey (sinzui)
Changed in juju-core:
milestone: none → 1.25.8
importance: Undecided → Critical
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
Revision history for this message
Curtis Hovey (sinzui) wrote :

As this issue is fixed in a supported old version, the clients can be found in the 1.25-proposed PPA
     https://launchpad.net/~juju/+archive/ubuntu/1.25-proposed
The successful candidate will graduate to the
     https://launchpad.net/~juju/+archive/ubuntu/1.25
PPA

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.