1245558 - PolyCrashStorage fails to write to other destinations after a failure

Status: RESOLVED FIXED

(Socorro :: Backend, task)

Description

[[DEACTIVATED] Adrian Gaudebert]

Supposedly, the PolyCrashStorage class writes to a list of destinations, and if one of them fails, it's supposed to still write to the others. While failing to switch our ES clusters, we realized that it was failing at writing to other stores. 

We'll want to have some form of test that covers this case. Here's a scenario: 

1. storage0 passes
2. storage1 fails
3. storage2 should be attempted and pass
4. an error is raised and the job is marked as failing (and should be retried later on)

At the moment, storage2 is never even attempted.

Comment 1

[[DEACTIVATED] Adrian Gaudebert]

Attachment: Link to Github pull-request: https://github.com/mozilla/socorro/pull/3183

Comment 2

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/877850109804124ba010dac6d8d743d7259592f0
Fixes bug 1245558 - Added failures handling to save_raw_and_processed method of PolyCrashStorage class. r=peterbe