Skip to content

Commit

Permalink
permit_concurrent_loads is not needed with zeitwerk autoloader
Browse files Browse the repository at this point in the history
This was only needed for classic autoloader.  The core "freedom" patch made zeitwerk autoloader
bypass the interlock anyway, so now that we're only supporting zeitwerk, this is no longer
needed.

Co-dependency:
ManageIQ/manageiq#22801

Part of the rails 7 upgrade: ManageIQ/manageiq#22052
  • Loading branch information
jrafanie committed Dec 4, 2023
1 parent 9d874ff commit 31df202
Showing 1 changed file with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -145,10 +145,8 @@ def self.verbose_rc(return_code)
def self.run_ruby_method(code, miq_request_id)
ActiveRecord::Base.connection_pool.release_connection unless Rails.env.test?
with_automation_env do
ActiveSupport::Dependencies.interlock.permit_concurrent_loads do
run_method(Gem.ruby, miq_request_id) do |stdin|
stdin.puts(code)
end
run_method(Gem.ruby, miq_request_id) do |stdin|
stdin.puts(code)
end
end
end
Expand Down

0 comments on commit 31df202

Please sign in to comment.