Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use SideKiq for email ActiveJob queues #4609

Open
2 of 6 tasks
rladdusaw opened this issue Dec 3, 2024 · 1 comment · Fixed by #4700
Open
2 of 6 tasks

Use SideKiq for email ActiveJob queues #4609

rladdusaw opened this issue Dec 3, 2024 · 1 comment · Fixed by #4700
Assignees
Labels
feature Updates on existing feature or a new feature request

Comments

@rladdusaw
Copy link
Contributor

rladdusaw commented Dec 3, 2024

User story

While the default Rails queue seems to work, SideKiq would give us better visibility and potentially better reliability for jobs

Acceptance criteria

The application uses SideKiq for queues and can be deployed

  • I can send emails using SideKiq
  • I can track asynchronous email jobs in SideKiq
  • Emails sent on the staging and qa environments are caught by mailcatcher
  • Deployment does not cause emails to get lost.
  • Use the catalog-indexers for sidekiq
  • Run it a systemd service (ansible update)

Implementation notes, if any

The endpoint for /sidekiq will need some form of authentication to prevent unauthorized users from accessing the web frontend.

@rladdusaw rladdusaw added the feature Updates on existing feature or a new feature request label Dec 3, 2024
@sandbergja sandbergja self-assigned this Dec 30, 2024
@sandbergja
Copy link
Member

Re-opening since we reverted #4702. Emails sent from Requests are not compatible with sidekiq. We need to refactor these emails to only use data that sidekiq can handle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Updates on existing feature or a new feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants