Merge pull request #50213 from zhangxiaoyu-zidif/fix-comment-cronjob-utils

Automatic merge from submit-queue (batch tested with PRs 50213, 50707, 49502, 51230, 50848) Fix comment of cronjob utils.go **What this PR does / why we need it**: **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes https://github.com/kubernetes/kubernetes/issues/50951 **Special notes for your reviewer**: **Release note**: ```release-note NONE ```

Merge pull request #50213 from zhangxiaoyu-zidif/fix-comment-cronjob-utils
1d5b365e · Kubernetes Submit Queue · GitHub · 0cf928f6 · 71501d03 · 1d5b365e
Commit 1d5b365e authored Aug 25, 2017 by Kubernetes Submit Queue Committed by GitHub Aug 25, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

utils.go pkg/controller/cronjob/utils.go +1 -3

No files found.
--- a/pkg/controller/cronjob/utils.go
+++ b/pkg/controller/cronjob/utils.go
@@ -169,8 +169,6 @@ func getRecentUnmetScheduleTimes(sj batchv2alpha1.CronJob, now time.Time) ([]tim
 	return starts, nil
 }
-// XXX unit test this
 // getJobFromTemplate makes a Job from a CronJob
 func getJobFromTemplate(sj *batchv2alpha1.CronJob, scheduledTime time.Time) (*batchv1.Job, error) {
 	// TODO: consider adding the following labels:
@@ -200,7 +198,7 @@ func getJobFromTemplate(sj *batchv2alpha1.CronJob, scheduledTime time.Time) (*ba
 	return job, nil
 }
-// Return Unix Epoch Time
+// getTimeHash returns Unix Epoch Time
 func getTimeHash(scheduledTime time.Time) int64 {
 	return scheduledTime.Unix()
 }