fix: remove used-by label on workload (#250)

Author: 0x5457

internal/constants/constants.go

@@ -14,7 +14,6 @@ const (
 	Finalizer       = Domain + "/" + FinalizerSuffix
 
 	LabelKeyOwner           = Domain + "/managed-by"
-	LabelKeyUser            = Domain + "/used-by"
 	LabelKeyClusterOwner    = Domain + "/cluster"
 	LabelKeyNodeClass       = Domain + "/node-class"
 	LabelKeyPodTemplateHash = Domain + "/pod-template-hash"

internal/gpuallocator/gpuallocator.go

@@ -564,7 +564,7 @@ func (s *GpuAllocator) markGPUDirty(key types.NamespacedName) {
 	s.dirtyQueue[key] = struct{}{}
 }
 
-func (s *GpuAllocator) markGPUDirtyLoced(key types.NamespacedName) {
+func (s *GpuAllocator) markGPUDirtyLocked(key types.NamespacedName) {
 	s.dirtyQueue[key] = struct{}{}
 }
 
@@ -632,7 +632,7 @@ func (s *GpuAllocator) reconcileAllocationState(ctx context.Context) {
 		if !sameTflops || !sameVRAM {
 			gpu.Status.Available.Tflops = tflopsCapacityMap[gpuKey]
 			gpu.Status.Available.Vram = vramCapacityMap[gpuKey]
-			s.markGPUDirtyLoced(gpuKey)
+			s.markGPUDirtyLocked(gpuKey)
 			log.FromContext(ctx).Info("Correcting gpu available resources", "gpu", gpuKey.Name, "tflops", gpu.Status.Available.Tflops.String(), "vram", gpu.Status.Available.Vram.String())
 		}
 	}

internal/webhook/v1/pod_webhook.go

@@ -213,12 +213,6 @@ func (m *TensorFusionPodMutator) createOrUpdateWorkload(ctx context.Context, pod
 			},
 		}
 
-		// Add related Deployment's ReplicaSet for frontend to get related client side workload for this TensorFusionWorkload
-		// TODO: support multiple client workloads using the same TF workload
-		if len(pod.OwnerReferences) > 0 {
-			workload.Labels[constants.LabelKeyUser] = pod.OwnerReferences[0].Kind + "_" + pod.OwnerReferences[0].Name
-		}
-
 		if rootOwnerRef != nil {
 			workload.OwnerReferences = []metav1.OwnerReference{*rootOwnerRef}
 		}