Yaniv Bronhaim has posted comments on this change.

Change subject: utils: Create AsyncProcessOperation
......................................................................


Patch Set 10: Code-Review+1

(1 comment)

....................................................
File lib/vdsm/utils.py
Line 938:         the process"""
Line 939:         self._proc.kill()
Line 940: 
Line 941:     def result(self):
Line 942:         """Returns the result in the as a tuple of (result, error).
you can still omit the "in the"
Line 943:         If the operation is still running it will block until it 
returns.
Line 944: 
Line 945:         If no resultParser has been set the default result
Line 946:         is (rc, out, err) """


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I79d0eefc9d917a4a93916d52867fb4f1e793c60e
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Ayal Baron <aba...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Eduardo <ewars...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to