Skip to content

Commit

Permalink
Merge pull request #321 from p0lyn0mial/fix_encryption_deployer_conve…
Browse files Browse the repository at this point in the history
…rged

logs when the encryption deployer hasn't yet converged
  • Loading branch information
openshift-merge-robot committed Aug 10, 2020
2 parents c0f61dd + fe76a2b commit 377c358
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import (
"time"

metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/klog"

operatorv1 "github.com/openshift/api/operator/v1"
operatorconfigclient "github.com/openshift/client-go/operator/clientset/versioned/typed/operator/v1"
Expand Down Expand Up @@ -63,7 +64,8 @@ func syncManageAPIServicesController(deployer statemachine.Deployer, authOperato
return err
}
if !converged && !operator.Status.ManagingOAuthAPIServer {
syncContext.Recorder().Warning("PrereqNotReady", "the encryption deployer hasn't yet converged, retrying in 5 minutes")
// !converged will also be reported during the oauth apiserver installation
klog.V(2).Infof("the encryption deployer hasn't yet converged, retrying in 5 minutes")
syncContext.Queue().AddAfter(syncContext.QueueKey(), 5*time.Minute)
return nil
}
Expand Down

0 comments on commit 377c358

Please sign in to comment.