|
 |
1de2d2 |
From 032129d0eb47d08eff7bd8b4c82a835a8929e9d3 Mon Sep 17 00:00:00 2001
|
|
|
1de2d2 |
From: Martin Wilck <mwilck@suse.de>
|
|
|
1de2d2 |
Date: Thu, 4 Apr 2019 15:29:04 +0200
|
|
|
1de2d2 |
Subject: [PATCH] iscsiroot: parse_iscsi_root overwrites command line args
|
|
|
1de2d2 |
|
|
|
1de2d2 |
iscsi_target_name, iscsi_target_ip, iscsi_target_port are
|
|
|
1de2d2 |
unconditionally overwritten by parse_iscsi_root. Don't set
|
|
|
1de2d2 |
them here, for code clarity.
|
|
|
1de2d2 |
---
|
|
|
1de2d2 |
modules.d/95iscsi/iscsiroot.sh | 8 ++------
|
|
|
1de2d2 |
1 file changed, 2 insertions(+), 6 deletions(-)
|
|
|
1de2d2 |
|
|
|
1de2d2 |
diff --git a/modules.d/95iscsi/iscsiroot.sh b/modules.d/95iscsi/iscsiroot.sh
|
|
|
1de2d2 |
index f3f88259..45db89c1 100755
|
|
|
1de2d2 |
--- a/modules.d/95iscsi/iscsiroot.sh
|
|
|
1de2d2 |
+++ b/modules.d/95iscsi/iscsiroot.sh
|
|
|
1de2d2 |
@@ -92,12 +92,6 @@ handle_netroot()
|
|
|
1de2d2 |
# override conf settings by command line options
|
|
|
1de2d2 |
arg=$(getarg rd.iscsi.initiator -d iscsi_initiator=)
|
|
|
1de2d2 |
[ -n "$arg" ] && iscsi_initiator=$arg
|
|
|
1de2d2 |
- arg=$(getargs rd.iscsi.target.name -d iscsi_target_name=)
|
|
|
1de2d2 |
- [ -n "$arg" ] && iscsi_target_name=$arg
|
|
|
1de2d2 |
- arg=$(getarg rd.iscsi.target.ip -d iscsi_target_ip)
|
|
|
1de2d2 |
- [ -n "$arg" ] && iscsi_target_ip=$arg
|
|
|
1de2d2 |
- arg=$(getarg rd.iscsi.target.port -d iscsi_target_port=)
|
|
|
1de2d2 |
- [ -n "$arg" ] && iscsi_target_port=$arg
|
|
|
1de2d2 |
arg=$(getarg rd.iscsi.target.group -d iscsi_target_group=)
|
|
|
1de2d2 |
[ -n "$arg" ] && iscsi_target_group=$arg
|
|
|
1de2d2 |
arg=$(getarg rd.iscsi.username -d iscsi_username=)
|
|
|
1de2d2 |
@@ -112,6 +106,8 @@ handle_netroot()
|
|
|
1de2d2 |
iscsi_param="$iscsi_param $p"
|
|
|
1de2d2 |
done
|
|
|
1de2d2 |
|
|
|
1de2d2 |
+ # this sets iscsi_target_name and possibly overwrites most
|
|
|
1de2d2 |
+ # parameters read from the command line above
|
|
|
1de2d2 |
parse_iscsi_root "$1" || return 1
|
|
|
1de2d2 |
|
|
|
1de2d2 |
# Bail out early, if there is no route to the destination
|
|
|
1de2d2 |
|