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

Proposed by James Troup
Status: Merged
Approved by: James Troup
Approved revision: f04685cf714dc99e42d08c78ce6358848658acd2
Merged at revision: 8ce484a87bdc85622d1d65c1271fe5d367cfc588
Proposed branch: ~elmo/charm-elasticsearch:juju-config-not-set
Merge into: charm-elasticsearch:master
Diff against target: 13 lines (+1/-1)
1 file modified
README.md (+1/-1)
Reviewer Review Type Date Requested Status
Celia Wang Approve
🤖 prod-jenkaas-bootstack (community) continuous-integration Needs Fixing
BootStack Reviewers Pending
Review via email: mp+406775@code.launchpad.net

Commit message

Replace references 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: Needs Fixing (continuous-integration)
Revision history for this message
Celia Wang (ziyiwang) wrote :

The CI failure is not caused by this MR.
+1 to merge this one.

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

Change successfully merged at revision 8ce484a87bdc85622d1d65c1271fe5d367cfc588

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/README.md b/README.md
2index 5dee831..f90f394 100644
3--- a/README.md
4+++ b/README.md
5@@ -33,7 +33,7 @@ the instance itself and any clients that join. Similarly the node-to-node
6 communication port (9300) is only available to other units in the elasticsearch
7 service. You can change this explicitly with:
8
9- juju set elasticsearch firewall_enabled=false
10+ juju config elasticsearch firewall_enabled=false
11
12 See the separate HACKING.md for information about deploying this charm
13 from a local repository.

Subscribers

People subscribed via source and target branches

to all changes: