Merge lp:~dpb/landscape-bundles/readme-update-juju2 into lp:landscape-bundles

Proposed by 🤖 Landscape Builder
Status: Merged
Approved by: David Britton
Approved revision: 47
Merged at revision: 40
Proposed branch: lp:~dpb/landscape-bundles/readme-update-juju2
Merge into: lp:landscape-bundles
Diff against target: 42 lines (+10/-16)
1 file modified
README.md (+10/-16)
To merge this branch: bzr merge lp:~dpb/landscape-bundles/readme-update-juju2
Reviewer Review Type Date Requested Status
🤖 Landscape Builder test results Approve
Andreas Hasenack Approve
Chad Smith Approve
Francis Ginther (community) Approve
Review via email: mp+321361@code.launchpad.net

Commit message

Update bundle readme for juju2.

Description of the change

Update bundle readme for juju2.

To post a comment you must log in.
40. By David Britton

[andreas] remove extra |.* at end of link

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Easy comments inline

review: Needs Fixing
41. By David Britton

remove trailing slash on dense deployment.

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
42. By David Britton

[andreas] s/stanza/command/

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
43. By David Britton

remove deployment links, they were confusing.

Revision history for this message
Chad Smith (chad.smith) :
Revision history for this message
Francis Ginther (fginther) wrote :

Minor comment.

Revision history for this message
Francis Ginther (fginther) wrote :

Otherwise +1.

review: Approve
44. By David Britton

[chad] s/:/./

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Some suggestions added inline.

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
Revision history for this message
Chad Smith (chad.smith) :
review: Approve
45. By David Britton

[andreas] wording changes.

Revision history for this message
David Britton (dpb) :
46. By David Britton

[bbsw] period after link.

Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1, just one nit/grammar.

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
47. By David Britton

[andreas] s/this/This/, and period.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'README.md'
2--- README.md 2015-10-14 13:38:20 +0000
3+++ README.md 2017-03-29 20:57:27 +0000
4@@ -1,28 +1,22 @@
5 Overview
6 ========
7
8-This branch contains the necessary config and instructions for using
9-juju-deployer/juju-quickstart to deploy the landscape charm.
10-
11-Deployment should be straightforward.
12+This bundle will deploy [Landscape On-Premises](https://landscape.canonical.com/landscape-features).
13+There are three slightly different versions of this bundle, explained below.
14
15 Dense Deployment - MAAS
16 =======================
17-For MAAS where LXCs can be addressed externally, you can deploy to a single
18-machine while at the same time making the service scalable in the future:
19-
20- juju quickstart u/landscape/landscape-dense-maas/
21+`landscape-dense-maas` - For MAAS where LXDs can be addressed externally, you
22+can deploy to a single machine while at the same time making the service
23+scalable in the future.
24
25 Dense Deployment - Other
26 ========================
27-For other substrates, use the plain dense deployment which allows your
28-landscape server to be fully reachable, but will not allow easy scaling
29-should load increase.
30-
31- juju quickstart u/landscape/landscape-dense/
32+`landscape-dense` - For other machine providers where LXD containers do not get
33+externally routable IP addresses. This will deploy the frontend on the machine
34+itself, and the remaining backend services in containers. This approach is not
35+currently scalable to another physical system but conserves resources.
36
37 Scalable Deployment
38 ===================
39-For a truly scalable deployment, the following stanza should be used.
40-
41- juju quickstart u/landscape/landscape-scalable
42+`landscape-scalable` - A fully scalable multi-machine deployment.

Subscribers

People subscribed via source and target branches

to all changes: