verify max retries for sanity

DylanTinianov · DylanTinianov · commit 0c5217296ca8 · 2026-06-16T16:54:04.000-04:00
diff --git a/pkg/capabilities/base_trigger.go b/pkg/capabilities/base_trigger.go
@@ -202,6 +202,10 @@ func (b *BaseTriggerCapability[T]) maxRetries(ctx context.Context) int {
 		b.lggr.Warnw("CRE settings read failed for BaseTriggerMaxRetries; using default (20)", "err", err)
 		return defaultMaxRetries
 	}
+	if v <= 0 {
+		b.lggr.Warnw("CRE settings max retries <= 0; using default (20)", "err", err)
+		v = defaultMaxRetries
+	}
 	return v
 }
 

Original file line number	Diff line number	Diff line change
`@@ -202,6 +202,10 @@ func (b *BaseTriggerCapability[T]) maxRetries(ctx context.Context) int {`
`202`	`202`	`b.lggr.Warnw("CRE settings read failed for BaseTriggerMaxRetries; using default (20)", "err", err)`
`203`	`203`	`return defaultMaxRetries`
`204`	`204`	`}`
	`205`	`+ if v <= 0 {`
	`206`	`+ b.lggr.Warnw("CRE settings max retries <= 0; using default (20)", "err", err)`
	`207`	`+ v = defaultMaxRetries`
	`208`	`+ }`
`205`	`209`	`return v`
`206`	`210`	`}`
`207`	`211`