Code review comment for lp:~thumper/juju-core/really-unit-agent-proxy-settings

Revision history for this message
Tim Penhey (thumper) wrote :

On 2014/01/23 21:27:37, wallyworld wrote:
> New to lose the global proxy vars. But bigger issue is a missing
end-end test to
> ensure we can make a change it state and it is propagated through and
used next
> time a hook is run.

This is now there.

> https://codereview.appspot.com/53240044/diff/1/worker/uniter/proxy.go
> File worker/uniter/proxy.go (right):

https://codereview.appspot.com/53240044/diff/1/worker/uniter/proxy.go#newcode16
> worker/uniter/proxy.go:16: proxyMutex sync.Mutex
> Please move these from being global to attrs on Uniter

Done.

https://codereview.appspot.com/53240044/diff/1/worker/uniter/proxy.go#newcode26
> worker/uniter/proxy.go:26: func UpdatePackageProxy(cfg *config.Config)
{
> This could then be moved as well.

Yes.

https://codereview.appspot.com/53240044/

« Back to merge proposal