Merge lp:~allenap/maas/ipmi-power-confusion--bug-1560830 into lp:~maas-committers/maas/trunk
Proposed by
Gavin Panella
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Gavin Panella | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 5034 | ||||
Proposed branch: | lp:~allenap/maas/ipmi-power-confusion--bug-1560830 | ||||
Merge into: | lp:~maas-committers/maas/trunk | ||||
Diff against target: |
49 lines (+15/-6) 2 files modified
src/provisioningserver/drivers/power/ipmi.py (+3/-6) src/provisioningserver/drivers/power/tests/test_ipmi.py (+12/-0) |
||||
To merge this branch: | bzr merge lp:~allenap/maas/ipmi-power-confusion--bug-1560830 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Newell Jensen (community) | Approve | ||
Andres Rodriguez (community) | Needs Information | ||
Review via email: mp+295018@code.launchpad.net |
Commit message
In the IPMI power driver, don't get confused when ipmipower prints status for a host with 'on' or 'off' in its name.
To post a comment you must log in.
Hi Gavin,
Newell fixed this for 1.9, but It seems the patch never made it to 2.0, as I don't know if it is actually needed depending on the way how power drivers changed for 2.0, but it may still be there. Please syncup with him, but otherwise look at:
https:/ /bugs.launchpad .net/maas/ +bug/1508741
Also, question inline.