opm validate: make sure skipped bundles are not included in replaces chain

joelanford · joelanford · commit a00c1908b3a0 · 2025-08-20T17:03:43.000-04:00
Signed-off-by: Joe Lanford &lt;joe.lanford@gmail.com&gt;
diff --git a/alpha/model/model.go b/alpha/model/model.go
@@ -288,6 +288,9 @@ func (c *Channel) validateReplacesChain() error {
 		if _, ok := chainFrom[cur.Name]; !ok {
 			chainFrom[cur.Name] = []string{cur.Name}
 		}
+		if skippedBundles.Has(cur.Replaces) {
+			break
+		}
 		for k := range chainFrom {
 			chainFrom[k] = append(chainFrom[k], cur.Replaces)
 		}

Original file line number	Diff line number	Diff line change
`@@ -288,6 +288,9 @@ func (c *Channel) validateReplacesChain() error {`
`288`	`288`	`if _, ok := chainFrom[cur.Name]; !ok {`
`289`	`289`	`chainFrom[cur.Name] = []string{cur.Name}`
`290`	`290`	`}`
	`291`	`+ if skippedBundles.Has(cur.Replaces) {`
	`292`	`+ break`
	`293`	`+ }`
`291`	`294`	`for k := range chainFrom {`
`292`	`295`	`chainFrom[k] = append(chainFrom[k], cur.Replaces)`
`293`	`296`	`}`