darcs

Issue 636 Replying to bugs@darcs.net generates a bounce / new bug reports arrive twice

Title Replying to bugs@darcs.net generates a bounce / new bug reports arrive twice
Priority urgent Status resolved
Milestone Resolved in
Superseder Nosy List darcs-devel, dmitry.kurochkin, kowey, markstos, thorkilnaur, tommy
Assigned To simon
Topics BugTracker

Created on 2008-02-01.17:47:17 by kowey, last changed 2009-08-27.14:03:22 by admin.

Messages
msg3035 (view) Author: kowey Date: 2008-02-01.17:47:15
...strangely, the message makes it through and is noted by the bug tracker.
msg3036 (view) Author: kowey Date: 2008-02-01.17:48:11
Oh, and David thinks it's happening to everybody.  Any thoughts, Mark?
msg3037 (view) Author: markstos Date: 2008-02-01.19:07:42
Eric,

I'm really not sure. We could try tracing the flow of e-mail, and also reviewing
the ChangeLog of newer Round-up releases to see if there's something that looks
like a related bug fix. 

I'm not much of an expert with Round-Up, Python or the e-mail system in use,
unfortunately. 

  Mark
msg3040 (view) Author: droundy Date: 2008-02-01.19:48:42
This message is an attempt to debug the bouncing trouble.
-- 
David Roundy
Department of Physics
Oregon State University
msg3404 (view) Author: markstos Date: 2008-02-14.04:44:05
Tonight I reviewed the mail flow for bugs@darcs.net. Here's some details from that: 

- The mail is managed by exim4. 
- It appears that the bugs address is managed throug here:
  /etc/exim4/conf.d/router/459_bugs

Which has these contents:
darcs_bugs:
  driver = redirect
  local_parts = darcs-bugs
  senders = bugs@darcs.net : bugs-bounces@darcs.net
  user = www-data
  data = "|/usr/bin/procmail /etc/darcs_bugs_procmailrc"
  pipe_transport = address_pipe
#####

The procmailrc file it refers to looks like this:
###
:0 chfw
SUBJECT=| formail -xSubject: | perl -pe 's/\[darcs-[^\]]+\]//'

:0 hfw
| /usr/bin/formail -i "Subject: $SUBJECT"

:0
|/usr/bin/roundup-mailgw /var/lib/roundup/trackers/darcs

####

I found it notable that Roundup includes documentation for integrating with Exim
( search for Exim on this page:   
  http://roundup.sourceforge.net/doc-1.0/installation.html ).

However, the examples they cite there have noticeable differences from the
system we are using. 

So, without fully understanding how Roundup or Exim work, I advise evaluating
configuration changes that would bring us more inline with the official
Roundup/Exim example. 

I hope that might solve the dupe bug reports that get happen frequently now.
msg3422 (view) Author: droundy Date: 2008-02-14.18:51:48
On Thu, Feb 14, 2008 at 04:44:06AM -0000, Mark Stosberg wrote:
> However, the examples they cite there have noticeable differences from the
> system we are using. 
> 
> So, without fully understanding how Roundup or Exim work, I advise evaluating
> configuration changes that would bring us more inline with the official
> Roundup/Exim example. 

We've got a major difference, which is that we filter all our bugs email
through a mailman mailing list in order to achieve moderation with a
reasonable user-interface for the moderator (me) that includes
whitelisting.  Perhaps this is no longer necessary, as it seems that the
bugs@darcs.net only gets one spam/day or less (I've no idea why,
darcs-devel and darcs-users both get far more spam).

It also might work this way because I've been hesitant to change a working
configuration.  You're welcome to make (cautious) changes to the exim
setup, or to make more specific suggestions.

> I hope that might solve the dupe bug reports that get happen frequently now.

That would be nice.
-- 
David Roundy
Department of Physics
Oregon State University
msg3424 (view) Author: markstos Date: 2008-02-14.18:55:30
My personal hope is that with my initial research, someone else will see the bug. :)

I'll study it some more, but will be hesitant to make changes until I under
understand what's going. If I do change anything, I'll have good backups and
documentation. 

   Mark
msg3516 (view) Author: markstos Date: 2008-02-16.23:05:06
I can look at this a little more.
msg4938 (view) Author: kowey Date: 2008-06-04.09:50:12
If this provides any sort of clue, darcs-devel seems to get second bug report,
not the first one.
msg4967 (view) Author: droundy Date: 2008-06-05.11:41:58
Eric, that clue may have given me the answer:  it looks like bugs is subscribed
to darcs-devel and may have been getting an extra copy sent by darcs-devel. 
I've left bugs subscribed to darcs-devel (so that bug email messages will be
delivered) but set it to not receive any darcs-devel traffic.  Maybe this will
fix it?

