Merge lp:~rvb/juju-core/fix-maas-pvd-conf into lp:~juju/juju-core/trunk
Status: | Rejected |
---|---|
Rejected by: | Dimiter Naydenov |
Proposed branch: | lp:~rvb/juju-core/fix-maas-pvd-conf |
Merge into: | lp:~juju/juju-core/trunk |
Diff against target: |
45 lines (+26/-1) 2 files modified
environs/maas/environprovider.go (+5/-1) environs/maas/environprovider_test.go (+21/-0) |
To merge this branch: | bzr merge lp:~rvb/juju-core/fix-maas-pvd-conf |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dimiter Naydenov (community) | Approve | ||
Review via email: mp+158938@code.launchpad.net |
Commit message
Fix maasEnvironProv
Description of the change
[Note: I did not use rietveld because I ran into errors like: http://
This branch fixes a problem we saw happening in the MAAS lab while testing the MAAS provider. The symptom was that, after having installed the bootstrap node ok, Juju was unable to deploy any service. The error found in the bootstrap node's logs: http://
The problem is that, from this code in cmd/jujud/
LGTM