Change in snap channel does not refresh the snap
Bug #1891582 reported by
Xav Paice
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Prometheus Ceph Exporter Charm |
Fix Released
|
High
|
David O Neill | ||
Bug Description
When the snap channel config is changed, e.g. from stable to edge, nothing in the charm triggers the change. The config-changed hook should do this.
E.g.:
juju config prometheus-
Should result in the snap from the edge channel getting installed. Instead, we have to:
$ sudo snap refresh prometheus-
Related branches
~afreiberger/charm-prometheus-ceph-exporter:LP1891582
Merged
into
charm-prometheus-ceph-exporter:master
at
revision 8e952472db03042c6862a8d7ae5b816bd4915053
- Xav Paice (community): Approve
- Celia Wang: Approve
- Alvaro Uria: Pending requested
- Drew Freiberger: Pending requested
- David O Neill: Pending requested
- James Hebden: Pending requested
-
Diff: 146 lines (+65/-14)5 files modifiedMakefile (+4/-0)
src/reactive/prometheus_ceph_exporter.py (+23/-6)
src/tests/functional/tests/test_prometheus_ceph_exporter.py (+29/-5)
src/tests/unit/test_reactive_ceph_exporter.py (+3/-3)
src/tox.ini (+6/-0)
charm-prometheus-ceph-exporter:LP1891582
- Drew Freiberger (community): Approve
- David O Neill (community): Abstain
- James Hebden (community): Needs Fixing
- Alvaro Uria (community): Needs Fixing
-
Diff: 146 lines (+65/-14)5 files modifiedMakefile (+4/-0)
src/reactive/prometheus_ceph_exporter.py (+23/-6)
src/tests/functional/tests/test_prometheus_ceph_exporter.py (+29/-5)
src/tests/unit/test_reactive_ceph_exporter.py (+3/-3)
src/tox.ini (+6/-0)
Revision history for this message
| Adam Dyess (addyess) wrote | #1 |
| Changed in charm-prometheus-ceph-exporter: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| importance: | Medium → High |
| Changed in charm-prometheus-ceph-exporter: | |
| assignee: | nobody → David O Neill (dmzoneill) |
| status: | Confirmed → In Progress |
| Changed in charm-prometheus-ceph-exporter: | |
| status: | In Progress → Fix Committed |
| Changed in charm-prometheus-ceph-exporter: | |
| status: | Fix Committed → Confirmed |
| Changed in charm-prometheus-ceph-exporter: | |
| status: | Confirmed → Fix Released |
| milestone: | none → 21.01 |
To post a comment you must log in.
There doesn't appear to be a config-changed hook in the charm.
This charm should handle any config-changes