strace test side-effect hangs selftest
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Fix Released
|
Medium
|
Vincent Ladeuil |
Bug Description
While submitting a merge request to pqm, selftest hanged.
It turned out the submission revealed an hidden bug (probably related to bug #103133).
I was able to reproduce it on gutsy (with the patch proposed at http://
python2.4 bzr selftest bzrlib.
Ommiting any of the three tests doesn't exhibit the hang. IMHO, that's sufficient to demonstrate the correlation between a test (bzrlib.
Researching strace and ptrace man pages (and bug #103133) suggests that we may be trying to use strace in a configuration that was not taken into account in its design.
Several ways to address the initial problem come to mind:
1) Disable strace tests for python2.4 (I wasn't able to reproduce it with python 2.5 but it occured on the pqm machine with 2.4.2 and I can reproduce it with 2.4.4)
2) Fix TestBzrServe.
3) Implement a protection mechanism in selftest as proposed in https:/
I'll bring the subject to the mailing list for discussion.
Related branches
Changed in bzr: | |
assignee: | nobody → vila |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in bzr: | |
status: | Confirmed → Fix Committed |
Changed in bzr: | |
milestone: | none → 1.5 |
status: | Fix Committed → Fix Released |
On Mon, May 5, 2008 at 4:51 PM, vila <email address hidden> wrote:
> Public bug reported:
>
> While submitting a merge request to pqm, selftest hanged.
Isn't English bizarre? <http:// englishplus. com/grammar/ 00000278. htm>
> Researching strace and ptrace man pages (and bug #103133) suggests that
> we may be trying to use strace in a configuration that was not taken
> into account in its design.
I think so too, and I said so last time this came up.
I don't think we even have any current code that makes use of the
strace function. Perhaps the most practical thing is just to disable
or delete it? If we do want to retain it for use in
testing/profiling, I think it would be reasonable to leave it in but
untested, and allow people to fix it if it is broken when they need
it.
-- launchpad. net/~mbp/>
Martin <http://