Skip to content

Commit

Permalink
Merge pull request #1569 from rzetelskik/job-sync-error-fix
Browse files Browse the repository at this point in the history
Fix error aggregation in scyllacluster controller job sync
  • Loading branch information
scylla-operator-bot[bot] authored Nov 14, 2023
2 parents d7601ad + f674ffb commit 04bf07f
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions pkg/controller/scyllacluster/sync_jobs.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ import (
corev1 "k8s.io/api/core/v1"
apimeta "k8s.io/apimachinery/pkg/api/meta"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
utilerrors "k8s.io/apimachinery/pkg/util/errors"
)

func (scc *Controller) syncJobs(
Expand Down Expand Up @@ -86,6 +87,10 @@ func (scc *Controller) syncJobs(
})
}
}
err = utilerrors.NewAggregate(errs)
if err != nil {
return progressingConditions, err
}

return progressingConditions, nil
}

0 comments on commit 04bf07f

Please sign in to comment.