Merge lp:~themue/juju-core/go-environs-mode-errors into lp:~juju/juju-core/trunk
Proposed by
Frank Mueller
Status: | Merged |
---|---|
Approved by: | Gustavo Niemeyer |
Approved revision: | no longer in the source branch. |
Merged at revision: | 682 |
Proposed branch: | lp:~themue/juju-core/go-environs-mode-errors |
Merge into: | lp:~juju/juju-core/trunk |
Diff against target: |
209 lines (+78/-8) 3 files modified
environs/dummy/environs.go (+34/-8) environs/ec2/ec2.go (+24/-0) environs/jujutest/livetests.go (+20/-0) |
To merge this branch: | bzr merge lp:~themue/juju-core/go-environs-mode-errors |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
The Go Language Gophers | Pending | ||
Review via email: mp+130061@code.launchpad.net |
Description of the change
environs: added error in case of illegal mode
Changed OpenPorts/
check if the correct firewall mode is set. That is "global"
for operations on the environ and "instance" for operations
on instances.
To post a comment you must log in.
https:/ /codereview. appspot. com/6715048/ diff/1/ environs/ dummy/environs. go dummy/environs. go (right):
File environs/
https:/ /codereview. appspot. com/6715048/ diff/1/ environs/ dummy/environs. go#newcode494 dummy/environs. go:494: env: e,
environs/
Doesn't look right. One environment state may be linked to from many
environment instances, and one instance is associated to one state.
Please have a look at the code you reverted to see how to do that
properly.
https:/ /codereview. appspot. com/6715048/