Wenchao Xia has uploaded a new change for review.
Change subject: walk around threading and pipe problem ......................................................................
walk around threading and pipe problem
Calling subprocess.Popen and threading.Thread at the same time have race condition in Python 2.7.2, this patch added a lock to avoid this situation. If next release of Python fixed it, this could be removed
Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Signed-off-by: wenchao xia xiawenc@linux.vnet.ibm.com --- M vdsm/betterThreading.py M vdsm/storage/misc.py 2 files changed, 24 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/38/3638/1 -- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: walk around threading and pipe problem ......................................................................
Patch Set 1:
Related bugzillar: https://bugzilla.redhat.com/show_bug.cgi?id=808853 What vdsm shows is that if you delete /rhev/data-center/ and reboot vdsm, /rhev/data-center/hsm-task is not always created, which happens because the thread try creating it hangs.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Ayal Baron has posted comments on this change.
Change subject: walk around threading and pipe problem ......................................................................
Patch Set 1: I would prefer that you didn't submit this
(6 inline comments)
][
.................................................... File vdsm/betterThreading.py Line 115: self.lockPipe = storage.misc.ThreadPipeLock I believe threadPipeLock should be defined here and imported in storage.misc, not the other way around.
.................................................... File vdsm/storage/misc.py Line 79: ThreadPipeLock = threading.Lock() s/Thread/thread/
Line 220: with disabledGcBlock: why not: with ThreadPipeLock, disabledGcBlock:
Line 226: trailing spaces are frowned upon.
Line 231: ThreadPipeLock.release() redundant if used with "with" as above
Line 248: ThreadPipeLock.release() redundant if used with "with" as above
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: walk around threading and pipe problem ......................................................................
Patch Set 1: I would prefer that you didn't submit this
(3 inline comments)
this is scary... I am a bit reluctant to take such an intrusive patch in order to work around a supposed serious bug in Python.
Could you find a reproducer for the bug outside of Vdsm? I must require that there is a Python bug# in this commit message - otherwise we would never know when to remove this code.
.................................................... Commit Message Line 7: walk around threading and pipe problem as a non-English-speaker to another: the term is "workaround".
Line 9: Calling subprocess.Popen and threading.Thread at the same time have race condition in Python 2.7.2, this patch added shorter lines, please... they break badly on my terminal.
.................................................... File vdsm/betterThreading.py Line 108: import storage.misc betterThreading must not depend on the rest of vdsm.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: walk around threading and pipe problem ......................................................................
Patch Set 1:
added an attachment in bug808853 https://bugzilla.redhat.com/show_bug.cgi?id=808853
Thread_Pipe_Test.py, a test python file to reproduce it without vdsm. Running it about 10 times should result in thread 1 hanging.
I am trying to build latest python release, so can't fire a bug in python issue tracker now. But there are some discuss about subprocess issue there: http://bugs.python.org/issue2320,
It is similar to what I observed, hangs on a thread calling subprocess pipe, conclusion is it is fixed in python 3.2, but not python 2.7. My test case is that one thread hanging at subprocess pipe call while another thread call start(). So more validation is needed on python 3.2 to see if it covers the situation.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: walk around threading and pipe problem ......................................................................
Patch Set 1:
Wenchao Xia, please add this information to the commit message - and open a Python bug for Fedora. Maybe we can get the 3.2 solution (haven't give it a glance yet, sorry) backported to Fedora.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 2: I would prefer that you didn't submit this
Wenchao, you are working around a Python bug. The commit message should state which bug it is. Please open a Python bug of Fedora's BZ if needed.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 2: (6 inline comments)
note my former comment about the python bug.
.................................................... File vdsm/betterThreading.py Line 108: ThreadPipeLock = threading.Lock() instances begin with lower case letter,
Classes with upper case.
Line 115: self.lockPipe = ThreadPipeLock private data members should begin with _
Line 118: self.lockPipe.acquire() use "with"
Line 122: threading.Thread = vdsmThread; please monkey-patch only pythons with this bug?
.................................................... File vdsm/storage/misc.py Line 190: ThreadPipeLock = betterThreading.ThreadPipeLock that's done nicer with
from betterThreading import threadPipeLock
Line 220: ThreadPipeLock.acquire() better use "with". I hate to deadlock due to a silly p.stdout.write exception.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 2:
Sure, fired a new bug in Fedora bugzilla. Also the patch code was changed to automatically adjust according to python version, for that test passed on Python v2.7.3 which was released a few days ago.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 3:
Please ignore patch set 3, some codes need to be adjusted.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 3: I would prefer that you didn't submit this
Given https://bugzilla.redhat.com/show_bug.cgi?id=814492#c1 saying that python 2.7.3 is available for Fedora, I'd prefer to replace this patch by a simple
Requires: python > 2.7.3
in the Fedora section of our spec file.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 3:
adjusted the codes according to comments.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Sheldon Feng has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: Verified; Looks good to me, but someone else must approve
(1 inline comment)
tested. it can work.
.................................................... File vdsm.spec.in Line 36: Requires: python >= 2.7.3 this bug has been fixed on python 2.7.3. what about 2.7.2 and before version. Will it be backported on 2.7.2 and before?
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Sheldon Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: (1 inline comment)
.................................................... File vdsm.spec.in Line 36: Requires: python >= 2.7.3 was the removal of "which" intentional? it may well be valid, but unrelated to this python patch.
more importantly - please make sure el6 already has this version of python, or have only
%if !0%{?rhel} Requires: python >= 2.7.3 %else Requires: python %end
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Sheldon Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: I would prefer that you didn't submit this
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Sheldon Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: (1 inline comment)
comments reply in lines.
.................................................... File vdsm.spec.in Line 36: Requires: python >= 2.7.3 I thought "Requires: python which" means no requirement for python version, so I made a request for it. "%else Requires: python" make issue still happen, is that acceptable?
fired a bug to confirm whether el6 have the package: https://bugzilla.redhat.com/show_bug.cgi?id=815176
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: (1 inline comment)
.................................................... File vdsm.spec.in Line 36: Requires: python >= 2.7.3 el6 will NOT have python 2.7.3. rhel6 is a stable distribution that does not take new upstream versions on a minor version change.
the request for python version should be specific to Fedora.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5: (1 inline comment)
.................................................... File vdsm.spec.in Line 36: Requires: python >= 2.7.3 OK. Then for el6 system, so far the solution I can think about are: 1 add a lock on it, similar to patch set 5. 2 or force user to update python himself in .spec file. Could u give some suggestion about it?
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5:
Since el6 have no python2.7.3 available, how should I deal with it? My opinion is adding a lock or null lock depends on the python version at runtime.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5:
Wenchao Xia, have you reproduced the bug in el6's Python 2.6.? If so, it should be fixed there, too. Not only in Fedora. If this is the case, please clone rhbz#814492 to RHEL6!
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Wenchao Xia has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 5:
Tested on el6.2 with python 2.6.6, unable to produce it, my mistake. Adjusted the spec file and rebased.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 7: Looks good to me, but someone else must approve
Please send an email to vdsm-devel about this change - it is going to disturb a lot of people, so I would like to inform them in advance.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
ShaoHe Feng has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 7: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Ayal Baron has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 7: I would prefer that you didn't submit this
(1 inline comment)
.................................................... File vdsm.spec.in Line 47: Requires: python which if which is not needed then why don't we get rid of it entirely? if it is needed then it is needed always and shouldn't be part of the if.
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 8: Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Lei Li has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 8: Verified; Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Saggi Mizrahi has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 8: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: workaround pipe and thread issue in python ......................................................................
Patch Set 8:
Ok, I think we've waited enough for objections. Thanks!
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has submitted this change and it was merged.
Change subject: workaround pipe and thread issue in python ......................................................................
workaround pipe and thread issue in python
Calling subprocess.Popen and threading.Thread at the same time is not safe in Python v2.7.2. Running test case for several release, results are: Fedora16: v3.2.2 and v2.7.3 pass, v2.7.2 fail. Redhat6.2: v2.6.6 pass. This patch modify the spec file to avoid the fails. bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=814492 cpython issue tracker solving the problem: http://bugs.python.org/issue13156
Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Signed-off-by: wenchao xia xiawenc@linux.vnet.ibm.com --- M vdsm.spec.in 1 file changed, 4 insertions(+), 1 deletion(-)
Approvals: Lei Li: Verified; Looks good to me, but someone else must approve Saggi Mizrahi: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/3638 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: I72e2eac7c15fd147b953eebf701257469546d62b Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Wenchao Xia xiawenc@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Mark Wu wudxw@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: ShaoHe Feng shaohef@linux.vnet.ibm.com Gerrit-Reviewer: Shu Ming sming56@gmail.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
vdsm-patches@lists.fedorahosted.org