Adds extra logging to telemetry in hope it helps diagnose issue 351143

Adds extra logging to telemetry in hope it helps diagnose issue 351143

It seems there is a problem running gsutil on cros dtus, so adding few
extra log statements around that code.

BUG=351143
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272838

[tbarzic, 2014-05-23 21:26:48]

Can you take a look at this? Hope it makes sense..

warning: I have almost zero experience with python, so be on a lookout for
obvious mistakes :)

[achuithb, 2014-05-23 21:33:34]

lgtm

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/cloud_storage.py (right):

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/cloud_storage.py:179: logging.warning('Hash file
not found: %s' % hash_path)
You can use , instead of % operator here and elsewhere

[tbarzic, 2014-05-23 21:38:14]

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/cloud_storage.py (right):

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/cloud_storage.py:179: logging.warning('Hash file
not found: %s' % hash_path)
On 2014/05/23 21:33:34, achuithb wrote:
> You can use , instead of % operator here and elsewhere

as far as I can see, other logging calls in the file use %, so I'd rather keep
it consistent :)

[achuithb, 2014-05-23 21:39:11]

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
File tools/telemetry/telemetry/page/cloud_storage.py (right):

https://codereview.chromium.org/298863016/diff/1/tools/telemetry/telemetry/pa...
tools/telemetry/telemetry/page/cloud_storage.py:179: logging.warning('Hash file
not found: %s' % hash_path)
On 2014/05/23 21:38:14, tbarzic wrote:
> On 2014/05/23 21:33:34, achuithb wrote:
> > You can use , instead of % operator here and elsewhere
> 
> as far as I can see, other logging calls in the file use %, so I'd rather keep
> it consistent :)

sure, in the cros codebase, there's actually a presubmit check for this. Anyway,
doesn't matter.

[tbarzic, 2014-05-25 21:30:41]

The CQ bit was checked by tbarzic@chromium.org

[commit-bot: I haz the power, 2014-05-25 21:30:53]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tbarzic@chromium.org/298863016/20001

[commit-bot: I haz the power, 2014-05-26 16:09:33]

Change committed as 272838