David
msg4968 (view) Author: kowey Date: 2008-06-05.11:54:42
Well, here is my reply via email...

2008/6/5 David Roundy <bugs@darcs.net>:
> Eric, that clue may have given me the answer:  it looks like bugs is subscribed
> to darcs-devel and may have been getting an extra copy sent by darcs-devel.
> I've left bugs subscribed to darcs-devel (so that bug email messages will be
> delivered) but set it to not receive any darcs-devel traffic.  Maybe this will
> fix it?
msg4972 (view) Author: kowey Date: 2008-06-05.11:58:22
Hmm... still get a failed tracker submission if I reply via email.
And as per issue908 and issue909, we're still getting duplicates.
Oh well...
msg5031 (view) Author: kowey Date: 2008-06-13.08:54:43
Whoops! Benjamin just provided a counter-example to my
darcs-devel-gets-second-bug observation (issue918 and issue919).  Hmm, I wonder
if this is related to bugs@darcs.net no longer receiving its messages, or if my
observation was just wrong.
msg5033 (view) Author: kowey Date: 2008-06-13.09:37:35
Another clue is that the bugtracker seems to have two entries for every user. 
Look up marlowsd, for example
msg5365 (view) Author: simon Date: 2008-08-10.23:29:32
Can I take a look at the exim/mailman/roundup setup ? I have experience with
debugging twisty email & python setups and might be able to help.
msg5465 (view) Author: simon Date: 2008-08-13.02:40:40
For the record: I did get this access and am looking at the setup on  
and off. This reply will test mail-in and probably show me a bounce.
msg5495 (view) Author: simon Date: 2008-08-13.23:49:44
There are a few problems discussed on this page. Focussing on the "bounce when
mailing in issue", it seems to sometimes just bounce without updating the
tracker. Also I seem to be blocked on this until I can get a bit more access to
exim logs and to edit roundup config and code, to increase logging. Until then I
can't trace the flow, but I will study the code a bit more in case something
jumps out.
msg5503 (view) Author: kowey Date: 2008-08-14.11:54:29
Mark! Help! :-)

Can you give Simon the needed access to the exim logs and roundup config (or
maybe dig up whatever information he needs?) 

[now new submissions seem to get 2 bounces and no entries on the tracker]
msg5509 (view) Author: simon Date: 2008-08-14.16:52:19
David took care of this, so I can see more. I'll return to this asap.
msg5512 (view) Author: simon Date: 2008-08-14.19:21:10
test 1
msg5515 (view) Author: simon Date: 2008-08-14.19:49:13
test 2, expecting no bounce.
msg5518 (view) Author: simon Date: 2008-08-14.20:11:30
test 3, let's see
msg5519 (view) Author: simon Date: 2008-08-14.20:14:17
Jason: much better. :) Thanks. With luck this will fix the duplicate  
new issues as well.
msg5521 (view) Author: simon Date: 2008-08-14.20:52:18
Duplicated new issues is also fixed. Re other thing mentioned on this page,
duplicate users, let's open a new issue if needed. I see that some users have
duplicates, but it doesn't seem to be hurting us yet. Closing this issue.
msg5524 (view) Author: kowey Date: 2008-08-14.21:10:54
On Thu, Aug 14, 2008 at 20:52:20 -0000, Simon Michael wrote:
> Duplicated new issues is also fixed. Re other thing mentioned on this page,
> duplicate users, let's open a new issue if needed. I see that some users have
> duplicates, but it doesn't seem to be hurting us yet. Closing this issue.

Just one more (minor) thing Simon...

If we're adding darcs-devel to the nosy list, I think this means we need
to stop having a special rule to forward new entries to darcs-devel as
well.

Not nearly as serious as the issue you fixed, however.
msg5571 (view) Author: kowey Date: 2008-08-18.09:11:02
Marked resolved again.

Also, as per issue999, I am censoring the messages below and removing them from
the bugtracker

=== msg 5517 censored ===
On Thu, Aug 14, 2008 at 12:43 PM, Simon Michael <bugs[REDACTED]> wrote:

>
> Simon Michael <simon[REDACTED]> added the comment:
>
> I have attached the exim and roundup logs from handling a reply to this
> issue.
> You can see that roundup is receiving the mail twice, the first is
> processed
> normally and the second fails "due to concurrent update". If I am reading
> the
> exim log rightly this machine is subscribed to darcs devel as
> darcs-bugs[REDACTED] and as darcs-bugs[REDACTED]. I think we need
> a
> list admin to remove the darcs-bugs[REDACTED] subscriber from the
> list(s).

