|
unit.SetAddress now takes a vararg
Update LP#1298770
The previous branch updated instance.NewAddress to be varargs and demonstated that most callers of NewAddresses were used to get a slice of addresses so they could pass them to unit.SetAddresses. By making SetAddresses vararg NewAddresses is unneeded. This will be next change.
https://code.launchpad.net/~dave-cheney/juju-core/108-set-addresses-take-varargs/+merge/213393
(do not edit description out of merge proposal)
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+45 lines, -65 lines) |
Patch |
|
A |
[revision details]
|
View
|
1
2
3
4
|
1 chunk |
+2 lines, -0 lines |
0 comments
|
Download
|
|
M |
cmd/juju/scp_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
cmd/juju/ssh_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
cmd/juju/status_test.go
|
View
|
1
2
3
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
cmd/jujud/machine_test.go
|
View
|
1
2
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
juju/testing/utils.go
|
View
|
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
state/api/deployer/deployer_test.go
|
View
|
|
2 chunks |
+2 lines, -5 lines |
0 comments
|
Download
|
|
M |
state/api/machiner/machiner_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
state/api/provisioner/provisioner_test.go
|
View
|
1
2
|
2 chunks |
+2 lines, -4 lines |
0 comments
|
Download
|
|
M |
state/apiserver/client/client_test.go
|
View
|
|
4 chunks |
+6 lines, -6 lines |
0 comments
|
Download
|
|
M |
state/apiserver/client/run_test.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
state/apiserver/deployer/deployer_test.go
|
View
|
|
2 chunks |
+2 lines, -6 lines |
0 comments
|
Download
|
|
M |
state/apiserver/uniter/uniter_test.go
|
View
|
1
2
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
state/machine.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
state/machine_test.go
|
View
|
1
2
3
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
state/megawatcher_internal_test.go
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
state/state_test.go
|
View
|
|
1 chunk |
+5 lines, -5 lines |
0 comments
|
Download
|
|
M |
state/unit_test.go
|
View
|
1
2
3
|
3 chunks |
+5 lines, -7 lines |
0 comments
|
Download
|
|
M |
worker/instancepoller/machine_test.go
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
worker/instancepoller/updater.go
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
worker/provisioner/kvm-broker_test.go
|
View
|
1
2
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
worker/provisioner/lxc-broker_test.go
|
View
|
1
2
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
worker/provisioner/provisioner_test.go
|
View
|
1
2
|
1 chunk |
+1 line, -3 lines |
0 comments
|
Download
|
|
M |
worker/uniter/uniter_test.go
|
View
|
1
2
3
4
|
1 chunk |
+3 lines, -3 lines |
0 comments
|
Download
|
Total messages: 4
|