import self-maintainerd ddc-helm in dockerfile#5163
import self-maintainerd ddc-helm in dockerfile#5163Syspretor wants to merge 1 commit intofluid-cloudnative:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Signed-off-by: 玖宇 <guotongyu.gty@alibaba-inc.com>
Syspretor
force-pushed
the
feature/import-self-maintained-helm-repo
branch
from
9ed3d2f to
798285f
Compare
|
| cp bin/alluxioruntime-controller /go/bin/alluxioruntime-controller | ||
|
|
||
| WORKDIR /go/src/github.com/fluid-cloudnative/ | ||
| RUN git clone https://github.com/fluid-cloudnative/helm.git |
There was a problem hiding this comment.
Maybe use git clone -b release-3.17-for-fluid instead?
There was a problem hiding this comment.
helm in branch release-3.17-for-fluid is using go 1.24.0 in the go.mod file. Is that ok we build helm in a golang 1.23.10 environment?
2 participants
In Fluid, there is only a unique version of runtime when deploying through Helm. Therefore, this PR import self-maintainerd ddc-helm and optimize the logic for confirming the version by listing release configmaps during helm install/uninstall to use the get method to retrieve version 1. This reduces the pressure of list operations on the apiserver.