MAAS

Merge lp:~blake-rouse/maas/fix-bug-1303915 into lp:~maas-committers/maas/trunk

  1. fix-bug-1303915
  2. Merge into trunk
Proposed by Blake Rouse
Status: Merged
Approved by: Blake Rouse
Approved revision: no longer in the source branch.
Merged at revision: 2239
Proposed branch: lp:~blake-rouse/maas/fix-bug-1303915
Merge into: lp:~maas-committers/maas/trunk
Diff against target: 12 lines (+2/-2)
1 file modified
src/provisioningserver/custom_hardware/seamicro.py (+2/-2)
To merge this branch: bzr merge lp:~blake-rouse/maas/fix-bug-1303915
Reviewer Review Type Date Requested Status
Julian Edwards (community) Needs Fixing
Andres Rodriguez (community) Approve
Review via email: mp+214588@code.launchpad.net

Commit message

Force SM15k nodes to boot using PXE when booting with RESTAPI v2, and to force powering off.

Description of the change

Now when powering on SM15k nodes with the REST API v2 the nodes are forced to boot with PXE. This ignores the BIOS settings for the node, making sure that the node uses MAAS to boot.

To post a comment you must log in.
Revision history for this message
Andres Rodriguez (andreserl) wrote :

lgtm! I'm sure this has been tested in real hardware and works just fine.

review: Approve
Revision history for this message
Julian Edwards (julian-edwards) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

 review: needs-fixing

On 08/04/14 07:39, Andres Rodriguez wrote:
> Review: Approve
>
> lgtm! I'm sure this has been tested in real hardware and works just
> fine.
>

Sadly, it still needs unit tests in here...
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlNDmMgACgkQWhGlTF8G/Hcf9ACfdeh9cfyR74BU6d1LObKRTM5e
Y1AAoIjZ+GYzf41IosaEclc5uLfR8XcP
=nLaF
-----END PGP SIGNATURE-----

review: Needs Fixing

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/provisioningserver/custom_hardware/seamicro.py'
--- src/provisioningserver/custom_hardware/seamicro.py 2014-04-07 10:39:19 +0000
+++ src/provisioningserver/custom_hardware/seamicro.py 2014-04-07 17:31:43 +0000
@@ -317,6 +317,6 @@
317 if api:317 if api:
318 server = api.servers.get(server_id)318 server = api.servers.get(server_id)
319 if power_change == "on":319 if power_change == "on":
320 server.power_on()320 server.power_on(using_pxe=True)
321 elif power_change == "off":321 elif power_change == "off":
322 server.power_off()322 server.power_off(force=True)