derekgottlieb
Mike TurnerFor #tailtuesday…
#buccaneer #blackburn #airbrake #tailplane #newark #museum #avgeek #aviation #flying
Forgot to followup on this. I later figured out the bug and submitted a PR (https://github.com/airbrake/airbrake/pull/1244) but it seems to be languishing as Airbrake may have lost their only Ruby dev? Until they merge that, I baked my fixed RetryableJobsFilter directly into my apps to workaround the issue. #ruby #Sidekiq #Airbrake
Separate var when overriding max retries before notify in Sidekiq jobs by dgsangoma · Pull Request #1244 · airbrake/airbrake
Previously, the @max_retries var was being used for both the global default retry limit as well as when a developer indicates they want error notifications before retries have been exhausted for a ...
Anyone use #Sidekiq with #Airbrake? Seeing premature error notifications when there are retries left. https://github.com/airbrake/airbrake/issues/1239
Premature notification in Airbrake::Sidekiq::RetryableJobsFilter · Issue #1239 · airbrake/airbrake
We've set a global Sidekiq default of 5 for max_retries, but overridden that in the code for particular jobs to be higher like in the ProgressWorker job referenced below that has been increased...