-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
8614 regex replication filters #18723
base: main
Are you sure you want to change the base?
8614 regex replication filters #18723
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #18723 +/- ##
==========================================
- Coverage 67.45% 67.40% -0.05%
==========================================
Files 996 993 -3
Lines 109773 109021 -752
Branches 2720 2763 +43
==========================================
- Hits 74044 73490 -554
+ Misses 31747 31574 -173
+ Partials 3982 3957 -25
Flags with carried forward coverage won't be shown. Click here to find out more.
|
I am aware that there are print statements in the code. They are there for personal debugging reference and will be removed in the mergeable PR. |
The latest commit edf8c20 removes some unnecessary and error causing Angular properties from the portal. The properties are for radio input for choosing filter mode between regex & double star |
@Vad1mo and @wy65701436 can you review this one please :) |
@WilfredAlmeida please fix DCO |
I'll fix the DCO issue and update here. |
b0294ad
to
84630f1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some tests seem to fail, and you are missing the DCO
@Vad1mo @OrlinVasilev In between my commits, I synced my fork with the main repo. Now when I'm trying to fix DCO it's adding it to commits that are not mine. My first commit was on May 10th and when I do a rebase its rewriting the commits of others as well. Is there a way I can only add DCO to my commits? |
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
…lRegex for policy creation Signed-off-by: Wilfred Almeida <[email protected]>
… creating replication filter rule for the following filter types - Double Star - Regular Expression Signed-off-by: Wilfred Almeida <[email protected]>
…s. The translations for `zh-cn` and `zh-tw` are done using Google's Bard LLM. Rest all except English are done via Google Translate. Signed-off-by: Wilfred Almeida <[email protected]>
Added display text translations for "Filter Type". Translations for `zh-cn` and `zh-tw` are done using Google's Bard LLM. Rest all except English are done via Google Translate. Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
…adio button is enabled. If yes then modify the filter type to be regex. Signed-off-by: Wilfred Almeida <[email protected]>
…ar property tags from replication filters radio input Signed-off-by: Wilfred Almeida <[email protected]>
84630f1
to
03f0e55
Compare
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
@Vad1mo @OrlinVasilev I've fixed the DCO and other occurring issues. Please let me know if any more actions are required from my side. |
This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days. |
Still relevant |
@WilfredAlmeida thanks for your contribution. To support regex in Harbor, I propose creating a shared library of regex patterns, similar to the double star we currently use. This library would apply across all Harbor functionalities, including replication and tag retention, making it easier to enable consistent functionality for tag retention. If you’re still willing to work on this pull request, @chlins and I will do our best to assist you from a technical perspective. |
Hello @wy65701436, currently I don't have the bandwidth to work this and won't be able to do so anytime soon |
Thanks, @WilfredAlmeida, for your response. Given this, I believe we can temporarily close this pull request until we have someone from the community who can assist. |
This PR aims to add regular expression based replication rules feature to Harbor.
Issue: 8614
Community Proposal: 221
The filter functions for regex-based filtering are defined in the
src/pkg/reg/filter/artifact.go
file alongside the existing filtering functions using the double star logic.Necessary schema properties are added in the
policy.go
file.The decision on the finalized approach to add the feature is discussed in the community proposal 221