Merge ~jacekn/charm-prometheus-alertmanager:snap_0_15 into ~prometheus-charmers/charm-prometheus-alertmanager:master

Proposed by Jacek Nykis
Status: Merged
Approved by: Alvaro Uria
Approved revision: ef7b996d3570c2f5d2db13ad66a374704dfa3693
Merged at revision: 708789ef03f767f4ecb82140620df0f932a1bdf0
Proposed branch: ~jacekn/charm-prometheus-alertmanager:snap_0_15
Merge into: ~prometheus-charmers/charm-prometheus-alertmanager:master
Diff against target: 78 lines (+35/-19)
2 files modified
reactive/alertmanager.py (+5/-4)
templates/daemon_arguments.j2 (+30/-15)
Reviewer Review Type Date Requested Status
Alvaro Uria (community) Approve
Review via email: mp+347915@code.staging.launchpad.net

Commit message

Update charm to support alertmanagaer v0.15 format arguments

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Alvaro Uria (aluria) wrote :

lgtm

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 708789ef03f767f4ecb82140620df0f932a1bdf0

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