diff --git a/pkg/operator/controllers/pullsecret/pullsecret_controller.go b/pkg/operator/controllers/pullsecret/pullsecret_controller.go index b90400df1d1..c57d84c3cbb 100644 --- a/pkg/operator/controllers/pullsecret/pullsecret_controller.go +++ b/pkg/operator/controllers/pullsecret/pullsecret_controller.go @@ -184,15 +184,22 @@ func (r *Reconciler) ensureGlobalPullSecret(ctx context.Context, operatorSecret, // allows for simpler logic flow, when delete and create are not handled separately // this call happens only when there is a need to change, it has no significant impact on performance err := r.client.Delete(ctx, secret) + r.log.Info("Global Pull secret Not Found, Creating Again") if err != nil && !kerrors.IsNotFound(err) { return nil, err } err = r.client.Create(ctx, secret) + if err == nil { + r.log.Info("Global Pull secret Created") + } return secret, err } err = r.client.Update(ctx, secret) + if err == nil { + r.log.Info("Updated Existing Global Pull secret") + } return secret, err } diff --git a/pkg/operator/controllers/pullsecret/pullsecret_controller_test.go b/pkg/operator/controllers/pullsecret/pullsecret_controller_test.go index 4d02241b837..8be8b80bde3 100644 --- a/pkg/operator/controllers/pullsecret/pullsecret_controller_test.go +++ b/pkg/operator/controllers/pullsecret/pullsecret_controller_test.go @@ -768,6 +768,7 @@ func TestEnsureGlobalPullSecret(t *testing.T) { r := &Reconciler{ client: clientBuilder.Build(), + log: logrus.NewEntry(logrus.StandardLogger()), } s, err := r.ensureGlobalPullSecret(ctx, tt.operatorPullSecret, tt.pullSecret)