Skip to content

Commit

Permalink
internal/ci: do not evict caches on push to ci/test
Browse files Browse the repository at this point in the history
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 <paul@myitcv.io>
Change-Id: Iee9727697975c6a8d9d610de7d5ee2b5cf76afd8
Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/551252
TryBot-Result: CUEcueckoo <cueckoo@cuelang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
  • Loading branch information
myitcv committed Mar 19, 2023
1 parent ee81e77 commit 254f0f1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
3 changes: 0 additions & 3 deletions .github/workflows/evict_caches.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,6 @@

name: Evict caches
"on":
push:
branches:
- ci/test
schedule:
- cron: 0 2 * * *
jobs:
Expand Down
3 changes: 0 additions & 3 deletions internal/ci/github/evict_caches.cue
Original file line number Diff line number Diff line change
Expand Up @@ -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 * * *"},
Expand Down

0 comments on commit 254f0f1

Please sign in to comment.