Merge lp:~adam-collard/landscape-charm/fix-landscape-readme into lp:~landscape/landscape-charm/stable

Proposed by Adam Collard
Status: Merged
Approved by: Adam Collard
Approved revision: 225
Merged at revision: 224
Proposed branch: lp:~adam-collard/landscape-charm/fix-landscape-readme
Merge into: lp:~landscape/landscape-charm/stable
Diff against target: 101 lines (+30/-23)
1 file modified
README.md (+30/-23)
To merge this branch: bzr merge lp:~adam-collard/landscape-charm/fix-landscape-readme
Reviewer Review Type Date Requested Status
Alberto Donato (community) Approve
David Britton (community) Approve
🤖 Landscape Builder test results Approve
Review via email: mp+255552@code.launchpad.net

Commit message

Update the README to fix some formatting issues and update out of date text.

Description of the change

Update the README to fix some formatting issues and update out of date text.

http://markable.in/editor/ is a handy way of checking formatting

To post a comment you must log in.
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make test
Result: Success
Revno: 224
Branch: lp:~adam-collard/landscape-charm/fix-landscape-readme
Jenkins: https://ci.lscape.net/job/latch-test/428/

review: Approve (test results)
Revision history for this message
David Britton (dpb) wrote :

+1 with small format changes inline

review: Approve
Revision history for this message
Alberto Donato (ack) wrote :

+1, just one formatting suggestion.

review: Approve
Revision history for this message
Adam Collard (adam-collard) :
225. By Adam Collard

Incorporate formatting suggestions from dpb and ack

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-03-23 19:23:57 +0000
3+++ README.md 2015-04-09 11:01:32 +0000
4@@ -10,7 +10,7 @@
5 connected to other charms to be fully functional. Example deployments are given
6 below.
7
8-For more information about Landscape, go to http://www.ubuntu.com/management
9+For more information about Landscape, go to [Ubuntu Advantage](http://www.ubuntu.com/management)
10
11 Standard Usage
12 ==============
13@@ -20,15 +20,16 @@
14
15 Please use one of the following bundle types depending on your needs:
16
17- https://jujucharms.com/u/landscape/landscape-scalable/
18- https://jujucharms.com/u/landscape/landscape-dense-maas/
19- https://jujucharms.com/u/landscape/landscape-dense/
20-
21-For the landscape-scalable case:
22-
23- sudo apt-add-repository ppa:juju/stable
24- sudo apt-get update
25- juju quickstart u/landscape/landscape-scalable
26+ * [landscape-scalable](https://jujucharms.com/u/landscape/landscape-scalable/)
27+ * [landscape-dense-maas](https://jujucharms.com/u/landscape/landscape-dense-maas/)
28+ * [landscape-dense](https://jujucharms.com/u/landscape/landscape-dense/)
29+
30+For the `landscape-scalable` case:
31+
32+ $ sudo apt-add-repository ppa:juju/stable
33+ $ sudo apt-get update
34+ $ sudo apt-get install juju-quickstart
35+ $ juju quickstart u/landscape/landscape-scalable
36
37
38 Customized Deployments
39@@ -45,17 +46,21 @@
40 Configuration
41 =============
42
43-Landscape is a commercial product and as such it needs configuration of a
44-license and password protected repository before deployment. Please login to
45-your "hosted account" (on landscape.canonical.com) to gather these details
46-after purchasing seats for LDS. All information is found by following a link
47-on the left side of the page called "access the Landscape Dedicated Server
48-archive"
49+Landscape is a commercial product which is bundled with a license
50+allowing management of up to 10 physical machines and 10 more virtual
51+machines, for a total of 20.
52+
53+If you need to manage more machines then you'll need to configure a
54+license and password protected repository before deployment. Please
55+login to your [hosted account](https://landscape.canonical.com/) to
56+gather these details after purchasing seats for LDS. All information
57+is found by following a link on the left side of the page called
58+"access the Landscape Dedicated Server archive"
59
60 license-file
61 ------------
62
63-You can set this as a juju configuration option after deployment
64+You can set this as a Juju configuration option after deployment
65 on each deployed landscape-service like:
66
67 $ juju set <landscape-service> "license-file=$(cat license-file)"
68@@ -68,15 +73,15 @@
69 While useful for testing, this must not be used for production deployments.
70
71 For production deployments, you should include a "real" SSL certificate key
72-pair that has been signed by a CA that your clients trust in the apache charm
73+pair that has been signed by a CA that your clients trust in the Apache charm
74 configuration.
75
76
77 Unit Testing
78 ============
79
80-The Landscape charm is fairly well unit tested and new code changes
81-should be submitted with unit tests. You can run them like this:
82+The Landscape charm is unit tested and new code changes should be
83+submitted with unit tests. You can run them like this:
84
85 $ sudo apt-get install python-twisted-core
86 $ make test
87@@ -85,9 +90,11 @@
88 Integration Testing
89 ===================
90
91-This charm makes use of juju-deployer and the charm-tools package to enable
92-end-to-end integration testing. This is how you proceed with running
93-them:
94+This charm makes use of
95+[juju-deployer](http://pythonhosted.org/juju-deployer/) and
96+[charm-tools](https://jujucharms.com/docs/1.20/tools-charm-tools) to
97+enable end-to-end integration testing. This is how you proceed with
98+running them:
99
100 # Make sure your JUJU_ENV is *not* bootstrapped, and:
101 $ sudo apt-get install python-pyscopg2 python-mocker python-psutil

Subscribers

People subscribed via source and target branches