progressutil: added mutexes to make go's race detector happy

[cgonyeo]

docker2aci has some race conditions, and go's race detector found some race-y accesses in progressutil, so this commit wraps some variables shared between goroutines in mutexes.

[cgonyeo]

Let this run for an hour and a half with Go's race detector going:

$(exit 0); while [ $? -eq 0 ]; do ./bin/docker2aci docker://dgonyeo/test; done

Found another issue, fixed it. I think this is good now.

[jonboulle]

@dgonyeo to be clear, did you reproduce the issue(s) we were experiencing even without the race detector?

Derek Gonyeo notifications@github.com schrieb am Fr., 27. Mai 2016, 23:54:

Let this run for an hour and a half with Go's race detector going:

$(exit 0); while [ $? -eq 0 ]; do ./bin/docker2aci docker://dgonyeo/test; done

Found another issue, fixed it. I think this is good now.

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/coreos/pkg/pull/61#issuecomment-222261808, or mute the thread https://github.com/notifications/unsubscribe/ACewN1lAcv4gSTmSKi1PxYgU92BqQL70ks5qF2gsgaJpZM4Ioxr9 .

[jonboulle]

This doesn't address all the issues but it's a step forward

[jonboulle]

62