From 3cbb4ef3ee8bdf999952437e514fac509e9ae0b5 Mon Sep 17 00:00:00 2001 From: d-kuro Date: Fri, 28 Feb 2020 14:28:27 +0900 Subject: [PATCH 1/2] Fix ownerReferences.apiVersion for kubectl create job --from=cronjob Kubernetes-commit: b78ab3515e85bb391e6cadc3374da7ac1992db1f --- pkg/cmd/create/create_job.go | 3 +-- pkg/cmd/create/create_job_test.go | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/cmd/create/create_job.go b/pkg/cmd/create/create_job.go index a295b6a0..364b9a2a 100644 --- a/pkg/cmd/create/create_job.go +++ b/pkg/cmd/create/create_job.go @@ -22,7 +22,6 @@ import ( "github.com/spf13/cobra" - appsv1 "k8s.io/api/apps/v1" batchv1 "k8s.io/api/batch/v1" batchv1beta1 "k8s.io/api/batch/v1beta1" corev1 "k8s.io/api/core/v1" @@ -259,7 +258,7 @@ func (o *CreateJobOptions) createJobFromCronJob(cronJob *batchv1beta1.CronJob) * Annotations: annotations, Labels: cronJob.Spec.JobTemplate.Labels, OwnerReferences: []metav1.OwnerReference{ - *metav1.NewControllerRef(cronJob, appsv1.SchemeGroupVersion.WithKind("CronJob")), + *metav1.NewControllerRef(cronJob, cronJob.GroupVersionKind()), }, }, Spec: cronJob.Spec.JobTemplate.Spec, diff --git a/pkg/cmd/create/create_job_test.go b/pkg/cmd/create/create_job_test.go index 585d4529..df2df9dd 100644 --- a/pkg/cmd/create/create_job_test.go +++ b/pkg/cmd/create/create_job_test.go @@ -20,7 +20,6 @@ import ( "strings" "testing" - apps "k8s.io/api/apps/v1" batchv1 "k8s.io/api/batch/v1" batchv1beta1 "k8s.io/api/batch/v1beta1" corev1 "k8s.io/api/core/v1" @@ -165,7 +164,7 @@ func TestCreateJobFromCronJob(t *testing.T) { ObjectMeta: metav1.ObjectMeta{ Name: jobName, Annotations: map[string]string{"cronjob.kubernetes.io/instantiate": "manual"}, - OwnerReferences: []metav1.OwnerReference{*metav1.NewControllerRef(cronJob, apps.SchemeGroupVersion.WithKind("CronJob"))}, + OwnerReferences: []metav1.OwnerReference{*metav1.NewControllerRef(cronJob, cronJob.GroupVersionKind())}, }, Spec: batchv1.JobSpec{ Template: corev1.PodTemplateSpec{