chore: address cache client review feedback

Signed-off-by: CAICAIIs <3360776475@qq.com>

Author: CAICAIIs

pkg/ddc/cache/engine/status.go

@@ -72,7 +72,7 @@ func (e *CacheEngine) setWorkerComponentStatus(componentValue *common.CacheRunti
 
 	return ready, err
 }
-func (e *CacheEngine) setClientComponentStatus(componentValue *common.CacheRuntimeComponentValue, status *fluidapi.CacheRuntimeStatus) (ready bool, err error) {
+func (e *CacheEngine) setClientComponentStatus(componentValue *common.CacheRuntimeComponentValue, status *fluidapi.CacheRuntimeStatus) (fullyReady bool, err error) {
 	manager := component.NewComponentHelper(componentValue.WorkloadType, e.Scheme, e.Client)
 
 	clientStatus, err := manager.ConstructComponentStatus(context.TODO(), componentValue)
@@ -81,22 +81,22 @@ func (e *CacheEngine) setClientComponentStatus(componentValue *common.CacheRunti
 	}
 	if clientStatus.DesiredReplicas > 0 && clientStatus.ReadyReplicas >= clientStatus.DesiredReplicas {
 		clientStatus.Phase = fluidapi.RuntimePhaseReady
-		ready = true
+		fullyReady = true
 	} else if clientStatus.ReadyReplicas > 0 {
 		clientStatus.Phase = fluidapi.RuntimePhasePartialReady
 	} else {
 		clientStatus.Phase = fluidapi.RuntimePhaseNotReady
 	}
 	status.Client = clientStatus
 
-	return ready, nil
+	return fullyReady, nil
 }
 func (e *CacheEngine) CheckAndUpdateRuntimeStatus(value *common.CacheRuntimeValue) (bool, error) {
 	runtimeReady := false
 
 	err := retry.RetryOnConflict(retry.DefaultBackoff, func() error {
 		// Reset readiness on each retry to avoid stale state after conflicts.
-		masterReady, workerReady, clientFullyReady := true, true, true
+		masterReady, workerReady, clientFullyReady := true, true, false
 		runtimeReady = false
 
 		runtime, err := e.getRuntime()

pkg/ddc/cache/engine/status_test.go

@@ -103,7 +103,7 @@ func TestCheckAndUpdateRuntimeStatusClientPartialReadyDoesNotBlockRuntimeReady(t
 	}
 }
 
-func TestCheckAndUpdateRuntimeStatusClientZeroDesiredReplicasStayNotReady(t *testing.T) {
+func TestCheckAndUpdateRuntimeStatusClientZeroDesiredReplicasReportsNotReady(t *testing.T) {
 	engine, client := newStatusTestEngineWithClient(
 		t,
 		fake.NewFakeClientWithScheme(