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

Unified Diff: slave/skia_slave_scripts/utils/sync_bucket_subdir.py

Issue 304393002: Adds uploading picture benchmark JSON data (Closed) Base URL: https://skia.googlesource.com/buildbot.git@master
Patch Set: Created 6 years, 7 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
Index: slave/skia_slave_scripts/utils/sync_bucket_subdir.py
diff --git a/slave/skia_slave_scripts/utils/sync_bucket_subdir.py b/slave/skia_slave_scripts/utils/sync_bucket_subdir.py
index 01de8b6c98022c6f65135f914dd40849b944ae72..10465e83fff54e3c1f369a76dc08c98ae17792f8 100755
--- a/slave/skia_slave_scripts/utils/sync_bucket_subdir.py
+++ b/slave/skia_slave_scripts/utils/sync_bucket_subdir.py
@@ -30,6 +30,7 @@ IGNORE_UPLOAD_FILENAMES = ('.DS_Store')
def SyncBucketSubdir(directory, dest_gsbase=DEFAULT_PERFDATA_GS_BASE, subdir='',
do_upload=True, do_download=True, filenames_filter=KNOWN_FILENAMES,
+ exclude_json=True,
benchen 2014/05/29 22:20:09 Do we upload json files somewhere else, like gm? A
kelvinly 2014/05/30 00:44:51 Ah, good catch. Fixed
min_download_revision=0):
""" synchronizes a local directory with a cloud one
@@ -85,6 +86,9 @@ def SyncBucketSubdir(directory, dest_gsbase=DEFAULT_PERFDATA_GS_BASE, subdir='',
if not match:
# Ignore other files, rather than raising an exception
continue
+ # Ignore JSON files if the flag is set.
+ if exclude_json and file_name.endswith('.json'):
+ continue
# Ignore force builds without a revision number.
if match.group(1) != '':
upload_to_bucket.upload_to_bucket(os.path.join(directory, file_name),

Powered by Google App Engine
This is Rietveld 408576698