Federico Simoncelli has posted comments on this change.

Change subject: BZ#844180: Change scsi scan to asynchronous
......................................................................


Patch Set 9: (1 inline comment)

....................................................
File vdsm/storage/iscsi.py
Line 365:                     "scsi_rescan_maximal_timeout. Set to %s and %s 
seconds ",
Line 366:                     "respectively.", minTimeout, maxTimeout)
Line 367:     log.debug("Performing SCSI scan, this will take up to %s seconds",
Line 368:                 maxTimeout)
Line 369:     time.sleep(minTimeout)
It doesn't make it correct :-)
Line 370:     for i in xrange(maxTimeout - minTimeout):
Line 371:         for p in processes[:]:
Line 372:             if p[1].wait(0):
Line 373:                 if p[1].returncode != 0:


--
To view, visit http://gerrit.ovirt.org/6796
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7665ebaed716f75af4d03ec1b6a9ff9ff5d84853
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Daniel Paikov <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Haim Ateya <[email protected]>
Gerrit-Reviewer: Igor Lvovsky <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to