collect-metrics fail because "not a local charm"

[Junien F]

Hi,

I'm trying to force metrics collection on a redis charm (deployed from the store), and juju (2.4.3) refuses to do so :

$ juju metrics redis
UNIT TIMESTAMP METRIC VALUE LABELS
redis/0 2018-10-05T15:00:41Z blocked_clients 2
redis/1 2018-10-05T15:03:14Z blocked_clients 0
redis/0 2018-10-05T15:00:41Z connected_clients 36
redis/1 2018-10-05T15:03:14Z connected_clients 1
redis/0 2018-10-05T15:00:40Z juju-units 1
redis/1 2018-10-05T15:03:13Z juju-units 1
redis/0 2018-10-05T15:00:41Z used_memory 2557056
redis/1 2018-10-05T15:03:14Z used_memory 570968
$ juju collect-metrics redis
ERROR "redis" is not a local charm
$ juju collect-metrics redis/0
ERROR "redis/0" is not a local charm

Thanks

[Casey Marshall (cmars)]

Need to remove the restriction that collect-metrics only works on a locally-deployed charm.

[Casey Marshall (cmars)]

@knguyen93 This demonstrates the issue: https://pastebin.canonical.com/p/yKPnVmrXh8/. Instead of "ERROR "redis/0" is not a local charm", we'd want Juju to fire the collect-metrics hook.

[Khanh Nguyen (knguyen93)]

This has been fixed and merged into develop.

[Canonical Juju QA Bot (juju-qa-bot)]

This bug has not been updated in 2 years, so we're marking it Low importance. If you believe this is incorrect, please update the importance.