Reflect API review: fix +listType and improve some API descriptions

Author: KobayashiD27

keps/sig-scheduling/5007-device-attach-before-pod-scheduled/README.md

@@ -343,25 +343,37 @@ These fields will be used by the controller that exposes the `ResourceSlice` to
 ```go
 // BasicDevice represents a basic device instance.
 type BasicDevice struct {
-    // BindingConditions defines the conditions for proceeding with binding. All listed conditions must be True to proceed with binding. 
+  ...
+    // UsageRestrictedToNode indicates if the usage of an allocation involving this device
+    // has to be limited to exactly the node that was chosen when allocating the claim.
     //
     // +optional
-    BindingConditions []string
+    UsageRestrictedToNode bool
 
-    // BindingFailureConditions defines the conditions for binding failure.
-    // If true, a binding failure occurred.
+    // BindingConditions defines the conditions for proceeding with binding.
+    // All of these conditions must be set in the per-device status
+    // conditions with a value of True to proceed with binding the pod to the node
+    // while scheduling the pod.
+    // The maximum number of binding conditions is 4.
     //
     // +optional
-    BindingFailureConditions []string
+    // +listType=atomic
+    BindingConditions []string
 
-    // UsageRestrictedToNode indicates if the usage of an allocation involving this device
-    // has to be limited to exactly the node that was chosen when allocating the claim.
+    // BindingFailureConditions defines the conditions for binding failure.
+    // They may be set in the per-device status conditions.
+    // If any is true, a binding failure occurred.
+    // The maximum number of binding failure conditions is 4.
     //
     // +optional
-    UsageRestrictedToNode bool
+    // +listType=atomic
+    BindingFailureConditions []string
 
     // BindingTimeout indicates the prepare timeout period.
-    // If the timeout period is exceeded before all BindingConditions reach a True state, the scheduler clears the allocation in the ResourceClaim and reschedules the Pod.
+    // If the timeout period is exceeded before all BindingConditions reach a True state,
+    // the scheduler clears the allocation in the ResourceClaim and reschedules the Pod.
+    //
+    // The default timeout if not set is 10 minutes.
     //
     // +optional
     BindingTimeout *metav1.Duration
@@ -384,26 +396,36 @@ For this feature, following fields are added:
 // AllocatedDeviceStatus contains the status of an allocated device, if the
 // driver chooses to report it. This may include driver-specific information.
 type AllocatedDeviceStatus struct {
-    ...
-    // BindingConditions defines the conditions for proceeding with binding. All listed conditions must be True to proceed with binding. 
+  ...
+    // UsageRestrictedToNode is a copy of the UsageRestrictedToNode
+    // as defined for the device at the time when it was allocated.
+    // If true, the node selector of the allocation matches exactly
+    // the node that was chosen for the pod which triggered allocation.
+    //
+    // +optional
+    UsageRestrictedToNode bool
+  
+    // BindingConditions is a copy of the BindingConditions
+    // as defined for the device at the time when it was allocated.
+    // All of these conditions must be to True to proceed with binding the pod to the node
+    // while scheduling the pod.
     //
     // +optional
     BindingConditions []string
 
-    // BindingFailureConditions defines the conditions for binding failure.
+    // BindingFailureConditions is a copy of the BindingFailureConditions
+    // as defined for the device at the time when it was allocated.
     // If any is True, a binding failure occurred.
     //
     // +optional
     BindingFailureConditions []string
 
-    // UsageRestrictedToNode indicates if the usage of an allocation involving this device
-    // has to be limited to exactly the node that was chosen when allocating the claim.
+    // BindingTimeout is a copy of the BindingTimeout
+    // as defined for the device at the time when it was allocated.
+    // If the timeout period is exceeded before all BindingConditions reach a True state, 
+    // the scheduler clears the allocation in the ResourceClaim and reschedules the Pod.
     //
-    // +optional
-    UsageRestrictedToNode bool
-
-    // BindingTimeout indicates the prepare timeout period.
-    // If the timeout period is exceeded before all BindingConditions reach a True state, the scheduler clears the allocation in the ResourceClaim and reschedules the Pod.
+    // The default timeout if not set is 10 minutes.
     //
     // +optional
     BindingTimeout *metav1.Duration