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

replication rule filter with label not been updated via UI #19839

Closed
MinerYang opened this issue Jan 12, 2024 · 0 comments · Fixed by #19850
Closed

replication rule filter with label not been updated via UI #19839

MinerYang opened this issue Jan 12, 2024 · 0 comments · Fixed by #19850

Comments

@MinerYang
Copy link
Contributor

If you are reporting a problem, please make sure the following information are provided:

Expected behavior and actual behavior:
when editing a existing replication rule with label filter and save, the rule not been updated via UI , request still send filter with label

Steps to reproduce the problem:

  • add a replication rule with label filter
Screen Shot 2024-01-12 at 16 25 00
  • edit the rule removing label and save show update succeeded.
Screen Shot 2024-01-12 at 16 25 25 Screen Shot 2024-01-12 at 16 25 30
  • check the rule , will see the label still exist after update
Screen Shot 2024-01-12 at 16 27 08

Versions:

  • harbor version: v2.10.0-969dd1be

Additional context:
the update request still send label as a filter

{"name":"test","description":null,"src_registry":null,"dest_registry":{"creation_time":"2024-01-09T08:28:21.392Z","credential":{"access_key":"admin","access_secret":"*****","type":"basic"},"id":1,"insecure":true,"name":"test","status":"healthy","type":"harbor","update_time":"2024-01-09T09:08:18.838Z","url":"https://harbor.10.83.35.133.nip.io"},"dest_namespace":null,"dest_namespace_replace_count":-1,"trigger":{"type":"manual","trigger_settings":{"cron":""}},"enabled":true,"deletion":null,"override":true,"speed":-1,"copy_by_chunk":false,"filters":[{"type":"name","value":"{library,goharbor}/**"},{"type":"label","decoration":"matches","value":["critical-image-test"]}]}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants