Override HOME dir for gsutil when running on cros device

Override HOME dir for gsutil when running on cros device

Gsutil tries to create a dir (for keeping track of resumable downloads)
under home dir. On ChromeOS this is, given that the tests must be run
by root, read-only '/root/'. This causes gsutil commands to fail. To
avoid this, override HOME dir to writable '/home/chromeos-test'.

(this will also require gsuitl to be configured with the same HOME dir)

BUG=359293, 386416
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278329

[achuithb, 2014-06-19 00:27:39]

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/page/cloud_storage.py (right):

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/cloud_storage.py:27: _CROS_GSUTIL_HOME =
'/home/chromeos-test/'
Rename to _CROS_GSUTIL_HOME_WAR, and add a TODO + comment saying that this is a
workaround for 386416.

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/cloud_storage.py:105: # On cros device, as
telemetry is running as root, home will be set to /root/,
Also reference bug 386416 here and add a TODO to remove this code one we have
gsutil working properly on cros DUT.

[tbarzic, 2014-06-19 00:47:13]

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/page/cloud_storage.py (right):

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/cloud_storage.py:27: _CROS_GSUTIL_HOME =
'/home/chromeos-test/'
On 2014/06/19 00:27:38, achuithb wrote:
> Rename to _CROS_GSUTIL_HOME_WAR, and add a TODO + comment saying that this is
a
> workaround for 386416.

Done.

https://codereview.chromium.org/334603004/diff/60001/tools/telemetry/telemetr...
tools/telemetry/telemetry/page/cloud_storage.py:105: # On cros device, as
telemetry is running as root, home will be set to /root/,
On 2014/06/19 00:27:39, achuithb wrote:
> Also reference bug 386416 here and add a TODO to remove this code one we have
> gsutil working properly on cros DUT.

Done.

Though, I wouldn't be surprised if we ended up doing something similar in the
end, at least for issue 359293.

[achuithb, 2014-06-19 00:49:27]

lgtm

[achuithb, 2014-06-19 00:50:20]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2014-06-19 00:54:06]

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

[commit-bot: I haz the power, 2014-06-19 11:33:01]

Change committed as 278329