Add enabled flag for try builders (#20460)

keyonghan · web-flow · commit 9cd99d2655dd · 2020-08-13T13:36:18.000-07:00
* add enabled flag

* update doc

* update doc
diff --git a/ci/dev/README.md b/ci/dev/README.md
@@ -8,16 +8,25 @@ supported luci try/prod builders for engine. It follows format:
 {
     "builders":[
         {
-            "name":"xxx",
-            "repo":"engine"
-        },
+            "name":"yyy",
+            "repo":"engine",
+            "enabled":true
+        }
+    ]
+}
+```
+for `try_builders.json`, and follows format:
+```json
+{
+    "builders":[
         {
             "name":"yyy",
             "repo":"engine"
         }
     ]
 }
 ```
-This file will be mainly used in [`flutter/cocoon`](https://github.com/flutter/cocoon)
-to trigger/update engine luci tasks.
-
+for `prod_builders.json`. `try_builders.json` will be mainly used in 
+[`flutter/cocoon`](https://github.com/flutter/cocoon) to trigger/update pre-submit
+engine luci tasks, whereas `prod_builders.json` will be mainly used in `flutter/cocoon`
+to push luci task statuses to GitHub.
diff --git a/ci/dev/try_builders.json b/ci/dev/try_builders.json
@@ -2,55 +2,68 @@
    "builders":[
       {
          "name":"Linux Host Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Linux Fuchsia",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Linux Android AOT Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Linux Android Debug Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Linux Web Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Mac Host Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Mac Android AOT Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Mac Android Debug Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Mac iOS Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Windows Host Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Windows Android AOT Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Windows Web Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       },
       {
          "name":"Mac Web Engine",
-         "repo":"engine"
+         "repo":"engine",
+         "enabled": true
       }
    ]
 }

Original file line number	Diff line number	Diff line change
`@@ -8,16 +8,25 @@ supported luci try/prod builders for engine. It follows format:`
`8`	`8`	`{`
`9`	`9`	`"builders":[`
`10`	`10`	`{`
`11`		`- "name":"xxx",`
`12`		`- "repo":"engine"`
`13`		`- },`
	`11`	`+ "name":"yyy",`
	`12`	`+ "repo":"engine",`
	`13`	`+ "enabled":true`
	`14`	`+ }`
	`15`	`+ ]`
	`16`	`+}`
	`17`	+```
	`18`	+for `try_builders.json`, and follows format:
	`19`	+```json
	`20`	`+{`
	`21`	`+ "builders":[`
`14`	`22`	`{`
`15`	`23`	`"name":"yyy",`
`16`	`24`	`"repo":"engine"`
`17`	`25`	`}`
`18`	`26`	`]`
`19`	`27`	`}`
`20`	`28`	```
`21`		-This file will be mainly used in [`flutter/cocoon`](https://github.com/flutter/cocoon)
`22`		`-to trigger/update engine luci tasks.`
`23`		`-`
	`29`	+for `prod_builders.json`. `try_builders.json` will be mainly used in
	`30`	+[`flutter/cocoon`](https://github.com/flutter/cocoon) to trigger/update pre-submit
	`31`	+engine luci tasks, whereas `prod_builders.json` will be mainly used in `flutter/cocoon`
	`32`	`+to push luci task statuses to GitHub.`
Original file line number	Diff line number	Diff line change
`@@ -2,55 +2,68 @@`
`2`	`2`	`"builders":[`
`3`	`3`	`{`
`4`	`4`	`"name":"Linux Host Engine",`
`5`		`- "repo":"engine"`
	`5`	`+ "repo":"engine",`
	`6`	`+ "enabled": true`
`6`	`7`	`},`
`7`	`8`	`{`
`8`	`9`	`"name":"Linux Fuchsia",`
`9`		`- "repo":"engine"`
	`10`	`+ "repo":"engine",`
	`11`	`+ "enabled": true`
`10`	`12`	`},`
`11`	`13`	`{`
`12`	`14`	`"name":"Linux Android AOT Engine",`
`13`		`- "repo":"engine"`
	`15`	`+ "repo":"engine",`
	`16`	`+ "enabled": true`
`14`	`17`	`},`
`15`	`18`	`{`
`16`	`19`	`"name":"Linux Android Debug Engine",`
`17`		`- "repo":"engine"`
	`20`	`+ "repo":"engine",`
	`21`	`+ "enabled": true`
`18`	`22`	`},`
`19`	`23`	`{`
`20`	`24`	`"name":"Linux Web Engine",`
`21`		`- "repo":"engine"`
	`25`	`+ "repo":"engine",`
	`26`	`+ "enabled": true`
`22`	`27`	`},`
`23`	`28`	`{`
`24`	`29`	`"name":"Mac Host Engine",`
`25`		`- "repo":"engine"`
	`30`	`+ "repo":"engine",`
	`31`	`+ "enabled": true`
`26`	`32`	`},`
`27`	`33`	`{`
`28`	`34`	`"name":"Mac Android AOT Engine",`
`29`		`- "repo":"engine"`
	`35`	`+ "repo":"engine",`
	`36`	`+ "enabled": true`
`30`	`37`	`},`
`31`	`38`	`{`
`32`	`39`	`"name":"Mac Android Debug Engine",`
`33`		`- "repo":"engine"`
	`40`	`+ "repo":"engine",`
	`41`	`+ "enabled": true`
`34`	`42`	`},`
`35`	`43`	`{`
`36`	`44`	`"name":"Mac iOS Engine",`
`37`		`- "repo":"engine"`
	`45`	`+ "repo":"engine",`
	`46`	`+ "enabled": true`
`38`	`47`	`},`
`39`	`48`	`{`
`40`	`49`	`"name":"Windows Host Engine",`
`41`		`- "repo":"engine"`
	`50`	`+ "repo":"engine",`
	`51`	`+ "enabled": true`
`42`	`52`	`},`
`43`	`53`	`{`
`44`	`54`	`"name":"Windows Android AOT Engine",`
`45`		`- "repo":"engine"`
	`55`	`+ "repo":"engine",`
	`56`	`+ "enabled": true`
`46`	`57`	`},`
`47`	`58`	`{`
`48`	`59`	`"name":"Windows Web Engine",`
`49`		`- "repo":"engine"`
	`60`	`+ "repo":"engine",`
	`61`	`+ "enabled": true`
`50`	`62`	`},`
`51`	`63`	`{`
`52`	`64`	`"name":"Mac Web Engine",`
`53`		`- "repo":"engine"`
	`65`	`+ "repo":"engine",`
	`66`	`+ "enabled": true`
`54`	`67`	`}`
`55`	`68`	`]`
`56`	`69`	`}`