commit 38befdd44040e03bbe1c4da1f878ba980eae40ad
author Ravi Mistry <rmistry@google.com> Fri Aug 16 15:36:29 2019 -0400
committer Ravi Mistry <rmistry@google.com> Fri Aug 16 19:37:47 2019 +0000
tree 9d78b277937a26e9177dc12eb312b704bc31f7a0
parent 52eaa331c3d0aae5af097c49f12bf1228d5fce6a

[CT] Make skia_hash optional in csv_comparer.py

Change-Id: I9827cdcea6f3ae1a4927b264b53dc85b4b2b07d5
Reviewed-on: https://skia-review.googlesource.com/c/buildbot/+/235208
Reviewed-by: Ravi Mistry <rmistry@google.com>

ct/py/csv_comparer.py

diff --git a/ct/py/csv_comparer.py b/ct/py/csv_comparer.py
index f90345c..984aa61 100644
--- a/ct/py/csv_comparer.py
+++ b/ct/py/csv_comparer.py
@@ -423,7 +423,8 @@
       help='The chromium git hash that was used for this run.')
   option_parser.add_option(
       '', '--skia_hash',
-      help='The skia git hash that was used for this run.')
+      help='The skia git hash that was used for this run.',
+      default='')
   option_parser.add_option(
       '', '--missing_output_slaves',
       help='Slaves which had no output for this run.')
@@ -444,14 +445,13 @@
           and options.skia_patch_link and options.raw_csv_nopatch
           and options.raw_csv_withpatch and options.num_repeated
           and options.target_platform and options.pageset_type
-          and options.chromium_hash and options.skia_hash
-          and options.description):
+          and options.chromium_hash and options.description):
     option_parser.error('Must specify csv_file1, csv_file2, output_html_dir, '
                         'variance_threshold, requester_email, '
                         'chromium_patch_link, '
                         'skia_patch_link, raw_csv_nopatch, description, '
                         'raw_csv_withpatch, num_repeated, pageset_type, '
-                        'chromium_hash, skia_hash and target_platform')
+                        'chromium_hash and target_platform')
 
   sys.exit(CsvComparer(
       options.csv_file1, options.csv_file2, options.output_html_dir,