Merge ~elmo/charm-prometheus-alertmanager:juju-config-not-set into charm-prometheus-alertmanager:master

Proposed by James Troup
Status: Merged
Approved by: James Troup
Approved revision: eeaf63f2b712ae287dec0a95ad71f99e12ab517d
Merged at revision: 6f5b0caad480e63edfa3a37aa51d6a1c23ed8e3c
Proposed branch: ~elmo/charm-prometheus-alertmanager:juju-config-not-set
Merge into: charm-prometheus-alertmanager:master
Diff against target: 13 lines (+1/-1)
1 file modified
src/config.yaml (+1/-1)
Reviewer Review Type Date Requested Status
Celia Wang Approve
🤖 prod-jenkaas-bootstack (community) continuous-integration Approve
BootStack Reviewers Pending
Review via email: mp+406771@code.launchpad.net

Commit message

Replace reference to 'juju set' with 'juju config'.

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
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

A CI job is currently in progress. A follow up comment will be added when it completes.

Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :
review: Approve (continuous-integration)
Revision history for this message
Celia Wang (ziyiwang) wrote :

lgtm

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

Change successfully merged at revision 6f5b0caad480e63edfa3a37aa51d6a1c23ed8e3c

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/config.yaml b/src/config.yaml
2index fe0c1b1..96392df 100644
3--- a/src/config.yaml
4+++ b/src/config.yaml
5@@ -27,7 +27,7 @@ options:
6 URL components will be derived automatically.
7 It's templatized to accept the following vars: {private_address},
8 {public_address}, {port}, thus you can use e.g.:
9- juju set alertmanager external_url="http://{private_address}:{port}/"
10+ juju config alertmanager external_url="http://{private_address}:{port}/"
11 raw_config:
12 default: ""
13 type: string

Subscribers

People subscribed via source and target branches

to all changes: