|
|
DescriptionSet BOTO_CONFIG correctly in cloud_storage.py
When running the telemetry perf tests on a VM host as root, the .boto was not
being found.
This patch simply sets the BOTO_CONFIG environment variable to point at
the correct location.
Bug: chromium:712349
Review-Url: https://codereview.chromium.org/2923123006
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/23ac69b72e05e6d1788c915353bdd75dd596f1e7
Patch Set 1 #
Messages
Total messages: 20 (11 generated)
Description was changed from ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: 712349 ========== to ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: 712349 ==========
lpique@chromium.org changed reviewers: + nednguyen@google.com
PTAL.
lgtm
The CQ bit was checked by lpique@chromium.org
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)
Description was changed from ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: 712349 ========== to ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: crbug:712349 ==========
The CQ bit was checked by lpique@chromium.org
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: Catapult Presubmit on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Pr...)
Description was changed from ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: crbug:712349 ========== to ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: chromium:712349 ==========
The CQ bit was checked by lpique@chromium.org
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...
CQ is committing da patch. Bot data: {"patchset_id": 1, "attempt_start_ts": 1496785611108620, "parent_rev": "dc460e541ad9d8ce33e8adc3aa46bd283ee26e39", "commit_rev": "23ac69b72e05e6d1788c915353bdd75dd596f1e7"}
Message was sent while issue was closed.
Description was changed from ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: chromium:712349 ========== to ========== Set BOTO_CONFIG correctly in cloud_storage.py When running the telemetry perf tests on a VM host as root, the .boto was not being found. This patch simply sets the BOTO_CONFIG environment variable to point at the correct location. Bug: chromium:712349 Review-Url: https://codereview.chromium.org/2923123006 Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapu... ==========
Message was sent while issue was closed.
Committed patchset #1 (id:1) as https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2931173002/ by lpique@chromium.org. The reason for reverting is: It turns out this did not fix the issue on the buildbot. The .boto file might be on the buildbot at this path, but the tests that are failing are running inside a VM, and don't have access to it in the first place. Another approach is needed to fix the linked bug, and we will likely put a .boto file in the home directory that is being set when running gsutil.. |