Skip to content

Commit 4745cd5

Browse files
[release-1.13] Remove sinkbindings OIDC token secret when not needed and fix status setting (#7839)
* Remove Sinkbindings OIDC token secret, when not needed * Set Sinkbindings OIDCTokenSecretName in its status correctly --------- Co-authored-by: Christoph Stäbler <[email protected]>
1 parent d045c98 commit 4745cd5

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

pkg/reconciler/sinkbinding/sinkbinding.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,10 @@ func (s *SinkBindingSubResourcesReconciler) Reconcile(ctx context.Context, b psb
114114
// sink has no audience set -> don't create token secret
115115
sb.Status.MarkOIDCIdentityCreatedSucceededWithReason("Sink has no audience defined", "")
116116
sb.Status.MarkOIDCTokenSecretCreatedSuccceededWithReason("Sink has no audience defined", "")
117+
118+
if err := s.removeOIDCTokenSecretEventually(ctx, sb); err != nil {
119+
return err
120+
}
117121
sb.Status.OIDCTokenSecretName = nil
118122
}
119123
} else {
@@ -164,6 +168,8 @@ func (s *SinkBindingSubResourcesReconciler) reconcileOIDCTokenSecret(ctx context
164168
logger.Debugf("OIDC token secret for %s/%s sinkbinding still valid for > %s (expires %s). Will not update secret", sb.Name, sb.Namespace, resyncAndBufferDuration, expiry)
165169
// token is still valid for resync period + buffer --> we're fine
166170

171+
sb.Status.OIDCTokenSecretName = &secretName
172+
167173
return nil
168174
}
169175

0 commit comments

Comments
 (0)