From be70217e85e906aeb5bc7a875cc826b2278149b1 Mon Sep 17 00:00:00 2001 From: Mike Jensen Date: Thu, 30 Mar 2023 12:22:26 -0600 Subject: [PATCH] Remove `push` workflow for jobs that already run on PR and merge (#23843) In an attempt to reduce our Actions usage this PR removes the workflow execution for `push` actions on several jobs. The following files were left as an exception to make sure flakey tests are discovered: * integration-tests-non-root.yaml * integration-tests-root.yaml * unit-tests-code.yaml * unit-tests-integrations.yaml --- .github/workflows/build-macos.yaml | 3 --- .github/workflows/build-windows.yaml | 3 --- .github/workflows/doc-tests.yaml | 3 --- .github/workflows/lint-bypass.yaml | 3 --- .github/workflows/lint-ui.yaml | 3 --- .github/workflows/lint.yaml | 3 --- .github/workflows/os-compatibility-test-bypass.yaml | 3 --- .github/workflows/os-compatibility-test.yaml | 4 ---- .github/workflows/terraform-lint.yaml | 8 -------- .github/workflows/trivy.yaml | 4 ---- .github/workflows/unit-tests-helm.yaml | 3 --- .github/workflows/unit-tests-rust.yaml | 3 --- .github/workflows/unit-tests-ui.yaml | 3 --- 13 files changed, 46 deletions(-) diff --git a/.github/workflows/build-macos.yaml b/.github/workflows/build-macos.yaml index 53e07ff5cee6c..0861d5cef66e2 100644 --- a/.github/workflows/build-macos.yaml +++ b/.github/workflows/build-macos.yaml @@ -2,9 +2,6 @@ name: Build on Mac OS run-name: Build on Mac OS on: - push: - branches: - - master pull_request: paths: - '**.go' diff --git a/.github/workflows/build-windows.yaml b/.github/workflows/build-windows.yaml index dcfae9a44954e..2034816ec1838 100644 --- a/.github/workflows/build-windows.yaml +++ b/.github/workflows/build-windows.yaml @@ -2,9 +2,6 @@ name: Build on Windows run-name: Build on Windows on: - push: - branches: - - master pull_request: # We only build tsh on Windows so only consider Go code as tsh doesn't # run any Rust. diff --git a/.github/workflows/doc-tests.yaml b/.github/workflows/doc-tests.yaml index 16383af8f4f7a..31033ef1e8986 100644 --- a/.github/workflows/doc-tests.yaml +++ b/.github/workflows/doc-tests.yaml @@ -1,9 +1,6 @@ name: Lint (Docs) run-name: Lint (Docs) on: - push: - branches: - - master pull_request: paths: - 'docs/**' diff --git a/.github/workflows/lint-bypass.yaml b/.github/workflows/lint-bypass.yaml index f93bc03cbb7fe..e10bfebe0e02b 100644 --- a/.github/workflows/lint-bypass.yaml +++ b/.github/workflows/lint-bypass.yaml @@ -10,9 +10,6 @@ name: Lint (Go) run-name: make lint on: - push: - branches: - - master pull_request: paths: - 'docs/**' diff --git a/.github/workflows/lint-ui.yaml b/.github/workflows/lint-ui.yaml index d39714a9fef2d..f71ca3248cbb8 100644 --- a/.github/workflows/lint-ui.yaml +++ b/.github/workflows/lint-ui.yaml @@ -2,9 +2,6 @@ name: Lint UI run-name: Lint UI - ${{ github.run_id }} - @${{ github.actor }} on: - push: - branches: - - master pull_request: paths: - 'web/**' diff --git a/.github/workflows/lint.yaml b/.github/workflows/lint.yaml index 179a282b6aaa6..55a3b1a07c86e 100644 --- a/.github/workflows/lint.yaml +++ b/.github/workflows/lint.yaml @@ -1,9 +1,6 @@ name: Lint (Go) run-name: make lint on: - push: - branches: - - master pull_request: paths-ignore: - 'docs/**' diff --git a/.github/workflows/os-compatibility-test-bypass.yaml b/.github/workflows/os-compatibility-test-bypass.yaml index 8f865b8e91aeb..54f1a11abb7b3 100644 --- a/.github/workflows/os-compatibility-test-bypass.yaml +++ b/.github/workflows/os-compatibility-test-bypass.yaml @@ -10,9 +10,6 @@ name: OS Compatibility Test run-name: OS Compatibility Test on: - push: - branches: - - master pull_request: paths: - 'docs/**' diff --git a/.github/workflows/os-compatibility-test.yaml b/.github/workflows/os-compatibility-test.yaml index 2d97b7af62f65..960160fc6af8c 100644 --- a/.github/workflows/os-compatibility-test.yaml +++ b/.github/workflows/os-compatibility-test.yaml @@ -1,10 +1,6 @@ name: OS Compatibility Test run-name: OS Compatibility Test on: - push: - branches: - - master - - branch/* pull_request: paths-ignore: - 'docs/**' diff --git a/.github/workflows/terraform-lint.yaml b/.github/workflows/terraform-lint.yaml index 82bb9781f80e2..6de96afd42005 100644 --- a/.github/workflows/terraform-lint.yaml +++ b/.github/workflows/terraform-lint.yaml @@ -1,14 +1,6 @@ name: Lint (Terraform) on: - push: - branches: - - master - - branch/* - paths: - - '**.tf' - - '**.tf.json' - - '**.hcl' pull_request: paths: - '**.tf' diff --git a/.github/workflows/trivy.yaml b/.github/workflows/trivy.yaml index 044879ebfb28c..7b207844a5850 100644 --- a/.github/workflows/trivy.yaml +++ b/.github/workflows/trivy.yaml @@ -1,10 +1,6 @@ name: Trivy on: - push: - branches: - - master - - branch/* pull_request: merge_group: diff --git a/.github/workflows/unit-tests-helm.yaml b/.github/workflows/unit-tests-helm.yaml index afa620b7eed74..65f72e3e2a8e7 100644 --- a/.github/workflows/unit-tests-helm.yaml +++ b/.github/workflows/unit-tests-helm.yaml @@ -2,9 +2,6 @@ name: Unit Tests (Helm) run-name: Unit Tests (Helm) - ${{ github.run_id }} - @${{ github.actor }} on: - push: - branches: - - master pull_request: paths: - 'examples/chart/**' diff --git a/.github/workflows/unit-tests-rust.yaml b/.github/workflows/unit-tests-rust.yaml index 4d003fc7038f8..919e307c3ca8d 100644 --- a/.github/workflows/unit-tests-rust.yaml +++ b/.github/workflows/unit-tests-rust.yaml @@ -2,9 +2,6 @@ name: Unit Tests (Rust) run-name: Unit Tests (Rust) - ${{ github.run_id }} - @${{ github.actor }} on: - push: - branches: - - master pull_request: paths: - '**.rs' diff --git a/.github/workflows/unit-tests-ui.yaml b/.github/workflows/unit-tests-ui.yaml index d877c60eedbdb..f98d99a6eaf29 100644 --- a/.github/workflows/unit-tests-ui.yaml +++ b/.github/workflows/unit-tests-ui.yaml @@ -2,9 +2,6 @@ name: Unit Tests UI run-name: Unit Tests UI - ${{ github.run_id }} - @${{ github.actor }} on: - push: - branches: - - master pull_request: paths: - 'web/**'