address review comment: fix pointer ref

Signed-off-by: Swati Gupta <swatig@nvidia.com>

Author: guptaNswati

cmd/dra-example-kubeletplugin/state.go

@@ -26,7 +26,6 @@ import (
 	resourceapi "k8s.io/api/resource/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
-	metav1apply "k8s.io/client-go/applyconfigurations/meta/v1"
 	resourceapply "k8s.io/client-go/applyconfigurations/resource/v1"
 	"k8s.io/klog/v2"
 	drapbv1 "k8s.io/kubelet/pkg/apis/dra/v1beta1"
@@ -215,7 +214,7 @@ func (s *DeviceState) prepareDevices(ctx context.Context, claim *resourceapi.Res
 			return nil, fmt.Errorf("requested GPU is not allocatable: %v", result.Device)
 		}
 
-		deviceStatus := s.buildDeviceStatus(&result)
+		deviceStatus := s.buildDeviceStatus(result)
 		devicesStatus = append(devicesStatus, deviceStatus)
 
 		for _, c := range slices.Backward(configs) {
@@ -401,23 +400,19 @@ func GetOpaqueDeviceConfigs(
 	return resultConfigs, nil
 }
 
-func (s *DeviceState) buildDeviceStatus(res *resourceapi.DeviceRequestAllocationResult) *resourceapply.AllocatedDeviceStatusApplyConfiguration {
+func (s *DeviceState) buildDeviceStatus(res resourceapi.DeviceRequestAllocationResult) *resourceapply.AllocatedDeviceStatusApplyConfiguration {
 	dn := res.Device
-	deviceInfo := make(map[string]interface{})
+	deviceInfo := make(map[string]resourceapi.DeviceAttribute)
 
 	if d, ok := s.allocatable[dn]; ok {
-		if d.Attributes != nil {
-			attributes := d.Attributes
-
-			if uuid, ok := attributes["uuid"]; ok {
-				deviceInfo["uuid"] = uuid
-			}
-			if model, ok := attributes["model"]; ok {
-				deviceInfo["model"] = model
-			}
-			if driverVersion, ok := attributes["driverVersion"]; ok {
-				deviceInfo["driverVersion"] = driverVersion
-			}
+		if uuid, ok := d.Attributes["uuid"]; ok {
+			deviceInfo["uuid"] = uuid
+		}
+		if model, ok := d.Attributes["model"]; ok {
+			deviceInfo["model"] = model
+		}
+		if driverVersion, ok := d.Attributes["driverVersion"]; ok {
+			deviceInfo["driverVersion"] = driverVersion
 		}
 	}
 
@@ -429,18 +424,18 @@ func (s *DeviceState) buildDeviceStatus(res *resourceapi.DeviceRequestAllocation
 	data := runtime.RawExtension{
 		Raw: jsonBytes,
 	}
-	cond := metav1apply.Condition().
-		WithType("Ready").
-		WithStatus(metav1.ConditionTrue).
-		WithReason("GPUDeviceReady").
-		WithMessage("GPUDeviceAllocated").
-		WithLastTransitionTime(metav1.Now())
 
 	return resourceapply.AllocatedDeviceStatus().
 		WithDevice(dn).
 		WithDriver(res.Driver).
 		WithPool(res.Pool).
-		WithConditions(cond).
+		// WithData records per-allocation metadata used for monitoring and debugging:
+		//   - Pod→GPU mapping: makes it easier to see which GPU a given pod is using,
+		//     which is not readily available elsewhere.
+		//   - Device attributes (e.g. UUID, model, driverVersion): remain available
+		//     even if the device is later removed from a ResourceSlice (for example,
+		//     because it becomes unhealthy), so past allocations can still be
+		//     correlated with later health or scheduling issues.
 		WithData(data)
 }