Merge lp:~andreserl/maas/maas_ipmi_lp1064224 into lp:~maas-committers/maas/trunk
Proposed by
Andres Rodriguez
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Andres Rodriguez | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 1349 | ||||
Proposed branch: | lp:~andreserl/maas/maas_ipmi_lp1064224 | ||||
Merge into: | lp:~maas-committers/maas/trunk | ||||
Diff against target: |
69 lines (+23/-5) 2 files modified
contrib/preseeds_v2/enlist_userdata (+12/-3) etc/maas/commissioning-user-data (+11/-2) |
||||
To merge this branch: | bzr merge lp:~andreserl/maas/maas_ipmi_lp1064224 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Julian Edwards (community) | Approve | ||
Review via email: mp+134144@code.launchpad.net |
Commit message
Do not set IPMI params if detect IPMI IP Address is 0.0.0.0
Description of the change
This change doesn't set the IPMI settings in MAAS for a node, if it was unable to obtain the IPMI IP Address or if the obtained IPMI address is 0.0.0.0.
Note that in order to make sure we detect the correct IP address, we've also increased the wait time to 120sec's if we are changing an IPMI card from Static to DHCP.
To post a comment you must log in.
Once this branch gets approved I'll propose a backport to the stabilization branch.