Write a new Drush job to create and run a GlobalCollectOrphanRectifier.
For early review:
https://gerrit.wikimedia.org/r/#/c/300708/
Deploy and enable job using Jenkins.
Write a new Drush job to create and run a GlobalCollectOrphanRectifier.
For early review:
https://gerrit.wikimedia.org/r/#/c/300708/
Deploy and enable job using Jenkins.
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | None | T117466 Q3 GOALS! (January-March) Keep at top of Q3 column | |||
Resolved | None | T108229 [Epic] SPOF: Replace ActiveMQ donation queues with a more robust software stack | |||
Resolved | None | T133108 [Epic] Rewrite all queue clients to use a single shim library, improve library | |||
Resolved | awight | T133754 Upstream whatever we can to PHP-Queue | |||
Resolved | awight | T133190 Remove KeyValueStore from PHP-Queue | |||
Open | None | T134191 Write CiviCRM extension to view pending db records | |||
Resolved | None | T130897 [Epic] Consolidate "pending" queue usages | |||
Resolved | awight | T131275 [Epic] Move orphan rectifier out of payments | |||
Resolved | awight | T141487 Run the orphan rectifier job from CRM Jenkins | |||
Open | Spike | None | T143831 Epic: Reconcile SmashPig and DonationInterface configuration |
Change 300708 had a related patch set uploaded (by Awight):
Orphan rectification for Drush (SEE NOTES)
@Danny_B I'm curious why you removed Technical-Debt ? I might agree with you, but perhaps you can share the reasoning behind this?
You created this as a subtask, so it inherited all tags even those unrelated such as Patch-For-Review or Technical-Debt. Herald can easily remove Patch-For-Review, but can't decide on other tags. Technical debt is obviously the epic parent itself, however, partial tasks to resolve the epic aren't. Cf. Technical-Debt description and https://www.mediawiki.org/wiki/Technical_debt and https://en.wikipedia.org/wiki/Technical_debt. This task is basically about creating new feature.
Still have some request stuff to workaround:
Fatal error: Class undefined: RequestContext in /vagrant/srv/org.wikimedia.civicrm/extensions/DonationInterface/gateway_common/gateway.adapter.php on line 231
Have some stuff for review in the config deployment repo, on the awight-orphanage branch.
One more important detail from code review: we need to run fraud hooks in the orphan rectifier, but not for recurring Ingenico charges. My current theory is that we set up all the fraud hook globals like on payments, but the recurring job somehow avoids calling those. That looks a tiny bit easier and more correct than making a new configuration for the orphan rectifier.
Change 306310 had a related patch set uploaded (by Awight):
Kill any configured fraud checks before making recurring charges
Change 306564 had a related patch set uploaded (by Awight):
[WIP] DonationInterface settings for running under the CRM
Change 306575 had a related patch set uploaded (by Awight):
Let DI globals come from configuration files.
Change 306705 had a related patch set uploaded (by Ejegg):
Add more dirs to composer classmap
Change 306711 had a related patch set uploaded (by Ejegg):
Enablement switch for BannerHistoryLog
Change 306730 had a related patch set uploaded (by Ejegg):
Add settings for donation-interface library
Change 306806 had a related patch set uploaded (by Ejegg):
Re-enable recurring GC tests, add more
This has been deployed and run one-off, but I haven't convinced myself that we're running fraud checks. In fact, one order which failed CVV on the frontend was happing SET_PAYMENT by the new rectifier. OID 3258119210.
Looks like the glitch is that post_process_get_orderstatus gets called too early, before we've pulled in the CVV and AVS responses. That was a wart anyway--we should go ahead and parse the response as part of normal unstaging.
Change 306852 had a related patch set uploaded (by Awight):
Be less magical about unstaging order status things
Change 306853 had a related patch set uploaded (by Awight):
Don't use frontend classes from fraud filters
Note: I was able to run all the fraud filtering with the pending patches applies, but something bad happened with action ranges, a score of 110 still left me with action='process'. To be continued...
Change 306564 merged by jenkins-bot:
DonationInterface settings for running under the CRM
Change 308786 had a related patch set uploaded (by Awight):
Always call antifraud hooks after get_orderstatus
Change 309196 had a related patch set uploaded (by Awight):
No longer testGetOrderStatusSkipsIfFail
Change 308786 merged by jenkins-bot:
Always call antifraud hooks after get_orderstatus
Consolidating the pending code review:
https://gerrit.wikimedia.org/r/#/c/309516/
https://gerrit.wikimedia.org/r/#/c/306852/
https://gerrit.wikimedia.org/r/#/c/306853/
Change 306852 merged by jenkins-bot:
Be less magical about unstaging order status things
Change 310933 had a related patch set uploaded (by Awight):
Re-enable recurring GC tests, add more
Change 306310 merged by jenkins-bot:
Kill any configured fraud checks before making recurring charges