Ticket #3370 defect closed fixed

Opened 5 years ago

Last modified 5 years ago

twisted.internet.test.reactormixins.ReactorBuilder.runReactor leaks state on glib2 and gtk2

Reported by: exarkun Owned by:
Priority: highest Milestone:
Component: core Keywords:
Cc: Branch: branches/reactorbuilder-cleanup-more-3370
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

It seems glib2 and gtk2 reactors live forever and can interfer with later tests. ReactorBuilder needs to do some extra work to clean these up, perhaps by cancelling all their outstanding DelayedCalls.

Change History

1

Changed 5 years ago by exarkun

  • branch set to branches/reactorbuilder-cleanup-more-3370
  • author set to exarkun

(In [24405]) Branching to 'reactorbuilder-cleanup-more-3370'

2

Changed 5 years ago by exarkun

  • owner exarkun deleted
  • keywords review added
  • priority changed from high to highest

Okay, should be fixed. You can observe the problem by running trial -u twisted.internet.test.test_tls and letting it go for two minutes. It will die right around the two minute mark. It should run indefinitely with the changes in the branch.

3

Changed 5 years ago by washort

  • owner set to exarkun

I get the same results here. Looks good to merge.

4

Changed 5 years ago by washort

  • keywords review removed

5

Changed 5 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [24419]) Merge reactorbuilder-cleanup-more-3370

Author: exarkun Reviewer: washort Fixes: #3370

Add code to cancel all pending timed events to the reactor cleanup code in twisted.internet.test.reactormixins so that they don't remain to interfere with future tests.

6

Changed 2 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.