MAAS

Merge lp://staging/~newell-jensen/maas/fix-1595753 into lp://staging/~maas-committers/maas/trunk

  1. fix-1595753
  2. Merge into trunk
Proposed by Newell Jensen
Status: Merged
Approved by: Newell Jensen
Approved revision: no longer in the source branch.
Merged at revision: 5142
Proposed branch: lp://staging/~newell-jensen/maas/fix-1595753
Merge into: lp://staging/~maas-committers/maas/trunk
Diff against target: 61 lines (+6/-6)
2 files modified
src/provisioningserver/drivers/power/hmc.py (+4/-4)
src/provisioningserver/drivers/power/tests/test_hmc.py (+2/-2)
To merge this branch: bzr merge lp://staging/~newell-jensen/maas/fix-1595753
Reviewer Review Type Date Requested Status
Newell Jensen (community) Approve
Review via email: mp+298265@code.staging.launchpad.net

Commit message

This branch fixes an HMC regression from MAAS 1.9 to MAAS 2.0 (this will be backported to 2.0 branch). Updates faulty unit tests as well.

To post a comment you must log in.
Revision history for this message
Newell Jensen (newell-jensen) wrote :

Self approving, simple fix.

review: Approve
Revision history for this message
Mike Pontillo (mpontillo) wrote :

Looks fine, just curious why the fix was needed. It might be good to explain that in the comments, as it's not too clear from the diff.

Revision history for this message
Newell Jensen (newell-jensen) wrote :

The proper parameters were not getting passed to the run_hmc_command method. The unit tests were faulty and did not catch this as the mocked method was expected to be called with the incorrect amount of arguments.

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.