Prometheus Alertmanager Charm

Merge ~prometheus-charmers/charm-prometheus-alertmanager:d01-fix-http-relation into ~prometheus-charmers/charm-prometheus-alertmanager:master

  1. Git
  2. lp:~prometheus-charmers/charm-prometheus-alertmanager
  3. d01-fix-http-relation
  4. Merge into master
Proposed by Domas Monkus
Status: Merged
Merged at revision: b58c986c60a277d0613c40cd14bb95a15bd62696
Proposed branch: ~prometheus-charmers/charm-prometheus-alertmanager:d01-fix-http-relation
Merge into: ~prometheus-charmers/charm-prometheus-alertmanager:master
Diff against target: 11 lines (+1/-2)
1 file modified
reactive/alertmanager.py (+1/-2)
Reviewer Review Type Date Requested Status
Prometheus Charmers Pending
Review via email: mp+314339@code.staging.launchpad.net

Description of the change

Current alert-manager charm does not work with the prometheus charm. The reason for that is that the port setting on the alertmanager-service relation is not set unless the setting had been changed. Without a configured port, the relation is never marked as available by the consumer.

To post a comment you must log in.

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: