Debug nightly bookmaker bot

* Output fiddle.json
* Do not send to CQ while we investigate

Bug: skia:8151
Change-Id: I309c732cc6b7db72054fd119b8c68dd32a765d6f
Reviewed-on: https://skia-review.googlesource.com/142301
Commit-Queue: Ravi Mistry <rmistry@google.com>
Reviewed-by: Cary Clark <caryclark@google.com>

infra/bots/recipes/bookmaker.expected/nightly_bookmaker.json

@@ -152,6 +152,20 @@
     },
     "name": "Extract all fiddles out of md files"
   },
+  {
+    "cmd": [
+      "cat",
+      "[START_DIR]/fiddle.json"
+    ],
+    "cwd": "[START_DIR]/cache/work/skia",
+    "env": {
+      "CHROME_HEADLESS": "1",
+      "GOPATH": "[START_DIR]/gopath",
+      "GOROOT": "[START_DIR]/go/go",
+      "PATH": "[START_DIR]/go/go/bin:[START_DIR]/gopath:<PATH>:RECIPE_PACKAGE_REPO[depot_tools]"
+    },
+    "name": "Output fiddle.json"
+  },
   {
     "cmd": [
       "[START_DIR]/gopath/bin/fiddlecli",

infra/bots/recipes/bookmaker.expected/nightly_failed_fiddlecli.json

@@ -152,6 +152,20 @@
     },
     "name": "Extract all fiddles out of md files"
   },
+  {
+    "cmd": [
+      "cat",
+      "[START_DIR]/fiddle.json"
+    ],
+    "cwd": "[START_DIR]/cache/work/skia",
+    "env": {
+      "CHROME_HEADLESS": "1",
+      "GOPATH": "[START_DIR]/gopath",
+      "GOROOT": "[START_DIR]/go/go",
+      "PATH": "[START_DIR]/go/go/bin:[START_DIR]/gopath:<PATH>:RECIPE_PACKAGE_REPO[depot_tools]"
+    },
+    "name": "Output fiddle.json"
+  },
   {
     "cmd": [
       "[START_DIR]/gopath/bin/fiddlecli",

infra/bots/recipes/bookmaker.expected/nightly_failed_fiddles.json

@@ -152,6 +152,20 @@
     },
     "name": "Extract all fiddles out of md files"
   },
+  {
+    "cmd": [
+      "cat",
+      "[START_DIR]/fiddle.json"
+    ],
+    "cwd": "[START_DIR]/cache/work/skia",
+    "env": {
+      "CHROME_HEADLESS": "1",
+      "GOPATH": "[START_DIR]/gopath",
+      "GOROOT": "[START_DIR]/go/go",
+      "PATH": "[START_DIR]/go/go/bin:[START_DIR]/gopath:<PATH>:RECIPE_PACKAGE_REPO[depot_tools]"
+    },
+    "name": "Output fiddle.json"
+  },
   {
     "cmd": [
       "[START_DIR]/gopath/bin/fiddlecli",

infra/bots/recipes/bookmaker.expected/nightly_failed_upload.json

@@ -152,6 +152,20 @@
     },
     "name": "Extract all fiddles out of md files"
   },
+  {
+    "cmd": [
+      "cat",
+      "[START_DIR]/fiddle.json"
+    ],
+    "cwd": "[START_DIR]/cache/work/skia",
+    "env": {
+      "CHROME_HEADLESS": "1",
+      "GOPATH": "[START_DIR]/gopath",
+      "GOROOT": "[START_DIR]/go/go",
+      "PATH": "[START_DIR]/go/go/bin:[START_DIR]/gopath:<PATH>:RECIPE_PACKAGE_REPO[depot_tools]"
+    },
+    "name": "Output fiddle.json"
+  },
   {
     "cmd": [
       "[START_DIR]/gopath/bin/fiddlecli",

infra/bots/recipes/bookmaker.py

@@ -78,6 +78,10 @@ def RunSteps(api):
              ]
       api.run(api.step, 'Extract all fiddles out of md files', cmd=cmd)
 
+      # Output fiddle.json for easy debugging.
+      api.run(api.step, 'Output fiddle.json',
+              cmd=['cat', fiddlecli_input])
+
       # Step 2: Forces fiddle.skia.org to compile all fiddles extracted out of
       #         markdown files and get output in JSON.
       cmd = [fiddlecli_binary,

infra/bots/upload_md.py

@@ -33,7 +33,7 @@ def main():
   with git_utils.NewGitCheckout(repository=SKIA_REPO):
     with git_utils.GitBranch(branch_name='update_md_files',
                              commit_msg=COMMIT_MSG,
-                             commit_queue=True,
+                             commit_queue=False,
                              upload=False,
                              cc_list=CC_LIST) as git_branch:
       # Run bookmaker binary.