feat: added exclude filter for purge command#428
Open
sashokbg wants to merge 1 commit intoAzure:mainfrom
Open
Conversation
sashokbg
requested review from
Wwwsylvia,
estebanreyl,
northtyphoon,
sajayantony,
shizhMSFT,
wangxiaoxuan273 and
wju-MSFT
as code owners
sashokbg
force-pushed
the
feat/added_purge_exclude_filter
branch
from
a0b56e1 to
b91358e
Compare
estebanreyl
requested changes
May 21, 2025
| return -1, -1, err | ||
| } | ||
|
|
||
| var excludeRegex *regexp2.Regexp = nil |
Contributor
There was a problem hiding this comment.
Nit. Assigning nil to a pointer is unnecessary
| | Untag all tags that are older than the duration in app repository | --filter `"app:.*"` | | ||
| | Untag all tags that are older than the duration in all repositories | --filter `".*:.*"` | | ||
|
|
||
| ##### Exclude filter tag |
Contributor
There was a problem hiding this comment.
I think we can keep this at 4#
|
|
||
| ##### Exclude filter tag | ||
|
|
||
| To exclude tags from the purging, use the --exclude-filter parameter. Exclude filters take precedence over filters. |
Contributor
There was a problem hiding this comment.
Can you note that these are optional?
| if excludeFilter != "" { | ||
| res, err := common.BuildRegexFilter(excludeFilter, regexMatchTimeout) | ||
| if err != nil { | ||
| return -1, -1, err |
Contributor
There was a problem hiding this comment.
Same as above, can you add the error wrap?
| var excludeRegex *regexp2.Regexp = nil | ||
|
|
||
| if excludeFilter != "" { | ||
| res, err := common.BuildRegexFilter(excludeFilter, regexpMatchTimeoutSeconds) |
Contributor
There was a problem hiding this comment.
Can you add an error wrap, just to make it clearer? Like fmt.Errorf("could not build excludeFilter regex: %w:" same for the actual filter since this change will make it a little unclear as to which failed to build.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose of the PR
Adds a new parameter "--exclude-filter" that will override any matches in the "--filter".
Useful when we want to purge everything except some tags (ex: dev, prod, release etc)
Fixes #
#60 #59