Skip to content

Commit 00d7b4d

Browse files
committed
[Misc] Operator: Labels/Annotations amended
Clean-up and get rid of some unnecessary labels/annotations. These are ancient leftovers and should be added by consumers if needed anyway!
1 parent 2154b64 commit 00d7b4d

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

internal/controller/reconcile-capapplicationversion.go

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -312,10 +312,6 @@ func (c *Controller) handleContentDeployJob(ca *v1alpha1.CAPApplication, cav *v1
312312

313313
// newContentDeploymentJob creates a Content Deployment Job for the CAV resource. It also sets the appropriate OwnerReferences.
314314
func newContentDeploymentJob(cav *v1alpha1.CAPApplicationVersion, workload *v1alpha1.WorkloadDetails, ownerRef metav1.OwnerReference, vcapSecretName string) *batchv1.Job {
315-
labels := copyMaps(workload.Labels, map[string]string{
316-
LabelDisableKarydia: "true",
317-
})
318-
319315
contentJobName := getContentJobName(workload.Name, cav)
320316

321317
util.LogInfo("Creating content job", string(Processing), cav, nil, "contentJobName", contentJobName, "version", cav.Spec.Version)
@@ -336,7 +332,7 @@ func newContentDeploymentJob(cav *v1alpha1.CAPApplicationVersion, workload *v1al
336332
Template: corev1.PodTemplateSpec{
337333
ObjectMeta: metav1.ObjectMeta{
338334
Annotations: workload.Annotations,
339-
Labels: labels,
335+
Labels: workload.Labels,
340336
},
341337
Spec: corev1.PodSpec{
342338
Containers: []corev1.Container{
@@ -367,7 +363,7 @@ func newContentDeploymentJob(cav *v1alpha1.CAPApplicationVersion, workload *v1al
367363
NodeName: workload.JobDefinition.NodeName,
368364
PriorityClassName: workload.JobDefinition.PriorityClassName,
369365
Affinity: workload.JobDefinition.Affinity,
370-
TopologySpreadConstraints: getTopologySpreadConstraints(workload.JobDefinition.TopologySpreadConstraints, labels, false),
366+
TopologySpreadConstraints: getTopologySpreadConstraints(workload.JobDefinition.TopologySpreadConstraints, workload.Labels, false),
371367
Tolerations: workload.JobDefinition.Tolerations,
372368
},
373369
},

internal/controller/reconcile.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ const (
4040
LabelTenantOperationStep = "sme.sap.com/tenant-operation-step"
4141
LabelCAVVersion = "sme.sap.com/cav-version"
4242
LabelRelevantDNSTarget = "sme.sap.com/relevant-dns-target-hash"
43-
LabelDisableKarydia = "x4.sap.com/disable-karydia"
4443
LabelExposedWorkload = "sme.sap.com/exposed-workload"
4544
LabelDNSNameHash = "sme.sap.com/dns-name-hash"
4645
LabelSubscriptionGUID = "sme.sap.com/subscription-guid"

0 commit comments

Comments
 (0)