CT Google Storage utils to download/upload artifacts for workers

CT Google Storage utils to download/upload artifacts for workers.

BUG=skia:3163
TBR=jcgreogrio

Committed: https://skia.googlesource.com/buildbot/+/eed8058e8bc24d0df656ba5d23a342e8a9ca4921

[rmistry, 2014-12-03 17:57:40]

rmistry@google.com changed reviewers:
+ jcgregorio@google.com

[rmistry, 2014-12-03 17:57:40]

rmistry@google.com changed required reviewers:
+ jcgregorio@google.com

[jcgregorio, 2014-12-03 19:22:14]

jcgregorio@google.com changed required reviewers:
- jcgregorio@google.com

[jcgregorio, 2014-12-03 19:22:15]

https://codereview.chromium.org/779633003/diff/20001/ct/go/util/constants.go
File ct/go/util/constants.go (right):

https://codereview.chromium.org/779633003/diff/20001/ct/go/util/constants.go#...
ct/go/util/constants.go:7: CT_USER                  = "rmistry"
Looks like a dup from the other CL?

[rmistry, 2014-12-03 19:23:20]

https://codereview.chromium.org/779633003/diff/20001/ct/go/util/constants.go
File ct/go/util/constants.go (right):

https://codereview.chromium.org/779633003/diff/20001/ct/go/util/constants.go#...
ct/go/util/constants.go:7: CT_USER                  = "rmistry"
On 2014/12/03 19:22:14, jcgregorio wrote:
> Looks like a dup from the other CL?

Yes, this file is the only dupe it will go away after I rebase. Please ignore :)

[rmistry, 2014-12-03 19:40:39]

Rebased

[jcgregorio, 2014-12-03 19:52:06]

https://codereview.chromium.org/779633003/diff/60001/ct/go/util/gs_test.go
File ct/go/util/gs_test.go (right):

https://codereview.chromium.org/779633003/diff/60001/ct/go/util/gs_test.go#ne...
ct/go/util/gs_test.go:15: // Will need a local valid google_storage_token.data
file to run the below test.
Can we protect these tests with -test.short so not having a
google_storage_token.data doesn't interfere with running presubmits?

https://github.com/google/skia-buildbot/blob/master/golden/go/analysis/analys...

https://github.com/google/skia-buildbot/blob/master/golden/Makefile#L16

[rmistry, 2014-12-04 12:17:02]

https://codereview.chromium.org/779633003/diff/60001/ct/go/util/gs_test.go
File ct/go/util/gs_test.go (right):

https://codereview.chromium.org/779633003/diff/60001/ct/go/util/gs_test.go#ne...
ct/go/util/gs_test.go:15: // Will need a local valid google_storage_token.data
file to run the below test.
On 2014/12/03 19:52:06, jcgregorio wrote:
> Can we protect these tests with -test.short so not having a
> google_storage_token.data doesn't interfere with running presubmits?
> 
>
https://github.com/google/skia-buildbot/blob/master/golden/go/analysis/analys...
> 
> https://github.com/google/skia-buildbot/blob/master/golden/Makefile#L16

Cool, I did not know about this. Done.

[rmistry, 2014-12-04 14:57:25]

TBR'ing this for now. Will address additional comments in a future CL.

[rmistry, 2014-12-04 14:57:29]

The CQ bit was checked by rmistry@google.com

[commit-bot: I haz the power, 2014-12-04 14:58:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/779633003/80001

[commit-bot: I haz the power, 2014-12-04 14:58:14]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/buildbot/+/eed8058e8bc24d0df656ba5d23a342e8a9ca...