Merge lp:~newell-jensen/maas/fix-1553841 into lp:~maas-committers/maas/trunk
Proposed by
Newell Jensen
Status: | Merged |
---|---|
Approved by: | Newell Jensen |
Approved revision: | no longer in the source branch. |
Merged at revision: | 4998 |
Proposed branch: | lp:~newell-jensen/maas/fix-1553841 |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
221 lines (+54/-18) 2 files modified
src/provisioningserver/drivers/power/amt.py (+26/-5) src/provisioningserver/drivers/power/tests/test_amt.py (+28/-13) |
To merge this branch: | bzr merge lp:~newell-jensen/maas/fix-1553841 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mike Pontillo (community) | Approve | ||
Review via email: mp+293819@code.launchpad.net |
Commit message
Update AMT error messages to instruct the user if they have entered an incorrect BMC configuration (power parameters).
To post a comment you must log in.
Code looks fine to me. Just a couple things I'm wondering:
* Can you update the commit message so that it is more accurate? (it makes it sound like error propagation in general, but the branch seems to just fix up the AMT exceptions so that they are thrown correctly.)
* Are we confident the AMT utilities will always print these particular strings? (I assume we're using a consistent locale, so that the strings don't show up translated into another language and surprise us. And I was wondering if using just the HTTP status code for matching might be more future-proof... assuming the output isn't too large, and we could match the number in multiple places in the string...?)