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

Can we add a "require 'rails/all'" to the initializer? #300

Open
eileen-jiang opened this issue Feb 26, 2024 · 1 comment
Open

Can we add a "require 'rails/all'" to the initializer? #300

eileen-jiang opened this issue Feb 26, 2024 · 1 comment

Comments

@eileen-jiang
Copy link

eileen-jiang commented Feb 26, 2024

Related to #288 (cc. @tgwizard).

Request: Can we add a require 'rails/all' to the Phonelib initializer? i.e.

if defined?(Rails)
    require 'rails/all' # <-- added
    class Phonelib::Railtie < Rails::Railtie

Happy to make this change myself, but wanted to see if there's a faster workaround.

Context: I'm having trouble starting up the multi-process feature of Sidekiq Enterprise. More details on the issue here.

At least Rails::Railtie and Rails::Engine are undefined.

Adding gem 'phonelib', require: 'rails/all' to my Gemfile does not work.

> bundle exec sidekiqswarm
[swarm] Preloading Bundler groups ["default"]
/Users/eileenj/.rvm/gems/ruby-3.1.4/gems/phonelib-0.8.7/lib/phonelib.rb:22:in `<top (required)>': uninitialized constant Rails::Railtie (NameError)

    class Phonelib::Railtie < Rails::Railtie
                                   ^^^^^^^^^
@daddyz
Copy link
Owner

daddyz commented Mar 16, 2024

@eileen-jiang did you try to use approach described in sidekiq/sidekiq#4766 (comment) and sidekiq/sidekiq#4766 (comment)

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

No branches or pull requests

2 participants