-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CA-346583: make set-iscsi-initiator idempotent #519
Conversation
scripts/set-iscsi-initiator
Outdated
echo "InitiatorName=$1" > $TMPFILE | ||
echo "InitiatorAlias=$2" >> $TMPFILE | ||
echo "InitiatorName=$INITIATORNAME" > $TMPFILE | ||
echo "InitiatorAlias=$INITIATORALIAS" >> $TMPFILE | ||
|
||
diff $INITIATORFILE $TMPFILE > /dev/null 2>&1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's now doubly idempotent, if i understand lines 63 to 69 correctly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can delete these lines if we are certain the new method works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point. The new bit is required to avoid getting hit with the
there are active sessions so not updating
message if we're not changing anything, which happens when HA is activated
Signed-off-by: Mark Syms <mark.syms@citrix.com>
055be64
to
893acaf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Mark Syms mark.syms@citrix.com