Users not notified in email when subscribed by someone else

Bug #494257 reported by Brian Murray
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Deryck Hodge

Bug Description

I set the contact address for the Ubuntu Bug Control team to the team's
mailing list - <email address hidden> . I then
subscribed the team to bug 481788 however, there has been no mail to the
mailing list about being subscribed to the bug report. Additionally,
there are no messages pending moderation.

Barry says:

14:57 <barry> i'll try to recreate this situation, but it should work
15:02 <barry> what kind of mail was this, a comment on a bug? a change
in status/importance/assignee?
15:13 <bdmurray> subscription to the bug report
15
:13 <bdmurray> so new notification of it
15:29 <barry> i can't any bugmail to work locally and i'm not sure why

 affects launchpad
 assignee sinzui

--
Brian Murray @ubuntu.com

Related branches

tags: added: ubuntu-qa
Revision history for this message
Barry Warsaw (barry) wrote :

Curtis: See lib/canonical/launchpad/mailman/monkeypatches/lphandler.py for the key bit and why I think this should work. Also, I definitely got it working for answers before I ran out of time. There are unit tests for answers (I'm sure) but not positive about bugmail.

Revision history for this message
Curtis Hovey (sinzui) wrote :

Sending to a team via the contact this team features works as expected for teams that have set their contact address to the list. I think this error is in launchpad-bugs.

affects: launchpad → malone
Changed in malone:
assignee: Curtis Hovey (sinzui) → nobody
Revision history for this message
Curtis Hovey (sinzui) wrote :

I subscribed ~haibunku to a real bug then updated the bug comment. I and the list both got the email. I cannot reproduce this problem. I did not test the case where the bug as zillions of subscribers with duplicates.

Revision history for this message
Curtis Hovey (sinzui) wrote :

barry, out of curiosity, did you confirm that this is broken in dev by changing a bug then running ./cronscripts/send-bug-notifications.py to process the queue? Answers does not use a queue; it sends directly.

Revision history for this message
Barry Warsaw (barry) wrote :

I did not. That makes perfect sense. I wonder, should 'make run_all' occasionally run the cronscripts?

Revision history for this message
Barry Warsaw (barry) wrote :

Oh, I should also mention that I tested this in staging and never saw the bugmails show up in the staging or staging-mailman inbox. We should double check that staging is running the cronscripts.

Revision history for this message
Martin Pitt (pitti) wrote :

I actually notice that ocasionally as well. Sometimes people subscribe me or ~ubuntu-sru to a bug, and I never get a mail, but often it also works. One recent particular example is bug 493788 where Chris Coulson subscribed me, and I never got a bug mail

Changed in malone:
status: New → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

This always seems to work for bug followups which happen after the subscription, just not if someone only subscribes me without changing/adding anything new.

Revision history for this message
Brian Murray (brian-murray) wrote :

I modified the tags on bug 481788, which I was testing with, and an e-mail did appear on the ubuntu bug control mailing list. However, I wanted to use this to notify the team by just subscribing them to a bug report - not subscribing them and modifying the bug.

Revision history for this message
Deryck Hodge (deryck) wrote :

So I I think this is a problem in our "You have been subscribed to a public bug by" message sending. I don't think this has anything to do with the mailing list. I know I didn't get a "you have been subscribed" message yesterday when I had Karl Fogel subscribe me to a bug. A search of my mail archives shows a July email as the latest email of this kind to me. I'm fairly certain I've been subscribed by people to bugs since then.

Changed in malone:
status: Confirmed → Triaged
importance: Undecided → High
tags: added: email
summary: - bug mail doesn't make it to a Launchpad mailing list
+ Users not notified in email when subscribed by someone else
Revision history for this message
Brian Murray (brian-murray) wrote :

Is there a time frame for fixing this at all?

Revision history for this message
Deryck Hodge (deryck) wrote :

Not yet, no. I'll look closer tomorrow at when this can be scheduled.

Changed in malone:
assignee: nobody → Tom Berger (intellectronica)
status: Triaged → In Progress
Deryck Hodge (deryck)
Changed in malone:
assignee: Tom Berger (intellectronica) → nobody
assignee: nobody → Deryck Hodge (deryck)
Deryck Hodge (deryck)
Changed in malone:
status: In Progress → Fix Committed
milestone: none → 10.04
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
tags: added: qa-needstesting
Revision history for this message
Deryck Hodge (deryck) wrote :

This still doesn't seem to be working, so I'm marking it qa-bad. I need to do a bit more poking later about what exactly is going wrong.

tags: added: qa-bad
removed: qa-needstesting
Deryck Hodge (deryck)
Changed in malone:
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote :

It's not exactly clear to me why this moved to Fix Released when the bug has the tag qa-bad. Additionally, I had someone else subscribe me to bug 583931 and I have received no email message indicating that I was subscribed.

Changed in malone:
status: Fix Released → Triaged
Revision history for this message
Deryck Hodge (deryck) wrote :

The branch attached to this bug is finished, qa-ok, and released. The part that was qa-bad was fixed in a related branch that was a security fix, so I didn't comment initially when that the work was done. I was just marking a group of bugs fix released and not paying much attention to the bugs.

The initial problem was that the notify call that generates the activity/email was done from browser code not model code, and the API calls are based on the model. So the js subscribe-someone-else widget wasn't hitting the notify call. My branch for this bug did the work to move all this to model code. That is done, tested, and deployed. There is still bug 571821 to do, which is an easy fix to call Bug.subscribe with the new parameter. I just haven't got to it yet, but I'll do it this week and notifications should appear again. Once 571821 is fixed, this bug can be closed.

Changed in malone:
status: Triaged → Fix Committed
tags: added: qa-ok
removed: qa-bad
Deryck Hodge (deryck)
Changed in malone:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.