|
environs: remove instances arg to Destroy
In all practical cases, this would not make a difference
and there weren't even any tests using the functionality.
Since it's always possible to get the same effect
by calling StopInstances before calling Destroy,
the argument is redundant and is therefore removed.
https://code.launchpad.net/~rogpeppe/juju-core/417-simplify-environ-destroy/+merge/187177
(do not edit description out of merge proposal)
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+30 lines, -93 lines) |
Patch |
|
A |
[revision details]
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
cmd/juju/destroyenvironment.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
environs/interface.go
|
View
|
|
1 chunk |
+4 lines, -6 lines |
0 comments
|
Download
|
|
M |
environs/jujutest/livetests.go
|
View
|
|
4 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
|
M |
environs/jujutest/tests.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
juju/conn_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
provider/azure/environ.go
|
View
|
|
2 chunks |
+1 line, -15 lines |
0 comments
|
Download
|
|
M |
provider/azure/environ_test.go
|
View
|
|
6 chunks |
+5 lines, -15 lines |
0 comments
|
Download
|
|
M |
provider/dummy/environs.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
provider/ec2/ec2.go
|
View
|
|
1 chunk |
+1 line, -13 lines |
0 comments
|
Download
|
|
M |
provider/ec2/local_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
provider/local/environ.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
provider/maas/environ.go
|
View
|
|
1 chunk |
+1 line, -15 lines |
0 comments
|
Download
|
|
M |
provider/maas/environ_test.go
|
View
|
|
1 chunk |
+3 lines, -4 lines |
0 comments
|
Download
|
|
M |
provider/openstack/local_test.go
|
View
|
|
1 chunk |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
provider/openstack/provider.go
|
View
|
|
1 chunk |
+1 line, -13 lines |
0 comments
|
Download
|
Total messages: 2
|