Use tf_job_name not tf_job_key as the label name.

* tf_job_name is consistent with v1alpha1; this desirable from a compatibility
  perspective.

* tf_job_key isn't quite right because the job key includes namespace
  but we are no longer using the namespace in the value

Related to kubeflow#672

pkg/generator/generator.go

@@ -27,7 +27,7 @@ import (
 
 const (
 	LabelGroupName = "group_name"
-	labelTFJobKey  = "tf_job_key"
+	labelTFJobName  = "tf_job_name"
 )
 
 var (
@@ -48,10 +48,10 @@ func GenOwnerReference(tfjob *tfv1alpha2.TFJob) *metav1.OwnerReference {
 	return controllerRef
 }
 
-func GenLabels(tfjobKey string) map[string]string {
+func GenLabels(tfJobName string) map[string]string {
 	return map[string]string{
 		LabelGroupName: tfv1alpha2.GroupName,
-		labelTFJobKey:  strings.Replace(tfjobKey, "/", "-", -1),
+		labelTFJobName:  strings.Replace(tfJobName, "/", "-", -1),
 	}
 }
 

pkg/generator/generator_test.go

@@ -52,8 +52,8 @@ func TestGenLabels(t *testing.T) {
 
 	labels := GenLabels(testKey)
 
-	if labels[labelTFJobKey] != expctedKey {
-		t.Errorf("Expected %s %s, got %s", labelTFJobKey, expctedKey, labels[labelTFJobKey])
+	if labels[labelTFJobName] != expctedKey {
+		t.Errorf("Expected %s %s, got %s", labelTFJobName, expctedKey, labels[labelTFJobName])
 	}
 	if labels[LabelGroupName] != tfv1alpha2.GroupName {
 		t.Errorf("Expected %s %s, got %s", LabelGroupName, tfv1alpha2.GroupName, labels[LabelGroupName])