From c669ce440c82e96cd1cc130a5bb8bccbc1bc7750 Mon Sep 17 00:00:00 2001
From: Damien Mathieu <42@dmathieu.com>
Date: Thu, 8 Feb 2018 08:32:29 +0100
Subject: [PATCH] remove todo to consider adding the cronjob name as a label

See #59473
---
 pkg/controller/cronjob/utils.go | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pkg/controller/cronjob/utils.go b/pkg/controller/cronjob/utils.go
index d3286d99ced..39b1101cef6 100644
--- a/pkg/controller/cronjob/utils.go
+++ b/pkg/controller/cronjob/utils.go
@@ -173,7 +173,6 @@ func getRecentUnmetScheduleTimes(sj batchv1beta1.CronJob, now time.Time) ([]time
 func getJobFromTemplate(sj *batchv1beta1.CronJob, scheduledTime time.Time) (*batchv1.Job, error) {
 	// TODO: consider adding the following labels:
 	// nominal-start-time=$RFC_3339_DATE_OF_INTENDED_START -- for user convenience
-	// scheduled-job-name=$SJ_NAME -- for user convenience
 	labels := copyLabels(&sj.Spec.JobTemplate)
 	annotations := copyAnnotations(&sj.Spec.JobTemplate)
 	// We want job names for a given nominal start time to have a deterministic name to avoid the same job being created twice