clean up project metadata for tag retention policy after deletion (goharbor#22174)

MinerYang · AYDEV-FR · commit f7979a8a3702 · 2025-09-19T22:18:46.000+02:00
Signed-off-by: my036811 &lt;miner.yang@broadcom.com&gt;
diff --git a/src/server/v2.0/handler/retention.go b/src/server/v2.0/handler/retention.go
@@ -260,6 +260,10 @@ func (r *retentionAPI) DeleteRetention(ctx context.Context, params operation.Del
 	if err = r.retentionCtl.DeleteRetention(ctx, params.ID); err != nil {
 		return r.SendError(ctx, err)
 	}
+	// delete retention data in project_metadata
+	if err := r.proMetaMgr.Delete(ctx, p.Scope.Reference, "retention_id"); err != nil {
+		return r.SendError(ctx, err)
+	}
 	return operation.NewDeleteRetentionOK()
 }
 

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,10 @@ func (r *retentionAPI) DeleteRetention(ctx context.Context, params operation.Del`
`260`	`260`	`if err = r.retentionCtl.DeleteRetention(ctx, params.ID); err != nil {`
`261`	`261`	`return r.SendError(ctx, err)`
`262`	`262`	`}`
	`263`	`+ // delete retention data in project_metadata`
	`264`	`+ if err := r.proMetaMgr.Delete(ctx, p.Scope.Reference, "retention_id"); err != nil {`
	`265`	`+ return r.SendError(ctx, err)`
	`266`	`+ }`
`263`	`267`	`return operation.NewDeleteRetentionOK()`
`264`	`268`	`}`
`265`	`269`