[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>

infra/bots/recipes/bookmaker.py

@@ -79,6 +79,10 @@ def RunSteps(api):
       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 @@ def RunSteps(api):
           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(

infra/bots/upload_md.py

@@ -74,7 +74,7 @@ def main():
             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.'