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

🌱 Prerelease prune #51

Closed

Conversation

savitharaghunathan
Copy link
Member

This workflow deletes prerelease and associated tags older than 7 days.

@djzager @dymurray do you think we need the prerelease artifacts more than a week?

Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
Signed-off-by: Savitha Raghunathan <saveetha13@gmail.com>
@djzager
Copy link
Member

djzager commented Oct 11, 2024

I haven't really given a lot of thought to this. Having a rolling window doesn't seem to be a bad idea but I am also somewhat afraid to throw away the older bits.

What do you think about running the workflow once per week, say Sunday, throwing away all but the newest?

@savitharaghunathan
Copy link
Member Author

I haven't really given a lot of thought to this. Having a rolling window doesn't seem to be a bad idea but I am also somewhat afraid to throw away the older bits.

What do you think about running the workflow once per week, say Sunday, throwing away all but the newest?

One of the reasons I kept a rolling window is to see if we need time to test a build or two.
Are you saying we run it on every sunday and retain only the previous week's builds? that is sunday date - 7?

@savitharaghunathan
Copy link
Member Author

also with every day nightly release, whenever someone fetches upstream its gonna be a lot of tags if we dont purge. personally, I think it would be not helpful to see that many tags of prerelease version. May be I am wrong when I just fetch the whole of upstream and not everyone does that way.

@savitharaghunathan
Copy link
Member Author

closing this as we have changed the workflow - #63

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