Внесены изменения для поддержки Discourse в TruffleRuby

bjfish · 28.Апрель.2022 17:06:17

Я хотел бы запросить следующие 3 изменения для поддержки работы Discourse на TruffleRuby:

Запрошенное изменение
Обновить bin/rails, чтобы отключать unicorn, когда fork не поддерживается.

Ошибка

<internal:core> core/kernel.rb:764:in `fork': fork is not available (NotImplementedError)

Файлы
bin/rails

github.com/discourse/discourse

bin/rails

b266a3696

#!/usr/bin/env ruby
# frozen_string_literal: true

if !ENV["RAILS_ENV"] && (ARGV[0] == "s" || ARGV[0] == "server")
  ENV["UNICORN_PORT"] ||= "3000"

  if ARGV[1] == "-p" && (port = ARGV[2].to_i) > 0
    ENV["UNICORN_PORT"] = port.to_s
  end

  ENV["RAILS_LOGS_STDOUT"] ||= "1"

  exec File.expand_path("unicorn", __dir__)
end

APP_PATH = File.expand_path('../config/application', __dir__)
require_relative '../config/boot'
require 'rails/commands'

Причина
TruffleRuby не поддерживает fork.

Предложение
Отключать unicorn, если !Process.respond_to?(:fork)

========================================================

Запрошенное изменение
Отключить mini_profiler для TruffleRuby

Ошибка

<internal:core> core/kernel.rb:236:in `gem_original_require': cannot load such file -- stackprof (LoadError)

Файлы
config/initializers/006-mini_profiler.rb

github.com/discourse/discourse

config/initializers/006-mini_profiler.rb

b266a3696

# frozen_string_literal: true

# If Mini Profiler is included via gem
if Rails.configuration.respond_to?(:load_mini_profiler) && Rails.configuration.load_mini_profiler
  require 'rack-mini-profiler'
  require 'stackprof'

  begin
    require 'memory_profiler'
  rescue => e
    STDERR.put "#{e} failed to require mini profiler"
  end

  # initialization is skipped so trigger it
  Rack::MiniProfilerRails.initialize!(Rails.application)
end

if defined?(Rack::MiniProfiler) && defined?(Rack::MiniProfiler::Config)
  # note, we may want to add some extra security here that disables mini profiler in a multi hosted env unless user global admin
  #   raw_connection means results are not namespaced

This file has been truncated. show original

Причина
Stackprof в настоящее время не поддерживается в TruffleRuby.

Предложение
Отключать загрузку mini_profiler, когда RUBY_ENGINE равен “TruffleRuby”.

========================================================

Запрошенное изменение
Предотвратить загрузку rbtrace

Аналогичные случаи, где уже предотвращается загрузка rb_trace:

github.com/discourse/discourse

config/application.rb

b266a3696


      
                        discourse/tests/test-boot-rails.js
                        wizard/test/test_helper.js
                      }.include?(logical_path) ||
                      logical_path =~ /\/node_modules/ ||
                      logical_path =~ /\/dist/
                  end
                end
              end
          
              if ENV['RBTRACE'] == "1"
                require 'rbtrace'
              end
          
              config.generators do |g|
                g.test_framework :rspec, fixture: false
              end
            end
          end

github.com/discourse/discourse

config/environments/development.rb

b266a3696


      
            config.hosts.concat(hosts.split(","))
          end
          
          require 'middleware/turbo_dev'
          config.middleware.insert 0, Middleware::TurboDev
          require 'middleware/missing_avatars'
          config.middleware.insert 1, Middleware::MissingAvatars
          
          config.enable_anon_caching = false
          if RUBY_ENGINE == "ruby"
            require 'rbtrace'
          end
          
          if emails = GlobalSetting.developer_emails
            config.developer_emails = emails.split(",").map(&:downcase).map(&:strip)
          end
          
          if ENV["DISCOURSE_SKIP_CSS_WATCHER"] != "1" && (defined?(Rails::Server) || defined?(Puma) || defined?(Unicorn))
            require 'stylesheet/watcher'
            STDERR.puts "Starting CSS change watcher"
            @watcher = Stylesheet::Watcher.watch

