Skip to content

Commit 122c001

Browse files
authored
Merge pull request #109 from openebs/fix-helm-chart-name
fix: use mayastor sub chart name in openebs chart is used for installation
2 parents 24a8e78 + a6dd743 commit 122c001

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

common/k8sinstall/util.go

+10-3
Original file line numberDiff line numberDiff line change
@@ -222,9 +222,16 @@ func ScaleZfsControllerViaHelm(expected_replica int32) (int32, error) {
222222

223223
// SetRdmaViaHelm enable and disable RDMA
224224
func SetRdmaViaHelm(enableRdma bool, iface string, helmChart, helmRelease, helmVersion string) error {
225-
values := map[string]interface{}{
226-
"io_engine.target.nvmf.rdma.enabled": enableRdma,
227-
"io_engine.target.nvmf.iface": iface,
225+
226+
var values map[string]interface{}
227+
prefix := "io_engine.target.nvmf"
228+
if e2e_config.GetConfig().Product.UseUmbrellaOpenEBSChart {
229+
prefix = fmt.Sprintf("%s.%s", e2e_config.GetConfig().Product.ChartName, prefix)
230+
}
231+
232+
values = map[string]interface{}{
233+
prefix + ".rdma.enabled": enableRdma,
234+
prefix + ".iface": iface,
228235
}
229236

230237
_, err := k8stest.UpgradeHelmChart(helmChart,

0 commit comments

Comments
 (0)