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

Iseyer/replicate tags #21660

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ianseyer
Copy link

@ianseyer ianseyer commented Feb 21, 2025

Currently, harbor does not replicate image tags for event-based replications. Instead, if a new tag is pushed, a replication event is not fired.

This issue is easily replicated:

  • push an image with a tag
  • push the same image with a new tag
  • watch replication not happen

There should be followup work to correct the issue of webhooks not being fired for tags being created.

Issue being fixed

Fixes #21623

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.

@ianseyer ianseyer requested a review from a team as a code owner February 21, 2025 17:07
@ianseyer ianseyer force-pushed the iseyer/replicate-tags branch 2 times, most recently from 548afb9 to ba628f6 Compare February 21, 2025 17:09
@ianseyer ianseyer force-pushed the iseyer/replicate-tags branch from ba628f6 to 634b662 Compare February 21, 2025 17:10
@ianseyer
Copy link
Author

@Vad1mo @wy65701436 @reasonerjt

if you would be so kind as to review

@Vad1mo Vad1mo enabled auto-merge (squash) February 22, 2025 14:06
Copy link

codecov bot commented Feb 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 46.30%. Comparing base (c8c11b4) to head (634b662).
Report is 392 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #21660      +/-   ##
==========================================
+ Coverage   45.36%   46.30%   +0.93%     
==========================================
  Files         244      249       +5     
  Lines       13333    14047     +714     
  Branches     2719     2897     +178     
==========================================
+ Hits         6049     6504     +455     
- Misses       6983     7199     +216     
- Partials      301      344      +43     
Flag Coverage Δ
unittests 46.30% <ø> (+0.93%) ⬆️

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

see 493 files with indirect coverage changes

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.

"Event Based" replication fails to replicate all tags, "Scheduled" or "Manual" replicates properly.
5 participants