Skip to content

Commit

Permalink
Merge pull request #102 from layer5io/kumarabd/feature/broker
Browse files Browse the repository at this point in the history
fixes for the docker image
  • Loading branch information
kumarabd authored Jan 6, 2021
2 parents daaedf3 + 673ab72 commit 0065516
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion config/manifests/default.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,7 @@ spec:
- --enable-leader-election
command:
- /manager
image: layer5/meshery-operator:stable-latest
image: layer5io/meshery-operator:stable-latest
imagePullPolicy: Always
name: manager
ports:
Expand Down
6 changes: 5 additions & 1 deletion pkg/broker/broker.go
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,11 @@ func CheckHealth(ctx context.Context, m *mesheryv1alpha1.Broker, client *kuberne
}

if obj.Status.Replicas != obj.Status.ReadyReplicas {
return ErrReplicasNotReady(obj.Status.Conditions[0].Reason)
if len(obj.Status.Conditions) > 0 {

return ErrReplicasNotReady(obj.Status.Conditions[0].Reason)
}
return ErrReplicasNotReady("Condition Unknown")
}

if len(obj.Status.Conditions) > 0 && (obj.Status.Conditions[0].Status == corev1.ConditionFalse || obj.Status.Conditions[0].Status == corev1.ConditionUnknown) {
Expand Down

0 comments on commit 0065516

Please sign in to comment.