[00:01:02] Fundraising Sprint Rocket Surgery 2016, Fundraising Sprint Stirring The Pot, Fundraising Sprint Testing on Production, Fundraising-Backlog, and 3 others: Banner not showing up on site - https://phabricator.wikimedia.org/T144952#2711496 (AndyRussG) @K4-713 Thanks! I'm pretty sure it's somehow rela... [00:46:19] (PS2) Awight: Shuffle config usages around to be less of a jerk [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 [00:47:05] (CR) Awight: Shuffle config usages around to be less of a jerk (2 comments) [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 (owner: Awight) [00:51:01] (PS2) Awight: Enable flake8 tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314487 [00:51:18] (PS3) Awight: Regression test for T143903 [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314509 (https://phabricator.wikimedia.org/T143903) [00:55:33] (PS3) Awight: Enable flake8 tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314487 [00:55:35] (PS3) Awight: Shuffle config usages around to be less of a jerk [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 [00:55:37] (PS4) Awight: Regression test for T143903 [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314509 (https://phabricator.wikimedia.org/T143903) [00:56:09] cwd: ^ almost in shape to enable tox on tools btw! [00:58:21] (PS1) Cdentinger: Fix subscr_cancel error [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315613 [00:59:04] excellent [01:00:32] I'm wondering how to test the silverpop export tho [01:00:50] It would be great if I could get away with a sqlite::memory [01:01:08] But i don't think it's happening [01:04:48] i hear the words silverpop export a lot but i don't actually know anything about it [01:04:53] is it a csv ? [01:08:11] check it out... https://github.com/wikimedia/wikimedia-fundraising-tools/blob/master/silverpop_export/update_table.sql [01:08:25] Not actually "silverpop" anymore as they were absorbed by IBM or something [01:08:39] it's a nightly export of our everything. [01:09:12] yikes [01:09:25] best not to misplace those [01:09:58] i don't see why it wouldn't work with sqlite as long as you populate all the source data [01:10:14] might be a big pain [01:10:26] yeah looks like it [01:10:27] the key and collation stuff has slightly different syntax [01:10:41] rgh [01:10:57] shoudln't pdo handle that? [01:11:12] well not if they are raw queries [01:11:28] dag [01:11:31] there's no PDO layer or nothing yeah [01:11:42] ACtually there's one layer, but not something to write home about [01:12:01] We replace database names from python and run each statement in its own python thing [01:12:05] it's gnasty [01:13:47] yeah i can't think of a way besides firing up a test mysql [01:14:46] nothing besides mysql is probably going to understand that query [01:14:48] not the worst thing to do [01:14:50] SET storage_engine=MEMORY [01:15:42] but how does that work in CI? [01:16:29] I'm looking forward to finding out ;) [01:16:57] it should be a *lot* faster than disk-backed tho [01:17:57] gtg dine, see you tomorrow! [01:18:45] later! [03:55:34] (PS2) Cdentinger: Fix subscr_cancel error [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315613 [04:50:02] Fundraising Sprint Rocket Surgery 2016, Fundraising Sprint Stirring The Pot, Fundraising Sprint Testing on Production, Fundraising-Backlog, and 3 others: Banner not showing up on site - https://phabricator.wikimedia.org/T144952#2711798 (AndyRussG) Summary of what seem to be the most important dat... [06:20:13] (CR) Awight: [C: 2] "Great, but with the @dataProvider reservations I mentioned inline." (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315613 (owner: Cdentinger) [06:21:19] (Merged) jenkins-bot: Fix subscr_cancel error [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315613 (owner: Cdentinger) [08:09:53] (PS1) Awight: WIP Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [13:00:13] ejegg|away: are you still on east coast time? do you see the fail mail over the last 4 hours? [14:11:12] (PS1) Cdentinger: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315678 [14:14:17] (CR) Cdentinger: [C: 2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315678 (owner: Cdentinger) [14:14:29] (Merged) jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315678 (owner: Cdentinger) [14:18:17] !log updated smashpig from 28ba033be2f50fc7837c788dc452753ff82284e1 to e89f1b5316955b5d6576d4c71f8ab840db9b4de2 [14:18:22] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [15:21:40] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [15:39:07] (PS2) Awight: WIP Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [15:39:38] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [15:48:56] (PS3) Awight: WIP Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [15:49:53] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [15:58:24] (PS4) Awight: WIP Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [15:58:54] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [16:05:21] dstrine: sorry... on central time, but taking a look at the failmail now [16:08:01] ejegg: was my bad, should be fixed now [16:08:58] ah yeah, just saw what it was [16:09:00] thanks! [16:11:53] didn't know it was gonna get so noisy [16:15:22] So, I'm looking back over the stomp queues. something seems to still be sending refunds to stomp. trying to figure that one out [16:16:17] hmmm [16:16:38] wouldn't be the orphan rectifier since it just handles from end stuff huh? [16:17:33] right [16:17:41] either listener or audit [16:18:08] but the settings on production don't seem like they should be sending anything to stomp [16:18:33] hmm [16:18:35] something in tools? [16:18:52] let's see... [16:22:14] oh wait, these are from the listener, but they stopped yesterday. [16:23:07] adyen listener? [16:23:09] I definitely asked Jeff_Green to deploy an update to SmashPig.yaml yesterday, but I thought we had turned off the recurring and refund mirroring last week [16:23:12] amazon listener [16:24:49] gotta take the pooch for a jog, back soon [16:24:53] ejegg: i don't remember an update yesterday [16:25:19] last change was the 11th I think [16:26:35] Jeff_Green: didn't we just update it to add the jobs-adyen queue? [16:26:49] oh right, I guess that was the 11th [16:26:56] yeah that's right [16:27:13] speaking of listeners, what do you think the chances are that the listener can run cross-site [16:27:33] you mean in a different data center? [16:27:41] yeah, using the codfw listener for a while? [16:28:06] i want to upgrade thulium to jessie we won't have a replacement box until next FY [16:40:25] oho, we had configured refund and recurring mirroring in the defaults, not in the override file [16:40:36] so the code deploy yesterday was what turned it off [17:00:02] Fundraising Sprint Octopus Untangling, Fundraising Sprint Pretending This Isn't Happening, Fundraising Sprint Qwerty Thwacking, Fundraising Sprint Rocket Surgery 2016, and 4 others: Decimal dates are breaking our queue consumer - https://phabricator.wikimedia.org/T140667#2713565 (DStrine) [17:00:56] fr-tech: You will be surrounded by luxury. [17:00:56] -- discuss. [17:01:56] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Zero at the beginning of zip code not being reflected upon import - https://phabricator.wikimedia.org/T148056#2713567 (LeanneS) [17:03:27] makin coffee, be there soon [17:07:19] ooh coffee. now I'm making coffee too! [17:11:52] aaaarg! [17:13:09] aaargh? [17:15:17] oh yeah! [17:17:24] Really just an existential aaarg, not a specific something broke kinda aaaarg [18:10:26] dang--CI jobs don't report the machine name any more? [18:11:07] huh. Or they're respawned per job now? [18:19:03] Fundraising-Backlog: Remove refunds from Silverpop donation history - https://phabricator.wikimedia.org/T148062#2713848 (Krenair) [18:24:07] Fundraising Sprint Rocket Surgery 2016, Fundraising Sprint Stirring The Pot, Fundraising Sprint Testing on Production, Fundraising-Backlog, and 3 others: Banner not showing up on site - https://phabricator.wikimedia.org/T144952#2713885 (AndyRussG) Following up on the patch ^ by @Ejegg, addressing... [18:24:38] ejegg: ^ some more thoughts, dug a wee bit more... also thx!!!! [18:26:00] thank you! [18:30:30] ejegg: mebbe we can deploy ur patch to the beta cluster (before it gets merged) to see if the error still happens [18:30:56] AndyRussG|mostly: oh cool, can you show me how to do that? [18:31:46] * ejegg looks for docs [18:34:26] aha, https://wikitech.wikimedia.org/wiki/Nova_Resource:Deployment-prep/How_code_is_updated#Cherry-picking_a_patch_from_gerrit [18:35:05] XenoRyet: want to try figuring this out with me? ^^ [18:35:24] Sure [18:35:33] I think we might need to get access to the beta cluster labs project first [18:35:36] let's see [18:35:47] ejegg: I haven't worked out mysql password fu yet, but wanted you to know: https://gerrit.wikimedia.org/r/#/c/315638/4 [18:35:50] I'm psyched! [18:36:06] owow! [18:37:39] XenoRyet: https://wikitech.wikimedia.org/wiki/Help:Access [18:38:20] XenoRyet: have you logged into any labs stuff yet? You'd use your wikitech/gerrit password [18:38:27] No, this will be the first time. [18:38:59] you have a wikitech account though, right? [18:39:27] oh yah, of course you do, cause gerrit [18:39:37] Yea [18:40:42] OK, looks like we're both at this step: https://wikitech.wikimedia.org/wiki/Help:Getting_Started#Projects [18:41:21] so this is the project in question: https://wikitech.wikimedia.org/wiki/Nova_Resource:Deployment-prep [18:41:59] aha, and AndyRussG is listed as an admin [18:42:08] ejegg: XenoRyet I have access [18:42:15] AndyRussG|mostly: could you please add us to the project? [18:42:19] cool [18:44:39] ejegg: XenoRyet: yeah! what are ur wikitech usernames? [18:44:47] Just Ejegg and XenoRyet ? [18:44:56] Yep [18:45:08] yep [18:46:23] (PS5) Awight: WIP Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [18:46:37] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [18:47:33] ejegg: XenoRyet: K I just added u both! [18:47:50] You saw the extra .ssh config you need to add, right? [18:48:05] Cool, thanks. [18:48:18] Getting to that part, I think. [18:48:20] rockin, looking at the config thing [18:50:04] Running the Hive query in the notes again adding hostname [18:55:39] ok, here I am on deployment-puppetmaster [18:55:46] weee! [18:55:56] XenoRyet: are you able to get there? [18:56:00] Almost [18:58:45] Ok, yea, in now. [18:59:03] cool, I'm looking at the cherry-pick instructions [18:59:26] https://wikitech.wikimedia.org/wiki/Nova_Resource:Deployment-prep/How_code_is_updated#Cherry-picking_a_patch_from_gerrit [18:59:37] Yea, looking there too. [19:00:03] hmm, wait, is that puppet-specific? [19:00:55] (PS6) Awight: Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [19:01:13] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [19:03:54] Yea, this looks pretty puppetty [19:04:28] OK, let's look for something more like scap [19:06:38] AndyRussG|mostly: where are you changing the deployed core code on beta? That doesn't mess with the beta-code-update-eqiad job ? [19:07:18] ejegg: never done it actually :) [19:07:32] oh, I thought you did some livehacking earlier [19:08:24] will ask in #wikimedia-labs [19:10:13] ejegg: yeah that had been an idea but instead we took advantage of instrumentation already in the MessageCache class, and pushed a config change to turn it on [19:10:39] ah cool. asking in -releng per yuvi's suggestion [19:10:48] ah yes I was gonna say :) [19:11:49] ejegg: XenoRyet: here's the logstash query for MessageCache stuff on beta [19:12:06] (PS7) Awight: Tests for the bulk export [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) [19:12:36] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315638 (https://phabricator.wikimedia.org/T131770) (owner: Awight) [19:16:25] XenoRyet: AndyRussG|mostly - getting answers from Krenair in -labs [19:16:43] cool! [19:19:42] ejegg: silverpop_export tests pass on CI now, thanks to paladox! [19:20:08] awight: freaking cool! [19:20:28] oh, i see you reshuffled that config stuff too. will re-examine shortly [19:20:32] It was hairy, but a pretty lightweight amount of python glue [19:20:45] the SQL is... open to debate [19:21:26] I went minimal. lots of missing columns if we don't need them, I'm inserting contacts like "insert (id) values (2)" [19:21:29] :) [19:21:47] nice to be able to focus on the actual code being tested though [19:21:59] I love programming languages built for lazy, bad programmers :) [19:24:37] heh [19:26:53] AndyRussG|mostly: I need to eat and plug in the lappy, but I can test that core change in 1/2 hr or so [19:28:51] ejegg|food: also fin! [19:28:52] fine [19:34:10] (PS1) Awight: WIP fix py tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) [19:34:29] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) (owner: Awight) [19:38:54] (PS2) Awight: WIP fix py tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) [19:39:17] plz start standup without me fr-tech. i'll be 10 min late or so [19:41:16] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) (owner: Awight) [19:42:53] (PS3) Awight: WIP fix py tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) [19:43:09] (CR) Awight: "check experimental" [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/315741 (https://phabricator.wikimedia.org/T145012) (owner: Awight) [19:57:53] ejegg|food: I should run at 1:05 or 1:10 Pacific Time and then should back by 1:45 [20:05:27] * robla just read the backlog of T144952, and I'm happy to help if/how I can [20:17:48] Thanks robla! I just added you as a reviewer for a possibly-related patch: https://gerrit.wikimedia.org/r/315712 [20:20:34] * robla looks at the patch on the off chance that he might be able to say something constructive about it [20:33:00] (PS1) Ejegg: Update PopulateQueueFromDump for PHPQueue [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315754 (https://phabricator.wikimedia.org/T147982) [20:33:31] fr-tech I need to use that to move the 65 Stomp messages into Civi ^^ [20:38:04] (PS4) Ejegg: Shuffle config usages around to be less of a jerk [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 (owner: Awight) [20:38:10] (CR) Ejegg: [C: 2] Shuffle config usages around to be less of a jerk (1 comment) [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 (owner: Awight) [20:38:17] (Merged) jenkins-bot: Shuffle config usages around to be less of a jerk [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314486 (owner: Awight) [20:40:39] (PS4) Ejegg: Enable flake8 tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314487 (owner: Awight) [20:44:10] oh hey, Yakkety Yak is official [20:44:17] aka ubuntu 16.10 [20:45:10] haha [20:48:37] cwd got a minute to review a maintenance script? https://gerrit.wikimedia.org/r/315754 [20:50:42] i will look at it shortly, pulling on some strings with this adyen fraud thing [20:52:10] ah cool [20:52:16] let me know if I can help with that at all [20:55:36] ejegg: back, test-ready netime :) [20:55:42] AndyRussG: cool, let me get my bearings [20:56:03] K :) [20:56:28] I'm all here for all the rest of the day so no rush :) [20:56:45] (though if it works it'd be fun to deploy......!) [21:01:01] (CR) Ejegg: [C: 2 V: 2] Enable flake8 tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314487 (owner: Awight) [21:01:57] AndyRussG: ok, I'm all set to do that, let me just time it right after an automatic update [21:02:23] AndyRussG: looks like one will happen in a minute [21:02:39] aaaah they're on a schedule, cool! [21:02:57] yah, every 10 minutes on the 3s [21:05:30] (Merged) jenkins-bot: Enable flake8 tests [wikimedia/fundraising/tools] - https://gerrit.wikimedia.org/r/314487 (owner: Awight) [21:05:43] ok, doing the cherry-pick [21:08:17] ok, going to sync [21:08:45] weeeeeeeeeeeeee :) [21:09:38] AndyRussG: OK, all synced [21:10:07] :D [21:10:16] now we have like 3 minutes to test [21:10:29] sorry, didn't realize how long the real updater would take [21:12:45] AndyRussG: eek, about to be another auto-update. did you have time to do anything? [21:13:24] ejegg: yeah haven't seen the error yet! [21:13:40] hmm [21:13:52] that's not conclusive though, right? [21:14:11] ejegg: correct, it didn't always appen [21:15:06] well, the patch will be live for another 2 minutes [21:15:24] ejegg: ah k testing smore [21:15:53] (I un-picked in /srv/mediawiki-staging, but the scap jobs rebuilds the i18n cache before syncing files) [21:17:30] AndyRussG: oop, overwritten [21:17:49] I can keep doing that dance tho [21:18:48] AndyRussG: ok, re-applied :) [21:18:57] ahhh cool! [21:20:10] ejegg: no errors so far...! [21:20:12] https://logstash-beta.wmflabs.org/app/kibana#/dashboard/default?_g=(refreshInterval:(display:Off,pause:!f,value:0),time:(from:now-1h,mode:quick,to:now))&_a=(filters:!(),options:(darkTheme:!f),panels:!((col:1,id:Dashboards,panelIndex:1,row:1,size_x:12,size_y:1,type:visualization),(col:1,id:Events-Over-Time,panelIndex:2,row:2,size_x:12,size_y:2,type:visualization),(col:1,id:Event-Types,panelIndex [21:20:14] :3,row:4,size_x:9,size_y:2,type:visualization),(col:10,id:Event-Level,panelIndex:4,row:4,size_x:3,size_y:2,type:visualization),(col:1,columns:!(type,level,wiki,host,message),id:Default-Events-List,panelIndex:5,row:6,size_x:12,size_y:25,sort:!(%27@timestamp%27,desc),type:search)),query:(query_string:(analyze_wildcard:!t,query:%27centralnotice%20AND%20message%20AND%20key%27)),title:default,uiStat [21:20:16] e:(P-2:(vis:(legendOpen:!f)),P-3:(vis:(legendOpen:!f)),P-4:(vis:(legendOpen:!f)))) [21:20:28] hehe [21:20:35] I'll take your word for it [21:20:56] https://goo.gl/x0CpO4 [21:21:28] Feel free also to click on the new banner previews! [21:21:30] https://meta.wikimedia.beta.wmflabs.org/wiki/Special:CentralNoticeBanners [21:21:43] k, looks good! [21:22:31] hmm, i may have just gotten a blank [21:23:32] Ooof [21:23:37] Ah no, they're all blank [21:23:39] oh wait, that's just the default text [21:23:42] "bannerHtml":"\u003C!-- Empty banner --\u003E" [21:23:49] Yeah that's all good :) [21:24:01] Check network tab, BannerLoader response [21:24:04] ok, beta-scap's about to run us over again [21:27:19] ejegg: still looking great! [21:27:39] lmk when it's overwritten and back to master? if the error reproduces easily then, that'd also something! [21:27:55] oops, I just re-overwrote master [21:28:04] hehe all good :) [21:28:12] K I'll keep trying to get it! [21:30:42] on a less positive note, the meatloaf I made for today's midday meal tastes awful [21:30:54] we'll see if people eat it smothered in bbq sauce [21:31:25] * AndyRussG listens for gagging sounds from desk [21:32:43] still no errors! [21:33:39] nor gagging [21:33:50] I don't hear any "mmmmm ymmm" noises from the dining room either, though [21:37:22] ejegg: when's the next update? [21:37:44] AndyRussG: oop, let me see [21:38:09] AndyRussG: I think it just finished [21:38:20] K let's try now [21:38:21] yeah, looks like the patch is wiped out across beta [21:41:59] ejegg: hmmm still no errors happening also... the watched kettle [21:42:33] huh. you've been adding new banners? [21:44:20] ejegg: yea a million [21:44:26] are you sure he patch is wiped out? [21:44:35] hmm, I thought it was [21:44:51] it's definitely not in the /srv/mediawiki-staging tree any more [21:45:06] but I can do a sync-file to make sure it's across the cluster [21:45:15] ah K pls yeah that'd be phab [21:45:30] err, after the normal scap finishes and releases its lock [21:45:42] hmm [21:45:50] So maybe it wasn't synced on all hosts? [21:46:56] if this fixes it we can definitely ask for quick review and maybe deploy from core-ish folks [21:48:16] I was pretty sure the last scap wiped it out - i think that does a full rsync [21:48:23] but I'm re-syncing just the one file [21:48:30] K lmk [21:49:07] ok, definitely synced now [21:55:55] ejegg: nope no errors also [21:56:24] AndyRussG: hrm... [21:56:35] let's see what else has been happening in code lately [21:57:25] yeah exactly [22:03:12] definitely some cachey stuff in the recent logs [22:12:08] ejegg: I don't see anything that looks like it might impact, so far... Last error logged was Oct. 11 3 hrs UTC [22:12:25] ejegg: ah no [22:12:34] 2016-10-12T17:12:40 [22:14:00] (CR) XenoRyet: [C: 2] Update PopulateQueueFromDump for PHPQueue [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315754 (https://phabricator.wikimedia.org/T147982) (owner: Ejegg) [22:14:25] ty XenoRyet [22:14:32] No worries [22:14:47] relocating for outlet... [22:15:21] (Merged) jenkins-bot: Update PopulateQueueFromDump for PHPQueue [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315754 (https://phabricator.wikimedia.org/T147982) (owner: Ejegg) [22:16:52] maybe it's rather dependent on traffic volume somehow? [22:34:20] (PS1) Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315847 [22:34:30] (CR) Ejegg: [C: 2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315847 (owner: Ejegg) [22:40:28] (CR) Ejegg: [V: 2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - https://gerrit.wikimedia.org/r/315847 (owner: Ejegg) [22:41:18] !log updated SmashPig from e89f1b5316955b5d6576d4c71f8ab840db9b4de2 to 3c3d115d8388515eaf1f88b4263a56985454f3a1 [22:41:23] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [22:42:10] OK, I'm going to re-queue the 65 missing messages. Wouldn't be surprised if some of them fail! [22:43:24] ok, re-queued, let's see what the consumer does [23:06:02] (PS1) Ejegg: Retry capture later when donor details are missing [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315858 (https://phabricator.wikimedia.org/T147378) [23:06:32] cwd super-lazy version: ^^^ [23:07:03] not even adding the source_enqeueued_date field till we try to requeue the first time [23:07:12] but it should make sure we don't infinitely retry things [23:08:23] oh ha, that requeue delayed messages job should run more often [23:11:34] well, only 19 of those orphaned stomp messaged left. Guess I can fix the json dumps by hand [23:13:11] yikes [23:13:21] wont' the audit find em? [23:13:38] or wait, this was the audit? [23:20:27] cwd some of them were, i think [23:20:36] a bunch are from 9/29... [23:24:49] ok, re-queued the edited batch, let's hope these make it through [23:28:27] all but 2. huh [23:29:08] both amazon messages, from the listener [23:29:24] let's see if they got some janky order ids in the pending db [23:31:37] nope, not there at all [23:35:50] Fundraising-Backlog: Access for new contractor, Allen - https://phabricator.wikimedia.org/T148105#2715060 (LeanneS) [23:45:11] (PS2) Ejegg: Retry capture later when donor details are missing [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/315858 (https://phabricator.wikimedia.org/T147378) [23:45:29] fr-tech this is a potential fix for the Adyen thing dstrine said would be worth a friday deploy ^^^ [23:49:34] i've been thinking about it [23:49:44] What did the problem end up being? [23:50:04] my gut feeling is we shouldn't use exceptions for this but i'm not sure i have a better alternative [23:51:52] XenoRyet: the donor info wasn't in pending db yet when the auth came in [23:52:18] somebody entered cc details fast or our pending qc was lagging [23:52:33] anyway, we should be able to retry later when that happens [23:54:46] Ok, yea. I think I remember how that works. [23:55:03] But the important bit is that donor data will show up later, so we can try again. [23:56:22] Bah, didn't see the clock. I've got to go pick up the boy. I can look at it first thing in the morning if cwd doesn't get to it tonight. [23:56:31] k, thanks! [23:56:57] yeah, getting dark here, I suppose I'll pick this up in the morning [23:58:10] i also have to run and let out a friend's dog but i will CR this later