do not push cache runtime controller image as it is not ready

Signed-off-by: xliuqq <xlzq1992@gmail.com>

Author: xliuqq

Makefile

@@ -125,7 +125,7 @@ DOCKER_PUSH += docker-push-webhook
 DOCKER_PUSH += docker-push-goosefsruntime-controller
 DOCKER_PUSH += docker-push-juicefsruntime-controller
 DOCKER_PUSH += docker-push-thinruntime-controller
-DOCKER_PUSH += docker-push-cacheruntime-controller
+#DOCKER_PUSH += docker-push-cacheruntime-controller
 DOCKER_PUSH += docker-push-efcruntime-controller
 DOCKER_PUSH += docker-push-vineyardruntime-controller
 # Not need to push init-users image by default
@@ -143,7 +143,7 @@ DOCKER_BUILDX_PUSH += docker-buildx-push-csi
 DOCKER_BUILDX_PUSH += docker-buildx-push-webhook
 DOCKER_BUILDX_PUSH += docker-buildx-push-juicefsruntime-controller
 DOCKER_BUILDX_PUSH += docker-buildx-push-thinruntime-controller
-DOCKER_BUILDX_PUSH += docker-buildx-push-cacheruntime-controller
+#DOCKER_BUILDX_PUSH += docker-buildx-push-cacheruntime-controller
 DOCKER_BUILDX_PUSH += docker-buildx-push-efcruntime-controller
 DOCKER_BUILDX_PUSH += docker-buildx-push-vineyardruntime-controller
 # Not need to push init-users image by default

charts/fluid/fluid/templates/role/cache/rbac.yaml

@@ -143,7 +143,7 @@ rules:
 apiVersion: rbac.authorization.k8s.io/v1
 kind: Role
 metadata:
-  namespace: fluid-system
+  namespace: {{ include "fluid.namespace" . }}
   name: cacheruntime-controller
 rules:
   - apiGroups:
@@ -177,24 +177,24 @@ roleRef:
 subjects:
   - kind: ServiceAccount
     name: cacheruntime-controller
-    namespace: fluid-system
+    namespace: {{ include "fluid.namespace" . }}
 ---
 apiVersion: rbac.authorization.k8s.io/v1
 kind: RoleBinding
 metadata:
   name: cacheruntime-rolebinding
-  namespace: fluid-system
+  namespace: {{ include "fluid.namespace" . }}
 roleRef:
   apiGroup: rbac.authorization.k8s.io
   kind: Role
   name: cacheruntime-controller
 subjects:
   - kind: ServiceAccount
     name: cacheruntime-controller
-    namespace: fluid-system
+    namespace: {{ include "fluid.namespace" . }}
 ---
 apiVersion: v1
 kind: ServiceAccount
 metadata:
   name: cacheruntime-controller
-  namespace: fluid-system
+  namespace: {{ include "fluid.namespace" . }}

docker/Dockerfile.cacheruntime

@@ -23,7 +23,8 @@ RUN wget -O helm-${HELM_VERSION}-linux-${TARGETARCH}.tar.gz https://github.com/f
     tar -xvf helm-${HELM_VERSION}-linux-${TARGETARCH}.tar.gz && \
     mv linux-${TARGETARCH}/helm /usr/local/bin/ddc-helm && \
     chmod u+x /usr/local/bin/ddc-helm && \
-    rm -f ${HELM_VERSION}-linux-${TARGETARCH}.tar.gz
+    rm -f helm-${HELM_VERSION}-linux-${TARGETARCH}.tar.gz && \
+    rm -rf linux-${TARGETARCH}
 
 COPY --from=builder /go/src/github.com/fluid-cloudnative/fluid/charts/ /charts
 

pkg/ddc/cache/engine/dataset.go

@@ -80,11 +80,10 @@ func (e *CacheEngine) UpdateDatasetStatus(phase datav1alpha1.DatasetPhase) (err
 		// TODO(cache runtime): update datasetToUpdate.Status.CacheStates
 
 		if !reflect.DeepEqual(dataset.Status, datasetToUpdate.Status) {
+			e.Log.Info("the dataset status", "status", datasetToUpdate.Status)
 			err = e.Client.Status().Update(context.TODO(), datasetToUpdate)
 			if err != nil {
 				return err
-			} else {
-				e.Log.Info("No need to update the cache of the data")
 			}
 		}
 

pkg/ddc/cache/engine/status.go

@@ -65,7 +65,7 @@ func (e *CacheEngine) CheckAndUpdateRuntimeStatus(value *common.CacheRuntimeValu
 		}
 
 		if value.Client.Enabled {
-			err = e.setClientComponentStatus(&runtimeToUpdate.Status.Worker)
+			err = e.setClientComponentStatus(&runtimeToUpdate.Status.Client)
 			if err != nil {
 				return err
 			}