An example woo_mailer.conf should be checked into the OrangeFactor repo

RESOLVED FIXED

Status

Tree Management
OrangeFactor
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
https://hg.mozilla.org/automation/orangefactor/file/tip/woo_mailer.py uses woo_mailer.conf for various prefs. This file is in the .hgignore (since some of its contents are private), however we don't have an example config checked into the tree.

Once I can find on which box the prod woo_mailer.conf lives, I'll sanitise it and check it in.
(Assignee)

Comment 1

4 years ago
Gotchya:
[root@brasstacks1.dmz.scl3 home]# find . -name 'woo_mailer.conf'
./mcote/orangefactor/src/orangefactor/woo_mailer.conf

Mark, do you know why we run the mailer cron from your user account (and another checkout of the OF repo) rather than webtools?
(Assignee)

Comment 2

4 years ago
Created attachment 799506 [details] [diff] [review]
Example woo_mailer.conf
Attachment #799506 - Flags: review?(mcote)

Comment 3

4 years ago
I think I only did that because I was having problems with the mailer for a while, so I had my own checkout to play with the code, isolated from the main OF.  Still, you're right; it should probably be run by webtools and, since it seems to be working now, from the same code base as the web app.

Comment 4

4 years ago
Comment on attachment 799506 [details] [diff] [review]
Example woo_mailer.conf

Review of attachment 799506 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.  Honestly I don't remember why I even split the OF URLs.  We need the external one for the emails, but I don't know if there's a good reason it can't be used when grabbing the data too.
Attachment #799506 - Flags: review?(mcote) → review+
(Assignee)

Comment 5

4 years ago
https://hg.mozilla.org/automation/orangefactor/rev/09721bf44297
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
You need to log in before you can comment on or make changes to this bug.