-
Notifications
You must be signed in to change notification settings - Fork 885
ci: delete old workflow runs #8146
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
Conversation
This cleans up old workflow runs for `pr-deply.yaml` More workflows can be added by appending to the list e.g. `delete_workflow_pattern: ["pr-deploy.yaml", "pr-cleanup.yaml"]`
I find the old, failed workflows helpful when investigating flakiness and performance trends. |
Can we only clean up for |
Why do you want to cleanup at all? What bothers you about the current system? |
@ammario https://github.com/coder/coder/actions/workflows/pr-cleanup.yaml |
Oh, I misunderstood the purpose of this PR. |
This cleans up old workflow runs for
pr-cleanup.yaml
as this action runs on every PR close event, so we have a lot of failed workflow logs as it tried to clean up every pr, and we do not have deployments for every pr.More workflows can be added by appending to the list e.g.
delete_workflow_pattern: ["pr-deploy.yaml", "pr-cleanup.yaml"]