|
environs: move stuff to provider
The environs package currently serves two very
different purposes - it is designed to be called into
by external packages (and then it calls into the
registered providers), but it is also used
as a repository for common code shared between
provider implementations.
This dual purpose makes it confusing to see
what's what in the documentation, and also
makes it easy to get the layering abstractions wrong.
This change moves some (probably not all yet)
of the latter code into the provider package,
to make the boundaries more clear.
There was some similar code in the
provider package already.
https://code.launchpad.net/~rogpeppe/juju-core/376-factor-out-provider-utils/+merge/182465
(do not edit description out of merge proposal)
Total comments: 1
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+144 lines, -120 lines) |
Patch |
 |
A |
[revision details]
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
M |
cmd/jujud/bootstrap.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
cmd/jujud/bootstrap_test.go
|
View
|
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
environs/bootstrap/bootstrap.go
|
View
|
|
2 chunks |
+2 lines, -1 line |
0 comments
|
Download
|
 |
M |
environs/export_test.go
|
View
|
|
1 chunk |
+0 lines, -19 lines |
0 comments
|
Download
|
 |
M |
environs/testing/polling.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
environs/testing/polling_test.go
|
View
|
|
2 chunks |
+5 lines, -5 lines |
1 comment
|
Download
|
 |
M |
provider/azure/environ.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
provider/azure/environ_test.go
|
View
|
|
4 chunks |
+7 lines, -6 lines |
0 comments
|
Download
|
 |
M |
provider/azure/instance.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
provider/dummy/environs.go
|
View
|
|
2 chunks |
+2 lines, -1 line |
0 comments
|
Download
|
 |
M |
provider/ec2/ec2.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
provider/ec2/local_test.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
A |
provider/export_test.go
|
View
|
|
1 chunk |
+23 lines, -0 lines |
0 comments
|
Download
|
 |
M |
provider/instance.go
|
View
|
|
2 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
provider/local/environ.go
|
View
|
|
6 chunks |
+6 lines, -5 lines |
0 comments
|
Download
|
 |
M |
provider/local/environprovider.go
|
View
|
|
3 chunks |
+5 lines, -5 lines |
0 comments
|
Download
|
 |
M |
provider/local/export_test.go
|
View
|
|
2 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
 |
M |
provider/local/instance.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
provider/local/local_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
provider/maas/environ.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
provider/maas/environ_test.go
|
View
|
|
3 chunks |
+4 lines, -4 lines |
0 comments
|
Download
|
 |
M |
provider/maas/instance.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
provider/openstack/local_test.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
 |
M |
provider/openstack/provider.go
|
View
|
|
3 chunks |
+3 lines, -3 lines |
0 comments
|
Download
|
 |
M |
provider/polling.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
 |
M |
provider/polling_test.go
|
View
|
|
5 chunks |
+9 lines, -9 lines |
0 comments
|
Download
|
 |
M |
provider/provider.go
|
View
|
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
 |
A |
provider/provider_test.go
|
View
|
|
1 chunk |
+10 lines, -0 lines |
0 comments
|
Download
|
 |
M |
provider/state.go
|
View
|
|
7 chunks |
+8 lines, -7 lines |
0 comments
|
Download
|
 |
M |
provider/state_test.go
|
View
|
|
10 chunks |
+26 lines, -25 lines |
0 comments
|
Download
|
Total messages: 2
|