Merge lp://qastaging/~blake-rouse/maas/fix-1457990 into lp://qastaging/~maas-committers/maas/trunk

Proposed by Blake Rouse
Status: Merged
Approved by: Blake Rouse
Approved revision: no longer in the source branch.
Merged at revision: 3948
Proposed branch: lp://qastaging/~blake-rouse/maas/fix-1457990
Merge into: lp://qastaging/~maas-committers/maas/trunk
Diff against target: 42 lines (+13/-1)
2 files modified
src/maasserver/node_action.py (+3/-0)
src/maasserver/tests/test_node_action.py (+10/-1)
To merge this branch: bzr merge lp://qastaging/~blake-rouse/maas/fix-1457990
Reviewer Review Type Date Requested Status
Mike Pontillo (community) Approve
Review via email: mp+260229@code.qastaging.launchpad.net

Commit message

Prevent starting commissioning of a node that is new and on.

To post a comment you must log in.
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Code looks good, but minor nit about the wording of the error.

review: Approve
Revision history for this message
Blake Rouse (blake-rouse) :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.