Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Unified Diff: infra/bots/recipes/swarm_RecreateSKPs.py

Issue 2408793002: RecreateSKPs: Don't download and use the .boto file (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | infra/bots/recipes/swarm_RecreateSKPs.expected/Housekeeper-Nightly-RecreateSKPs_Canary.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/bots/recipes/swarm_RecreateSKPs.py
diff --git a/infra/bots/recipes/swarm_RecreateSKPs.py b/infra/bots/recipes/swarm_RecreateSKPs.py
index 38fd8534b355e6ad42a6586fcb8f6b107043523b..d7f05948e7873b5c9fdb1d596d9cf2ca28c274df 100644
--- a/infra/bots/recipes/swarm_RecreateSKPs.py
+++ b/infra/bots/recipes/swarm_RecreateSKPs.py
@@ -110,28 +110,6 @@ def RunSteps(api):
['ninja', '-C', out_dir, 'chrome'],
cwd=src_dir)
- # Download boto file (needed by recreate_skps.py) to tmp dir.
- boto_file = api.path['slave_build'].join('tmp', '.boto')
- api.python.inline(
- 'download boto file',
- """
-import os
-import urllib2
-
-BOTO_URL = 'http://metadata/computeMetadata/v1/project/attributes/boto-file'
-
-dest_path = '%s'
-dest_dir = os.path.dirname(dest_path)
-if not os.path.exists(dest_dir):
- os.makedirs(dest_dir)
-
-req = urllib2.Request(BOTO_URL, headers={'Metadata-Flavor': 'Google'})
-contents = urllib2.urlopen(req).read()
-
-with open(dest_path, 'w') as f:
- f.write(contents)
- """ % boto_file)
-
# Clean up the output dir.
output_dir = api.path['slave_build'].join('skp_output')
if api.path.exists(output_dir):
@@ -144,13 +122,8 @@ with open(dest_path, 'w') as f:
'CHROME_HEADLESS': '1',
'PATH': path_var,
}
- boto_env = {
- 'AWS_CREDENTIAL_FILE': boto_file,
- 'BOTO_CONFIG': boto_file,
- }
recreate_skps_env = {}
rmistry 2016/10/10 17:09:29 Delete these 2 lines and set env=env on line 137.
borenet 2016/10/10 17:17:47 Done.
recreate_skps_env.update(env)
- recreate_skps_env.update(boto_env)
asset_dir = api.vars.infrabots_dir.join('assets', 'skp')
cmd = ['python', asset_dir.join('create.py'),
'--chrome_src_path', src_dir,
« no previous file with comments | « no previous file | infra/bots/recipes/swarm_RecreateSKPs.expected/Housekeeper-Nightly-RecreateSKPs_Canary.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698