Code review comment for ~troyanov/maas:fix-2042645

Revision history for this message
Björn Tillenius (bjornt) wrote :

Maybe this is a good opportunity to clean things up.

For example, mac_address is interesting. Yes, IPMI accepts a mac_address. But it expects the MAC of the BMC port, which the host won't see. So the mac_address returned by get_effective_power_parameters() is not usable by the IPMI power driver.

« Back to merge proposal