chore: Don't use large runners for CLI workflow (#14692)

erezrokah · web-flow · commit 36ec245e03bc · 2023-10-19T09:01:35.000Z
#### Summary

CLI workflows are not CPU bound, so we could save some money and use regular runners

&lt;!--
diff --git a/.github/workflows/cli.yml b/.github/workflows/cli.yml
@@ -27,7 +27,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        os: [large-ubuntu-monorepo, macos-latest-xl, large-windows-monorepo]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
       - uses: actions/checkout@v4
         with:
@@ -40,7 +40,7 @@ jobs:
           cache-dependency-path: cli/go.sum
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v3
-        if: matrix.os == 'large-ubuntu-monorepo'
+        if: matrix.os == 'ubuntu-latest'
         with:
           version: v1.54.2
           working-directory: cli
@@ -51,14 +51,14 @@ jobs:
       - name: Test
         run: make test
       - name: gen-docs
-        if: matrix.os == 'large-ubuntu-monorepo'
+        if: matrix.os == 'ubuntu-latest'
         run: make gen-docs
       - name: Fail if docs are changed
-        if: matrix.os == 'large-ubuntu-monorepo'
+        if: matrix.os == 'ubuntu-latest'
         run: test "$(git status -s ../website/pages/docs/reference/cli/*.md | wc -l)" -eq 0
   validate-release:
     timeout-minutes: 30
-    runs-on: large-ubuntu-monorepo
+    runs-on: ubuntu-latest
     env:
       CGO_ENABLED: 0
     steps:
diff --git a/scripts/workflows/wait_for_required_workflows.js b/scripts/workflows/wait_for_required_workflows.js
@@ -41,7 +41,7 @@ module.exports = async ({github, context}) => {
     // We test the CLI on multiple OSes, so we need to wait for all of them
     if (actions.includes("cli")) {
         actions = actions.filter(action => action !== "cli")
-        actions = ["cli (large-ubuntu-monorepo)", "cli (large-windows-monorepo)", "cli (macos-latest-xl)", ...actions]
+        actions = ["cli (ubuntu-latest)", "cli (windows-latest)", "cli (macos-latest)", ...actions]
     }
 
     // Enforce policy tests for AWS, Azure, GCP and K8s plugins

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ module.exports = async ({github, context}) => {`
`41`	`41`	`// We test the CLI on multiple OSes, so we need to wait for all of them`
`42`	`42`	`if (actions.includes("cli")) {`
`43`	`43`	`actions = actions.filter(action => action !== "cli")`
`44`		`- actions = ["cli (large-ubuntu-monorepo)", "cli (large-windows-monorepo)", "cli (macos-latest-xl)", ...actions]`
	`44`	`+ actions = ["cli (ubuntu-latest)", "cli (windows-latest)", "cli (macos-latest)", ...actions]`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`// Enforce policy tests for AWS, Azure, GCP and K8s plugins`