[Bookmaker bot] Output fiddleout.json earlier and do not send to CQ for now
Bug: skia:7310
Change-Id: Id6c2f15b72d6837f6536e25dbc3d01cac32af74b
Reviewed-on: https://skia-review.googlesource.com/83640
Reviewed-by: Ravi Mistry <rmistry@google.com>
Commit-Queue: Ravi Mistry <rmistry@google.com>
diff --git a/infra/bots/recipes/bookmaker.py b/infra/bots/recipes/bookmaker.py
index 34f3494..62d53dd 100644
--- a/infra/bots/recipes/bookmaker.py
+++ b/infra/bots/recipes/bookmaker.py
@@ -79,6 +79,10 @@
content = api.file.read_text('Read fiddleout.json',
fiddlecli_output, test_data=test_data)
out = json.loads(content)
+ # Do a dump of fiddlecli_output. Will be useful for debugging.
+ print 'Dump of %s:' % fiddlecli_output
+ print json.dumps(out, indent=4)
+
failing_fiddles = []
for fiddle_name in out:
props = out[fiddle_name]
@@ -91,10 +95,6 @@
failure_msg += 'https://fiddle.skia.org/c/%s\n' % fiddle_hash
raise api.step.StepFailure(failure_msg)
- # Do a dump of fiddlecli_output. Will be useful for debugging.
- print 'Dump of %s:' % fiddlecli_output
- print json.dumps(out, indent=4)
-
# Step 4: Update docs in site/user/api/ with the output of fiddlecli.
# If there are any new changes then upload and commit the changes.
update_docs_gitcookies = api.path['start_dir'].join(
diff --git a/infra/bots/upload_md.py b/infra/bots/upload_md.py
index 6ee5ba1..fee4a1b 100644
--- a/infra/bots/upload_md.py
+++ b/infra/bots/upload_md.py
@@ -74,7 +74,7 @@
sys.exit(1)
if diff_files:
subprocess.check_call(['git', 'add', '-u'])
- git_branch.commit_and_upload(True)
+ git_branch.commit_and_upload(False)
else:
print 'No changes so nothing to upload.'