I found darcs-bugs@ subscribed to the bugs list and removed it from there.
That's the only place I could find it though.

=== msg5514 censored ===

I have attached the exim and roundup logs from handling a reply to this issue.
You can see that roundup is receiving the mail twice, the first is processed
normally and the second fails "due to concurrent update". If I am reading the
exim log rightly this machine is subscribed to darcs devel as
darcs-bugs[REDACTED (darcs)] and as darcs-bugs[REDACTED (osu)]. I think we need a
list admin to remove the darcs-bugs[REDACTED (osu)] subscriber from the list(s).
History
Date User Action Args
2008-02-01 17:47:17koweycreate
2008-02-01 17:48:12koweysetstatus: unread -> unknown
nosy: + markstos
messages: + msg3036
2008-02-01 19:07:45markstossetnosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3037
2008-02-01 19:48:44droundysetnosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3040
2008-02-14 04:44:06markstossetnosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3404
2008-02-14 18:51:49droundysetnosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3422
2008-02-14 18:55:32markstossetnosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3424
title: Replying to bugs@darcs.net generates a bounce -> Replying to bugs@darcs.net generates a bounce / new bug reports arrive twice
2008-02-16 23:05:07markstossetstatus: unknown -> has-patch
nosy: droundy, tommy, beschmi, kowey, markstos
messages: + msg3516
assignedto: markstos
2008-05-20 15:11:02koweylinkissue868 superseder
2008-06-04 09:50:14koweysetnosy: + dagit
messages: + msg4938
2008-06-05 07:44:09koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
assignedto: markstos -> (no value)
2008-06-05 11:41:59droundysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
messages: + msg4967
2008-06-05 11:54:44koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
messages: + msg4968
2008-06-05 11:58:24koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
messages: + msg4972
2008-06-13 08:54:44koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
messages: + msg5031
2008-06-13 09:37:37koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit
messages: + msg5033
2008-08-10 21:52:34koweysetstatus: has-patch -> unknown
nosy: droundy, tommy, beschmi, kowey, markstos, dagit
2008-08-10 23:29:36simonsetnosy: + simon
messages: + msg5365
2008-08-11 13:21:25koweysetpriority: bug -> urgent
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
2008-08-13 02:40:44simonsetnosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: + msg5465
2008-08-13 23:49:47simonsetnosy: + darcs-devel
messages: + msg5495
2008-08-14 11:54:31koweysetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5503
assignedto: markstos
2008-08-14 16:52:21simonsetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5509
assignedto: markstos -> simon
2008-08-14 19:21:13simonsetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5512
2008-08-14 19:43:05simonsetfiles: + mailin-error.log
nosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5514
2008-08-14 19:49:15simonsetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5515
2008-08-14 20:10:18dagitsetfiles: + unnamed
nosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5517
2008-08-14 20:11:32simonsetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5518
2008-08-14 20:14:19simonsetnosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5519
2008-08-14 20:52:20simonsetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5521
2008-08-14 20:53:02simonsetfiles: - unnamed
nosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
2008-08-14 21:10:55koweysetstatus: resolved -> unknown
nosy: droundy, tommy, beschmi, kowey, markstos, darcs-devel, dagit, simon
messages: + msg5524
2008-08-15 15:21:08droundysetnosy: - darcs-devel
2008-08-18 09:04:06koweysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
2008-08-18 09:10:02koweysetstatus: resolved -> unknown
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: + msg5570
2008-08-18 09:11:04koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: + msg5571
2008-08-18 09:11:15koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: - msg5570
2008-08-18 09:11:28koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: - msg5517
2008-08-18 09:11:46koweysetnosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
messages: - msg5514
2008-08-18 09:13:14koweysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
2008-08-30 18:53:06simonsetfiles: - mailin-error.log
nosy: droundy, tommy, beschmi, kowey, markstos, dagit, simon
2009-08-06 17:53:02adminsetnosy: + jast, Serware, dmitry.kurochkin, darcs-devel, zooko, mornfall, thorkilnaur, - droundy
2009-08-06 20:56:14adminsetnosy: - beschmi
2009-08-10 22:12:52adminsetnosy: - darcs-devel, zooko, jast, Serware, mornfall
2009-08-11 00:05:51adminsetnosy: - dagit
2009-08-25 17:23:55adminsetnosy: + darcs-devel, - simon
2009-08-27 14:03:22adminsetnosy: tommy, kowey, markstos, darcs-devel, thorkilnaur, dmitry.kurochkin