Skip to content

Drop sidekiq retries down to 3#1173

Open
steveyken wants to merge 1 commit intomasterfrom
GCW-3435-job-retry-limit
Open

Drop sidekiq retries down to 3#1173
steveyken wants to merge 1 commit intomasterfrom
GCW-3435-job-retry-limit

Conversation

@steveyken
Copy link
Copy Markdown
Member

Ticket Link:

https://jira.crossroads.org.hk/browse/GCW-3435

What does this PR do?

FEATURE: For push notification jobs, these jobs don't need to be retried 25 times. If they fail, we should try 3 times and then give up.

Impacted Areas

Push notifications sent to iOS and Android devices.

@steveyken
Copy link
Copy Markdown
Member Author

I'm going back to the drawing board on this... sidekiq_options requires sidekiq 6 which requires rails 6... we are on rails 5.2 at the moment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants