From 254f0f1f33a7cbbd2d07f7f850bef02bf03751cb Mon Sep 17 00:00:00 2001 From: Paul Jolly Date: Sun, 19 Mar 2023 23:14:34 +0000 Subject: [PATCH] internal/ci: do not evict caches on push to ci/test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Pushing to ci/test is a useful thing for trying out changes to various workflows. But the evict caches workflow is quite destructive. An innocuous push to ci/test to try out a change in, say, the trybot workflow would blast the actions caches in the main CUE repo and the trybot repo. Hence, do not run evict_caches on a push to ci/test. Signed-off-by: Paul Jolly Change-Id: Iee9727697975c6a8d9d610de7d5ee2b5cf76afd8 Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/551252 TryBot-Result: CUEcueckoo Reviewed-by: Daniel Martí --- .github/workflows/evict_caches.yml | 3 --- internal/ci/github/evict_caches.cue | 3 --- 2 files changed, 6 deletions(-) diff --git a/.github/workflows/evict_caches.yml b/.github/workflows/evict_caches.yml index 91447f7ec85..9160c8feac6 100644 --- a/.github/workflows/evict_caches.yml +++ b/.github/workflows/evict_caches.yml @@ -2,9 +2,6 @@ name: Evict caches "on": - push: - branches: - - ci/test schedule: - cron: 0 2 * * * jobs: diff --git a/internal/ci/github/evict_caches.cue b/internal/ci/github/evict_caches.cue index 7e17dbc0c37..c570b92d19e 100644 --- a/internal/ci/github/evict_caches.cue +++ b/internal/ci/github/evict_caches.cue @@ -44,9 +44,6 @@ evict_caches: _base.#bashWorkflow & { name: "Evict caches" on: { - push: { - branches: [_base.#testDefaultBranch] - } schedule: [ // We will run a schedule trybot build 15 minutes later to repopulate the caches {cron: "0 2 * * *"},