Merge pull request #68 from c3y1huang/fix-ignored-value

longhorn · Aug 2, 2023 · 4ec8eda · 4ec8eda
2 parents f16d600 + dec3341
commit 4ec8eda
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/iscsi/target.go b/iscsi/target.go
@@ -106,8 +106,7 @@ func UpdateLun(tid int, lun int, params map[string]string) error {
 		for k, v := range params {
 			paramStr += fmt.Sprintf("%s=%s,", k, v)
 		}
-		strings.TrimSuffix(paramStr, ",")
-		opts = append(opts, "--params", paramStr)
+		opts = append(opts, "--params", strings.TrimSuffix(paramStr, ","))
 	}
 	_, err := util.Execute(tgtBinary, opts)
 	return err