diff options
author | Hans de Goede <hdegoede@redhat.com> | 2008-11-06 17:36:35 +0100 |
---|---|---|
committer | Hans de Goede <hdegoede@redhat.com> | 2008-11-08 13:23:45 +0100 |
commit | 39f27c560defd737641a17b5052734d12e990e83 (patch) | |
tree | 58755c1ce47c94ecf07a7d8de9479968871b3b88 /kickstart.py | |
parent | 06618ab4c99656facc8795f6943c4fb4315e2edc (diff) | |
download | anaconda-39f27c560defd737641a17b5052734d12e990e83.tar.gz anaconda-39f27c560defd737641a17b5052734d12e990e83.tar.xz anaconda-39f27c560defd737641a17b5052734d12e990e83.zip |
kickstart, iscsi do not call iscsi.startup after startIBFT has been called
The current kickstart IscsiName code calls iscsi.startup after calling
iscsi.startIBFT, but startIBFT will call iscsi.addTarget when an IBFT target
is found, which will in turn then call iscsi.startup. iscsi.startup is not
protected against being called twice, and currently doing so happens to not
have adverse side effects, but we reallt should not be relying on that.
Note: when no IBFT target is found there is no reason to call iscsi.startup
Diffstat (limited to 'kickstart.py')
-rw-r--r-- | kickstart.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/kickstart.py b/kickstart.py index af2e06348..ef9488bdd 100644 --- a/kickstart.py +++ b/kickstart.py @@ -298,7 +298,6 @@ class IscsiName(commands.iscsiname.FC6_IscsiName): self.handler.id.iscsi.initiator = self.iscsiname self.handler.id.iscsi.startIBFT() - self.handler.id.iscsi.startup() # FIXME: flush the drive dict so we figure drives out again isys.flushDriveDict() |