Merge pull request #122493 from mengjiao-liu/fix-cronjob-error
cronjob: fix incorrect error checking
This commit is contained in:
@@ -203,8 +203,8 @@ func (jm *ControllerV2) sync(ctx context.Context, cronJobKey string) (*time.Dura
|
|||||||
updateStatusAfterCleanup := jm.cleanupFinishedJobs(ctx, cronJobCopy, jobsToBeReconciled)
|
updateStatusAfterCleanup := jm.cleanupFinishedJobs(ctx, cronJobCopy, jobsToBeReconciled)
|
||||||
|
|
||||||
requeueAfter, updateStatusAfterSync, syncErr := jm.syncCronJob(ctx, cronJobCopy, jobsToBeReconciled)
|
requeueAfter, updateStatusAfterSync, syncErr := jm.syncCronJob(ctx, cronJobCopy, jobsToBeReconciled)
|
||||||
if err != nil {
|
if syncErr != nil {
|
||||||
logger.V(2).Info("Error reconciling cronjob", "cronjob", klog.KObj(cronJob), "err", err)
|
logger.V(2).Info("Error reconciling cronjob", "cronjob", klog.KObj(cronJob), "err", syncErr)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Update the CronJob if needed
|
// Update the CronJob if needed
|
||||||
|
Reference in New Issue
Block a user