> If you promise to test me the script with the STP fixes I'll be willing > to add it. Sadly I don't have time currently to do any decent testing here, > but I'm always open to patches :-) > > Cheers, > > Hannes
Hi Hannes, Unfortunately I don't have any setup which I could test any script with STP fixes. As far as I understood, due to Mike's patch, there is no need to re-discover all nodes at startup. So I suggest to remove the re-discover and to logout from all nodes and not only from the automatic nodes. Please tell me if what is your opinion. Thanks, Doron revert some of the changes from commit 2146208ccd8c6579fa1accbe3dbe7181b46539b3. logout to all nodes when stopping open-iscsi. do not try to re-discover nodes on startup. Signed-off-by: Doron Shoham <[EMAIL PROTECTED]> --- etc/initd/initd.suse | 40 ++-------------------------------------- 1 files changed, 2 insertions(+), 38 deletions(-) diff --git a/etc/initd/initd.suse b/etc/initd/initd.suse index 23bbac0..4bf216c 100644 --- a/etc/initd/initd.suse +++ b/etc/initd/initd.suse @@ -39,8 +39,8 @@ iscsi_login_all_nodes() iscsi_logout_all_nodes() { echo -n "Closing all iSCSI connections: " - # Logout from all sessions marked automatic - if ! $ISCSIADM -m node --logoutall=automatic 2> /dev/null; then + # Logout from all sessions + if ! $ISCSIADM -m node --logoutall=all 2> /dev/null; then if [ $? == 19 ] ; then RETVAL=6 else @@ -101,38 +101,6 @@ iscsi_list_all_nodes() done } -iscsi_discover_all_targets() -{ - # Strip off any existing ID information - RAW_NODE_LIST=`iscsiadm -m node | sed -nre 's/^(\[[0-9a-f]*\] )?(.*)$/\2/p'` - # Obtain IPv4 list - IPV4_NODE_LIST=`echo "$RAW_NODE_LIST" | sed -nre 's/^([0-9]{1,3}(\.[0-9]{1,3}){3}):[^: ]* (.*)$/\1 \3/p'` - # Now obtain IPv6 list - IPV6_NODE_LIST=`echo "$RAW_NODE_LIST" | sed -nre 's/^([0-9a-f]{1,4}(:[0-9a-f]{0,4}){6}:[0-9a-f]{1,4}):[^: ]* (.*)$/\1 \3/p'` - - DISC_TARGETS="" - while read NODE_ADDR NODE_NAME; do - [ -z "$NODE_ADDR" -a -z "$NODE_NAME" ] && continue - NODE_ATTRS=`iscsiadm -m node -p "$NODE_ADDR" -T "$NODE_NAME"` - NODE_STATUS=`echo "$NODE_ATTRS" | sed -nre 's/^.*node\.conn\[0\]\.startup = ([a-z]*).*$/\1/p'` - - if [ "$NODE_STATUS" == 'automatic' ]; then - DISC_TARGETS=`echo "$DISC_TARGETS" | sed -re '/'"$NODE_ADDR"'/!{s/(.*)/\1 '"$NODE_ADDR"'/}'` - fi - done < <(echo "$IPV4_NODE_LIST"; echo "$IPV6_NODE_LIST") - - for TARGET_ADDR in $DISC_TARGETS; do - echo -n "Attempting discovery on target at ${TARGET_ADDR}: " - iscsiadm -m discovery -t st -p "$TARGET_ADDR" > /dev/null 2>&1 - if [ "$?" -ne 0 ]; then - rc_failed 1 - rc_status -v - return 1 - fi - rc_status -v - done -} - case "$1" in start) [ ! -d /var/lib/iscsi ] && mkdir -p /var/lib/iscsi @@ -147,10 +115,6 @@ case "$1" in rc_status -v fi if [ "$RETVAL" == "0" ]; then - iscsi_discover_all_targets - RETVAL=$? - fi - if [ "$RETVAL" == "0" ]; then iscsi_login_all_nodes fi ;; -- 1.5.3.8 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "open-iscsi" group. To post to this group, send email to open-iscsi@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/open-iscsi -~----------~----~----~----~------~----~------~--~---