looking at _stopIscsiDaemon() which gets called from iscsi.shutdown():

        result = iutil.execWithCapture(ISCSIADM, ["-k", "0"])
        result.strip()
        if result == "":
            return

        psout = iutil.execWithCapture("/usr/bin/pidof", ["iscsid"])
        if psout.strip() != "":
            log.info("iSCSI shutdown")
            for t in self.targets:
                t.logout()

            for pidstr in psout.split():
                pid = string.atoi(pidstr)
                log.info("killing %s %d" % (ISCSID, pid))

                os.kill(pid, signal.SIGKILL)

            self.iscsidStarted = False

The first line executes "iscsiadm -k 0", which will kill iscsid and will not output anything, so 'result == ""' will be true and the method ends then and there. Also note how this code path fails to set iscsidStarted = False, but we
are getting lucky here, as the only path where this function gets called, we always call iscsi.startup after the shutdown, thus restarting iscsid.

The rest of the code thus never gets reached, resulting in no logouts being done. This actually is a good thing, there is no reason to logout / login on iscsi shutdown / startup. However stopping iscsid when we are logged in to iscsi nodes is a bad idea, so shutdown should not call _stopIscsiDaemon(), see bug 470223.