Pulse warning emails mention exchange "could not be determined"
Categories
(Webtools :: Pulse, enhancement, P5)
Tracking
(Not tracked)
People
(Reporter: rail, Assigned: dustin, Mentored)
Details
(Whiteboard: [lang=python])
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Updated•6 years ago
|
|
||
Comment 1•6 years ago
|
||
I'm in!
|
|
Assignee | |
Comment 2•6 years ago
|
||
Awesome! So, I think the first bit is to figure out where this message is generated, and then work backward.
|
|
||
Comment 3•6 years ago
|
||
Seems like it is in the same location as the PR I did earlier today. In pulseguardian/guardian.py, in the function warning_email, Line 266. Seem strange? Dig deeper?
btw, I am not able to login with github at the link that was put in the earlier PR -> https://pulseguardian.mozilla.org/queues
I get an 500 Internal Server Error. I did get the email about OAuth authorization for Mozilla for my github account prior to my attempt. Was there something else I should have done?
Thanks
|
|
Assignee | |
Comment 4•6 years ago
|
||
That sounds like another bug! Would you file that in bugzilla with some details of how you logged in and what the result is? I was able to reproduce it with a test github account, so you're not alone :)
That doesn't seem terribly strange, since that's what sends the email. The key is to trace backward in the code to see where it gets the value for the exchange name from, and figure out why that's not working. It may help to set up a local run of the application and try to test some of the functions that _exchange_from_queue is calling. You might also set up the testing environment for the application and find the test which exercises this code. There are some instructions in the README for both of those.
As with the last bug you worked on, I'm not terribly familiar with this code so my advice is what I would do in this situation -- I don't actually know the answers!
|
|
||
Comment 5•6 years ago
|
||
Will file the bug.
RE: this bug, I will only be able to get to it next weekend. Sorry. If you need to reassign it, I understand. I will be able to get to it next weekend. Moving across state lines to get to an internship, I know TMI :)
|
|
Assignee | |
Comment 6•6 years ago
|
||
That's OK - thanks for letting me know!
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 7•4 years ago
|
||
|
|
Assignee | |
Comment 8•4 years ago
|
||
Warning: your queue "queue/dustin/testing-deleteme" is overgrowing (4167 ready messages,
4167 total messages).
The queue will be automatically deleted when it exceeds 20000 messages.
Make sure your clients are running correctly and are cleaning up unused
durable queues.
Check messages in the queue at: https://pulseguardian.mozilla.org/queues
Description
•