Ошибка
Failure/Error: require ‘rbtrace’

Предложение
Использовать один из методов, указанных в ссылках выше, чтобы предотвратить загрузку rbtrace.

Файлы
spec/rails_helper.rb

github.com/discourse/discourse

spec/rails_helper.rb

b266a3696


      
            SimpleCov.start 'rails' do
              add_group 'Libraries', /^\/lib\/(?!tasks).*$/
              add_group 'Scripts', 'script'
              add_group 'Serializers', 'app/serializers'
              add_group 'Services', 'app/services'
              add_group 'Tasks', 'lib/tasks'
            end
          end
          
          require 'rubygems'
          require 'rbtrace'
          require 'pry'
          require 'pry-byebug'
          require 'pry-rails'
          require 'fabrication'
          require 'mocha/api'
          require 'certified'
          require 'webmock/rspec'
          
          class RspecErrorTracker
            def self.last_exception=(ex)

gerhard · 29.Апрель.2022 12:12:41

Кажется, это разумные запросы.

Кстати: изменилось ли что-то в том, что mini_racer не поддерживается в TruffleRuby? Это серьёзное препятствие.

bjfish · 29.Апрель.2022 18:55:55

Кстати: изменилось ли что-то в том, что mini_racer не поддерживается в TruffleRuby? Это серьёзное препятствие.

У меня есть ветка, в которой большая часть функциональности mini_racer проходит набор тестов в TruffleRuby с использованием GraalJS. Я полагаю, что она покрывает все необходимые для Discourse функции. Её нужно только проверить, после чего я постараюсь добиться её включения в основную ветку. Вот как я запускаю это локально: https://twitter.com/brandonfish/status/1486755128577253376

bjfish · 02.Май.2022 15:44:49

Исправление:
Отключить загрузку mini_profiler, когда RUBY_ENGINE равен «truffleruby»

sam · 03.Май.2022 05:04:01

mini_profiler всё ещё должен работать нормально, я бы рекомендовал просто не загружать rbtrace / stackprof и т. д. Там есть нестрого связанные зависимости.

mini_profiler может по-прежнему приносить много пользы в мире Truffle, в этом нет ничего специфичного для MRI.

gerhard · 04.Май.2022 20:10:33

Вот PR с запрошенными изменениями.

Мне всё ещё не удалось запустить это успешно, возникла ошибка java.lang.NullPointerException, но это может быть связано с моим грубым отключением mini_racer. Я очень жду момента, когда полностью функциональный Discourse будет работать на TruffleRuby.

Я не углублялся в это, но заметил ошибку, когда отключил только загрузку stackprof и memory_profiler. Поэтому пока я полностью отключил mini_profiler.

2022-05-04 21:48:13 +0200 Rack app ("POST /mini-profiler-resources/results" - (127.0.0.1)): #<SystemStackError: stack level too deep
        from StackOverflowCheckImpl.java:328:in `com.oracle.svm.core.graal.snippets.StackOverflowCheckImpl.newStackOverflowError0'>

Это могло быть побочным эффектом других проблем. Думаю, мы сможем заняться этим, когда будут решены остальные проблемы.

Тема		Ответов	Просм.
Update Gemfile fast_xor platform to :ruby Development	5	1140	21.08.2022
Discourse on Truffleruby Development	14	1286	16.05.2025
Disable page rendering stats on every page Support	3	894	27.02.2016
Discourse Build Error: “An error occurred while installing rbtrace (0.5.1), and Bundler cannot continue Self-hosting	20	2707	25.01.2024
Turn off rack mini profiler Support	4	739	29.05.2022

Внесены изменения для поддержки Discourse в TruffleRuby

Связанные темы