Menu

#1164 Error mail may be accumulated in qfiles

closed-fixed
nobody
None
5
2007-05-14
2006-11-16
No

Mailman/Queue/Runner.py is called by other runners and resposible for dequeueing from one queue and enqueue into another after processing. In _oneloop() function it shunts error messages during processing but leaves them in the original queue directory by noting 'Ignoring ...' message in the error log. These left errorneous messages are never treated again and may be accumulated and slow down the queue processing. These messages should be shunted also.

Discussion

  • Tokio Kikuchi

    Tokio Kikuchi - 2006-11-16

    runner.patch

     
  • Mark Sapiro

    Mark Sapiro - 2006-11-17

    Logged In: YES
    user_id=1123998
    Originator: NO

    If I understand the issue correctly, the problem is that the .bak file is left in the original queue after the parse error. This is not a recoverable error as explained in the code comment, thus I don't think the proper action is to shunt the message. Rather, I think the proper action is to continue to log the event and just remove the .bak file. Not removing the .bak file was an oversight in the original backup implementation.

    My suggested patch is attached.

     
  • Mark Sapiro

    Mark Sapiro - 2006-11-17

    Suggested patch instead of original 203205: runner.patch

     
  • Mark Sapiro

    Mark Sapiro - 2007-05-14
    • status: open --> closed-fixed
     
  • Mark Sapiro

    Mark Sapiro - 2007-05-14

    Logged In: YES
    user_id=1123998
    Originator: NO

    This has now been fixed in SVN (r8204) for Mailman 2.1.10. The fix is different from both of the attached patches. The original queue entry will be moved to the shunt queue with a .psv extension and this fact will be logged.

    The end result is the parse error will be logged and the queue entry will be moved out of the original queue so it won't further impact processing. The queue entry will be preserved for analysis in the shunt queue, but with a .psv filename extension so it won't be processed by bin/unshunt.

     

Log in to post a comment.