Code review comment for lp:~axwalk/juju-core/agent-setapihostports

Revision history for this message
Andrew Wilkins (axwalk) wrote :

On 2014/03/31 06:44:02, dfc wrote:
> https://codereview.appspot.com/82520043/diff/1/agent/agent_test.go
> File agent/agent_test.go (right):

https://codereview.appspot.com/82520043/diff/1/agent/agent_test.go#newcode465
> agent/agent_test.go:465: c.Assert(addrs, gc.DeepEquals,
[]string{"0.1.2.3:123",
> "0.1.2.5:125"})
> Does conf.APIAddresses return a set or an ordered list ?

APIAddresses returns the same order as specified in the list in the
config file.
I'm told order is important for addresses, which is why I've done it
with DeepEquals.

https://codereview.appspot.com/82520043/

« Back to merge proposal