-
Notifications
You must be signed in to change notification settings - Fork 86
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
ignore-versions doesn't seem to work with container packages (ghcr) #88
Comments
Hello @DaveFPM The Also pls note that this action currently doesn't support filtering based on container tags & I suppose you might be looking out for something like that? May I suggest you please go through our marketplace documentation on how this input param works? |
Yes, maybe thats my confusion. I would have thought it would have worked to filter "Tagged image versions". I guess this is not the case, so more of a feature request then? |
@DaveFPM Yes, this can be taken as a new feature request for sure. 👍 |
Yes filtering on container tags would be great! |
@mikhailkoliada I don't understand. If this doesn't filter by tags then what does it do? Tags !== versions? To clarify, I'm trying to effectively purge my images to save space. I only need to keep |
I just came to the same problem as Dave here. There is no mention in documentation anywhere that this action can not / will not parse container tags. Could you add it please? Or even better merge #104 |
@nishthaGupta is this feature being worked on? It would be great for the official action to support containers since GHCR is part of GitHub packages. |
This is a surprising and annoying bug.
Can you please prioritize this as GHCR is an essential part many organizations workflow |
Trying to use
ignore-versions
with the new support for ghcr container package types, but it doesn't seem to work. I am trying to delete beta packages when pushing into the develop branch, and non beta packages when pushing into main. But ATM it just deletes the packages regardless of the name.My full example config is here:
The text was updated successfully, but these errors were encountered: