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

Sidekiq UI won't display (dor-services-app and sdr-api) #2638

Closed
ndushay opened this issue Apr 1, 2021 · 5 comments · Fixed by #2645
Closed

Sidekiq UI won't display (dor-services-app and sdr-api) #2638

ndushay opened this issue Apr 1, 2021 · 5 comments · Fixed by #2645

Comments

@ndushay
Copy link
Contributor

ndushay commented Apr 1, 2021

from https://app.honeybadger.io/projects/50568/faults/78615180

We get this error info

image

But it seems like we should be ok?

# frozen_string_literal: true

Rails.application.routes.draw do
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html

  require 'sidekiq/web'
  Sidekiq::Web.set :session_secret, Rails.application.secrets[:secret_key_base]
  mount Sidekiq::Web => '/queues'
...

from https://github.com/mperham/sidekiq/blob/master/Changes.md

" Numerous people have hit "Forbidden" errors and struggled with Sidekiq's Web UI session requirement. If you have code in your initializer for Web sessions, it's quite possible it will need to be removed. "

@jcoyne
Copy link
Contributor

jcoyne commented Apr 2, 2021

Take a look at sidekiq/sidekiq#4850 (comment)

@mjgiarlo
Copy link
Member

mjgiarlo commented Apr 2, 2021

See the updated Sidekiq documentation on how to render the UI in an API-only application.

@mjgiarlo
Copy link
Member

mjgiarlo commented Apr 2, 2021

@jcoyne @ndushay should this be a high-ish priority FR issue, given how it can be useful to see the UI? If so, we should get this on Vivian's radar.

@ndushay
Copy link
Contributor Author

ndushay commented Apr 2, 2021

It's on the production priorities board and I will bring it to her attention at the handoff meeting.

@ndushay ndushay moved this from New Issues (Needs Triage) to Ready (Ordered by Priority) in Infrastructure Portfolio Production Priorities (MOVED TO https://github.com/orgs/sul-dlss/projects/58) Apr 2, 2021
@ndushay ndushay added the bug label Apr 2, 2021
@andrewjbtw
Copy link

I also endorse this as a priority. The sidekiq UI is useful for understanding what's going on with the non-resque workflow steps, among other things.

@mjgiarlo mjgiarlo self-assigned this Apr 2, 2021
mjgiarlo added a commit that referenced this issue Apr 2, 2021
Fixes #2638

This PR follows the instructions from the Sidekiq docs: https://github.com/mperham/sidekiq/wiki/Monitoring#rails-api-application-session-configuration

It also adds a SECRET_KEY_BASE to the web docker-compose service, which is required now that we switched to RAILS_ENV=production. This value is not used anywhere else, so it seems safe to have here. No .env hijinks needed, I do not think.
mjgiarlo added a commit to sul-dlss/sdr-api that referenced this issue Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging a pull request may close this issue.

4 participants