Noisy failmail should end
Description
Details
Subject | Repo | Branch | Lines +/- | |
---|---|---|---|---|
silverpop connector update | wikimedia/fundraising/crm/vendor | master | +196 -34 | |
Add updated silverpop connector | wikimedia/fundraising/crm | master | +25 -10 |
Event Timeline
I'm trying to figure out which failmails we're trying to mute. I just looked at the failmail for Get group member data from Silverpop and the last few runs all errored out with a DB Lock. @Eileenmcnaughton do you happen to know what's going on with that?
I also just looked at Get recipent data and the most recent failure was "Login failed", with no additional data.
From the 20th, the Omnimail forget me job delete in Silverpop failed but the error message was Could not delete OmnimailJobProgress id.
I've claimed this - but I'm specifically focussing on catching the sessionId timeouts
I think this will work but the challenge is testing it
https://github.com/mrmarkfrench/silverpop-php-connector/pull/34
Currently it looks like there are 198516 rml contacts to be imported
https://engage4.silverpop.com/lists.do?action=listSummary&listId=18468760
I turned off the queues & did 2 runs - which should mean around 20k were imported which is about 10% of the outstanding. One run takes about 15 mins
I also pushed out Ejegg's 'turn it on for a bit of the day'
We won't see the number at the url change until
- silverpop export runs - at that point the contacts get ids and are no longer in the group and
- the recalculate is hit on the button on the page that url takes us to
But I would expect it to be down to 160k after the above happens
OK I have caught an error & updated the PR / test
https://github.com/mrmarkfrench/silverpop-php-connector/pull/34
But am having some trouble with tests locally - not sure if someone else can run it
Change 646895 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm@master] Add updated silverpop connector
Change 646901 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm/vendor@master] silverpop connector update
Change 646895 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Add updated silverpop connector
Change 646901 merged by jenkins-bot:
[wikimedia/fundraising/crm/vendor@master] silverpop connector update
The original bug is fixed - we did get a sniff of a new bug but it has gone away & I'm a bit stuck tracking it down now