Merge lp:~dduffey/maas/wedge-1.9 into lp:maas/1.9
Proposed by
David Duffey
Status: | Rejected |
---|---|
Rejected by: | Blake Rouse |
Proposed branch: | lp:~dduffey/maas/wedge-1.9 |
Merge into: | lp:maas/1.9 |
Diff against target: |
138 lines (+94/-0) 4 files modified
src/provisioningserver/drivers/power/__init__.py (+2/-0) src/provisioningserver/drivers/power/wedge.py (+81/-0) src/provisioningserver/power/__init__.py (+1/-0) src/provisioningserver/power/schema.py (+10/-0) |
To merge this branch: | bzr merge lp:~dduffey/maas/wedge-1.9 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Blake Rouse (community) | Needs Fixing | ||
Review via email: mp+285382@code.launchpad.net |
Description of the change
Added Facebook Wedge OpenBMC power driver support
To post a comment you must log in.
Unmerged revisions
- 4540. By David Duffey
-
Added Facebook Wedge OpenBMC Power Driver
Thanks for working on this. Many things need to be addressed in this branch. I have included many inline comments, but the real blockers are below.
1. We almost always land in trunk and then backport. So you need to have this working on MAAS trunk first. There is a big different between 1.9 and trunk now, and that is trunk is python 3 only. YOu need to make sure this works with MAAS trunk and on Xenial and then do the backport. That needs to be QA'd on both 1.9 and trunk. We cannot help you with the QA process as we do not have access to this hardware.
2. No unit tests have been provided with this branch. It is a requirement that any branch that lands in MAAS contain unit tests. This power driver will be something people will come to rely on working, adding unit tests will ensure that it does not get broken as other features, fixes, and more power drivers are added to MAAS.