~rogpeppe/juju-core/256-more-status

Viewing all changes in revision 1048.

state: add dialTimeout parameter to state.Open

Updates LP#1084867

Remove the package level dialTimeout and replace it with a parameter passed to state.Open and state.Initalise.

This is part 1 of 3 changes addressing jam's comments from https://codereview.appspot.com/7528047/#msg2. As jam suspected, we were not using the SetDialTimeout helper consistently, nor was it even available outside the state package. To resolve this the helper has been removed and replaced by an explicit value passed to state.Open and state.Initalise.

The next proposal will build on this to pass a retryDelay value as originally proposed in https://codereview.appspot.com/7528047. 

The last proposal will address the heavy duplication of calls to state.Open/Initalise as a cleanup before reapplying https://codereview.appspot.com/7528047.

R=fwereade, wallyworld
CC=
https://codereview.appspot.com/7782049

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: