Skip to content

Commit

Permalink
Remove pod count as it isn't needed to check pods are ready
Browse files Browse the repository at this point in the history
  • Loading branch information
CathalOConnorRH committed Mar 24, 2020
1 parent f8d6023 commit 3d64efb
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions pkg/products/amqstreams/reconciler.go
Expand Up @@ -214,12 +214,7 @@ func (r *Reconciler) handleProgressPhase(ctx context.Context, client k8sclient.C
return integreatlyv1alpha1.PhaseFailed, fmt.Errorf("failed to check amq streams installation: %w", err)
}

//expecting 8 pods in total
if len(pods.Items) < 8 {
return integreatlyv1alpha1.PhaseInProgress, nil
}

//and they should all be ready
//checking pods are all ready
checkPodStatus:
for _, pod := range pods.Items {
for _, cnd := range pod.Status.Conditions {
Expand Down

0 comments on commit 3d64efb

Please sign in to comment.