Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Update clear cache action to be smarter #13405

Merged
merged 17 commits into from
Aug 28, 2024
Merged

Conversation

mydea
Copy link
Member

@mydea mydea commented Aug 16, 2024

Previously, we had a CI action to manually clear all caches.

This PR adjusts this so this action can be used in a more granular way:

  • By default, the action will now delete caches of any PR runs that are successful, as well as any caches of release branches.
  • You can configure to also delete caches on the develop branch, and/or to also delete non-successful PR branches.

Additionally, this action will run every midnight, to automatically clear completed/outdated stuff.

The goal is to keep develop caches as long as possible, and clear out other caches, unless they failed (which indicates you may want to re-run some of the tests) and unless they are currently running (to not break ongoing tests). Ideally, we do not need to manually run this, but can rely on automated cleanup over night.

@mydea mydea self-assigned this Aug 16, 2024
Copy link
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 22.5 KB - -
@sentry/browser (incl. Tracing) 34.85 KB - -
@sentry/browser (incl. Tracing, Replay) 71.2 KB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 64.47 KB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 75.55 KB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 88.28 KB - -
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 90.12 KB - -
@sentry/browser (incl. metrics) 26.81 KB - -
@sentry/browser (incl. Feedback) 39.57 KB - -
@sentry/browser (incl. sendFeedback) 27.17 KB - -
@sentry/browser (incl. FeedbackAsync) 31.88 KB - -
@sentry/react 25.26 KB - -
@sentry/react (incl. Tracing) 37.83 KB - -
@sentry/vue 26.65 KB - -
@sentry/vue (incl. Tracing) 36.67 KB - -
@sentry/svelte 22.64 KB - -
CDN Bundle 23.76 KB +0.05% +10 B 🔺
CDN Bundle (incl. Tracing) 36.51 KB +0.02% +5 B 🔺
CDN Bundle (incl. Tracing, Replay) 70.85 KB +0.02% +8 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 76.16 KB +0.01% +7 B 🔺
CDN Bundle - uncompressed 69.69 KB +0.07% +49 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 108.35 KB +0.05% +49 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 219.73 KB +0.03% +49 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 232.92 KB +0.03% +49 B 🔺
@sentry/nextjs (client) 37.59 KB - -
@sentry/sveltekit (client) 35.45 KB - -
@sentry/node 115.71 KB -0.01% -1 B 🔽
@sentry/node - without tracing 90.1 KB +0.01% +1 B 🔺
@sentry/aws-serverless 99.51 KB - -

View base workflow run

@mydea mydea merged commit 3a1417f into develop Aug 28, 2024
128 checks passed
@mydea mydea deleted the fn/better-clear-cache branch August 28, 2024 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants