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

fix: event-based replication deletion not work when policy with label #21215

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

chlins
Copy link
Member

@chlins chlins commented Nov 19, 2024

Fix event-based replication deletion on remote registry not triggered when the replication policy configured the label filter.

Thank you for contributing to Harbor!

Comprehensive Summary of your change

Issue being fixed

Fixes #21213

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

@chlins chlins added the release-note/update Update or Fix label Nov 19, 2024
@chlins chlins requested a review from a team as a code owner November 19, 2024 04:33
@chlins chlins added the target/2.13.0 issues that are targeting v2.13.0 label Nov 19, 2024
Copy link

codecov bot commented Nov 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.21%. Comparing base (c8c11b4) to head (5757553).
Report is 335 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #21215      +/-   ##
==========================================
+ Coverage   45.36%   46.21%   +0.84%     
==========================================
  Files         244      247       +3     
  Lines       13333    13863     +530     
  Branches     2719     2867     +148     
==========================================
+ Hits         6049     6407     +358     
- Misses       6983     7119     +136     
- Partials      301      337      +36     
Flag Coverage Δ
unittests 46.21% <ø> (+0.84%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 491 files with indirect coverage changes

@chlins chlins force-pushed the fix/replication-deletion-on-label branch from f0a7b9a to 39807ae Compare November 19, 2024 07:00
Copy link
Contributor

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chlins chlins force-pushed the fix/replication-deletion-on-label branch from 39807ae to 41576dd Compare November 22, 2024 06:21
src/server/v2.0/handler/artifact.go Outdated Show resolved Hide resolved
@chlins chlins force-pushed the fix/replication-deletion-on-label branch 2 times, most recently from 777bbc1 to 8c2fdd4 Compare November 25, 2024 08:59
@chlins chlins enabled auto-merge (squash) November 25, 2024 09:00
… filter

Fix event-based replication deletion on remote registry not triggered
when the replication policy configured the label filter.

Signed-off-by: chlins <[email protected]>
@chlins chlins force-pushed the fix/replication-deletion-on-label branch from 8c2fdd4 to 5757553 Compare November 26, 2024 01:53
@chlins chlins merged commit 05233b0 into goharbor:main Nov 26, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/update Update or Fix target/2.13.0 issues that are targeting v2.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

replication with label source resource filter is not deleting images
5 participants