Merge lp:~evilnick/maas/cli-help into lp:~maas-committers/maas/trunk

Proposed by Nick Veitch
Status: Merged
Approved by: Raphaël Badin
Approved revision: no longer in the source branch.
Merged at revision: 1328
Proposed branch: lp:~evilnick/maas/cli-help
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 13 lines (+3/-0)
1 file modified
src/maasserver/api.py (+3/-0)
To merge this branch: bzr merge lp:~evilnick/maas/cli-help
Reviewer Review Type Date Requested Status
Julian Edwards (community) Approve
Review via email: mp+132903@code.launchpad.net

Commit message

Added minimum data requirements to help for maas-cli nodes new command

Description of the change

adds a little more explanation to the maas-cli nodes new commandline help

To post a comment you must log in.
Revision history for this message
Julian Edwards (julian-edwards) wrote :

Looks good. Can you merge this into lp:maas/1.2 as well please!

review: Approve
Revision history for this message
Nick Veitch (evilnick) wrote :

On 06/11/12 04:26, Julian Edwards wrote:
> Review: Approve
>
> Looks good. Can you merge this into lp:maas/1.2 as well please!
arr

--
Nick Veitch,
Technical Writer, Cloud Engineering, Canonical
https://launchpad.net/~evilnick

Revision history for this message
MAAS Lander (maas-lander) wrote :

The Jenkins job https://jenkins.qa.ubuntu.com/job/maas-merger-trunk/55/console reported an error when processing this lp:~evilnick/maas/cli-help branch.
Not merging it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/maasserver/api.py'
2--- src/maasserver/api.py 2012-11-01 11:45:29 +0000
3+++ src/maasserver/api.py 2012-11-05 14:18:25 +0000
4@@ -745,6 +745,9 @@
5
6 When a node has been added to MAAS by an admin MAAS user, it is
7 ready for allocation to services running on the MAAS.
8+ The minimum data required is:
9+ architecture=<arch string> (e.g "i386/generic")
10+ mac_address=<value>
11 """
12 node = create_node(request)
13 if request.user.is_superuser: