good_job 3.22.0 → 3.23.0

data/config/locales/ru.yml

@@ -2,55 +2,55 @@
 ru:
   good_job:
     actions:
-      destroy: Разрушать
-      discard: Отказаться
+      destroy: Удалить
+      discard: Отменить
       force_discard: Принудительно отменить
       inspect: Осмотреть
       reschedule: Перенести
       retry: Повторить попытку
     batches:
       index:
-        older_batches: Старые партии
+        older_batches: Старые группы заданий
         pending_migrations: У GoodJob есть ожидающие миграции базы данных.
-        title: Пакеты
+        title: Группы заданий
       jobs:
         actions:
-          confirm_destroy: Вы уверены, что хотите уничтожить эту работу?
-          confirm_discard: Вы уверены, что хотите отказаться от этого задания?
+          confirm_destroy: Вы уверены, что хотите удалить это задание?
+          confirm_discard: Вы уверены, что хотите отменить это задание?
           confirm_reschedule: Вы уверены, что хотите перенести это задание?
           confirm_retry: Вы уверены, что хотите повторить это задание?
-          destroy: Уничтожить работу
+          destroy: Удалить задание
           discard: Отменить задание
           reschedule: Перенести задание
           retry: Повторить задание
           title: Действия
-        no_jobs_found: Вакансий не найдено.
+        no_jobs_found: Заданий не найдено
       show:
         attributes: Атрибуты
-        batched_jobs: Пакетные задания
-        callback_jobs: Задания обратного вызова
+        batched_jobs: Группы заданий
+        callback_jobs: Коллбеки
       table:
-        no_batches_found: Партии не найдены.
+        no_batches_found: Группы заданий не найдены.
     cron_entries:
       actions:
-        confirm_disable: Вы уверены, что хотите отключить эту запись cron?
-        confirm_enable: Вы уверены, что хотите подтвердить эту запись cron?
-        confirm_enqueue: Вы уверены, что хотите поставить эту запись cron в очередь?
-        disable: Отключить запись cron
-        enable: Включить запись cron
-        enqueue: Поставить в очередь запись cron сейчас
+        confirm_disable: Вы уверены, что хотите отключить эту задачу cron?
+        confirm_enable: Вы уверены, что хотите подтвердить эту задачу cron?
+        confirm_enqueue: Вы уверены, что хотите поставить эту задачу cron в очередь?
+        disable: Отключить задачу cron
+        enable: Включить задачу cron
+        enqueue: Поставить в очередь задачу cron сейчас
       disable:
-        notice: Запись Cron отключена.
+        notice: Задача Cron отключена.
       enable:
-        notice: Запись Cron включена.
+        notice: Задача Cron включена.
       enqueue:
-        notice: Запись Cron поставлена в очередь.
+        notice: Задача Cron поставлена в очередь.
       index:
         no_cron_schedules_found: Расписания cron не найдены.
-        title: Расписания Cron
-      pending_migrations: Требуется отложенная миграция базы данных GoodJob.
+        title: Расписания cron
+      pending_migrations: У GoodJob есть ожидающие миграции базы данных.
       show:
-        cron_entry_key: Ключ входа в Крон
+        cron_entry_key: Ключ cron-задания
     datetime:
       distance_in_words:
         about_x_hours:
@@ -73,7 +73,7 @@ ru:
           many: почти %{count} лет
           one: почти 1 год
           other: почти %{count} лет
-        half_a_minute: полминуты
+        half_a_minute: 30 секунд
         less_than_x_minutes:
           few: меньше %{count} минут
           many: меньше %{count} минут
@@ -97,7 +97,7 @@ ru:
         x_minutes:
           few: "%{count} минуты"
           many: "%{count} минут"
-          one: 1 минуту
+          one: 1 минута
           other: "%{count} минуты"
         x_months:
           few: "%{count} месяца"
@@ -107,7 +107,7 @@ ru:
         x_seconds:
           few: "%{count} секунды"
           many: "%{count} секунд"
-          one: 1 секунду
+          one: 1 секунда
           other: "%{count} секунды"
         x_years:
           few: "%{count} года"
@@ -121,49 +121,47 @@ ru:
       minutes: "%{min}м %{sec}с"
       seconds: "%{sec}s"
     error_event:
-      discarded: Отброшено
+      discarded: Отклонено
       handled: Обработано
       interrupted: Прервано
       retried: Повторная попытка
-      retry_stopped: Повторить остановлено
-      unhandled: необработанный
+      retry_stopped: Повторная попытка отменена
+      unhandled: Не обработано
     helpers:
       relative_time:
         future: в %{time}
         past: "%{time} назад"
     jobs:
       actions:
-        confirm_destroy: Вы уверены, что хотите уничтожить задание?
-        confirm_discard: Вы уверены, что хотите отказаться от задания?
-        confirm_force_discard: 'Вы уверены, что хотите принудительно отменить это задание? Задание будет помечено как отброшенное, но выполняемое задание не будет остановлено, однако в случае сбоя оно не будет повторено.
-
-          '
+        confirm_destroy: Вы уверены, что хотите удалить это задание?
+        confirm_discard: Вы уверены, что хотите отменить это задание?
+        confirm_force_discard: Вы уверены, что хотите принудительно отменить это задание? Задание будет помечено как отмененное, но выполняемое в настоящий момент задание не будет остановлено. В случае сбоя отмененное задание не будет повторено.
         confirm_reschedule: Вы уверены, что хотите перенести задание?
         confirm_retry: Вы уверены, что хотите повторить задание?
-        destroy: Уничтожить работу
+        destroy: Удалить задание
         discard: Отменить задание
         force_discard: Принудительно отменить задание
         reschedule: Перенести задание
         retry: Повторить задание
       destroy:
-        notice: Работа была уничтожена
+        notice: Задание было удалено
       discard:
-        notice: Работа удалена
+        notice: Задание было отменено
       executions:
         in_queue: в очереди
         runtime: время выполнения
-        title: Казни
+        title: Выполнение заданий
       force_discard:
-        notice: Иов был принудительно отброшен. Он продолжит работу, но не будет повторяться в случае сбоя.
+        notice: Задание было принудительно отменено. Активное задание продолжит выполняться, но не будет повторно запущено в случае сбоя.
       index:
-        job_pagination: Пагинация вакансий
-        older_jobs: Старые рабочие места
+        job_pagination: Пагинация заданий
+        older_jobs: Старые задания
       reschedule:
-        notice: Работа была перенесена
+        notice: Время исполнения задания было перенесено
       retry:
         notice: Задание было повторено
       show:
-        jobs: Работа
+        jobs: Задания
       table:
         actions:
           apply_to_all:
@@ -171,40 +169,40 @@ ru:
             many: около %{count} часов
             one: около 1 часа
             other: около %{count} часа
-          confirm_destroy_all: Вы уверены, что хотите уничтожить выбранные задания?
-          confirm_discard_all: Вы действительно хотите отказаться от выбранных заданий?
-          confirm_reschedule_all: Вы уверены, что хотите изменить расписание выбранных заданий?
+          confirm_destroy_all: Вы уверены, что хотите удалить выбранные задания?
+          confirm_discard_all: Вы действительно хотите отменить выбранные задания?
+          confirm_reschedule_all: Вы уверены, что хотите изменить время исполнения выбранных заданий?
           confirm_retry_all: Вы уверены, что хотите повторить выбранные задания?
-          destroy_all: Уничтожить все
+          destroy_all: Удалить все
           discard_all: Отменить все
           reschedule_all: Перенести все
           retry_all: Повторить все
           title: Действия
-        no_jobs_found: Вакансий не найдено.
+        no_jobs_found: Заданий не найдено.
         toggle_actions: Переключить действия
         toggle_all_jobs: Переключить все задания
     models:
       batch:
         created: Созданный
         created_at: Создан в
-        discarded: Отброшено
-        discarded_at: Отброшено в
-        enqueued: В очереди
-        enqueued_at: В очереди в
-        finished: Законченный
-        finished_at: Закончено в
-        jobs: Работа
-        name: Имя
+        discarded: Отмененный
+        discarded_at: Отмененный в
+        enqueued: Запланированный
+        enqueued_at: Запланированный в
+        finished: Завершенный
+        finished_at: Завершенный в
+        jobs: Задание
+        name: Название
       cron:
-        class: Сорт
+        class: Класс
         last_run: Последний запуск
         next_scheduled: Следующий по расписанию
         schedule: Расписание
       job:
-        arguments: Аргументы
+        arguments: Параметры
         attempts: Попытки
-        priority: приоритет
-        queue: Очередь
+        priority: Приоритет
+        queue: Очередь исполнения
     number:
       format:
         delimiter: " "
@@ -224,7 +222,7 @@ ru:
             unit: ''
     processes:
       index:
-        cron_enabled: Крон включен
+        cron_enabled: Cron включен
         no_good_job_processes_found: Процессы GoodJob не найдены.
         process: Процесс
         schedulers: Планировщики
@@ -238,32 +236,32 @@ ru:
       error: Ошибка
       filter:
         all: Все
-        all_jobs: Все вакансии
+        all_jobs: Все задания
         all_queues: Все очереди
-        clear: Прозрачный
-        job_name: Название работы
-        placeholder: Поиск по классу, идентификатору задания, параметрам задания и тексту ошибки.
-        queue_name: Имя очереди
+        clear: Очистить
+        job_name: Название задания
+        placeholder: Поиск по классу, идентификатору задания, параметрам задания или тексту ошибки.
+        queue_name: Название очереди
         search: Поиск
       navbar:
-        batches: Batches
+        batches: Пакетные задания
         cron_schedules: Cron
-        jobs: Задачи
-        live_poll: Живой Опрос
+        jobs: Задания
+        live_poll: Обновления в реальном времени
         name: "GoodJob 👍"
         processes: Процессы
         theme:
           auto: Авто
           dark: Темный
-          light: Свет
-          theme: Тема
+          light: Светлая
+          theme: Тема офрмдления
       secondary_navbar:
-        inspiration: Запомни, ты делаешь Good Job тоже!
+        inspiration: Благодаря вам Good Job становится лучше! Спасибо!
         last_updated: Последнее обновление
     status:
-      discarded: Отброшено
+      discarded: Отменено
       queued: В очереди
       retried: Повторная попытка
-      running: Бег
-      scheduled: по расписанию
-      succeeded: удалось
+      running: Исполняется
+      scheduled: Запланировано
+      succeeded: Успешно

data/lib/generators/good_job/templates/install/migrations/create_good_jobs.rb.erb

@@ -80,6 +80,8 @@ class CreateGoodJobs < ActiveRecord::Migration<%= migration_version %>
     add_index :good_jobs, [:finished_at], where: "retried_good_job_id IS NULL AND finished_at IS NOT NULL", name: :index_good_jobs_jobs_on_finished_at
     add_index :good_jobs, [:priority, :created_at], order: { priority: "DESC NULLS LAST", created_at: :asc },
       where: "finished_at IS NULL", name: :index_good_jobs_jobs_on_priority_created_at_when_unfinished
+    add_index :good_jobs, [:priority, :created_at], order: { priority: "ASC NULLS LAST", created_at: :asc },
+      where: "finished_at IS NULL", name: :index_good_job_jobs_for_candidate_lookup
     add_index :good_jobs, [:batch_id], where: "batch_id IS NOT NULL"
     add_index :good_jobs, [:batch_callback_id], where: "batch_callback_id IS NOT NULL"
     add_index :good_jobs, :labels, using: :gin, where: "(labels IS NOT NULL)", name: :index_good_jobs_on_labels

data/lib/generators/good_job/templates/update/migrations/11_create_index_good_job_jobs_for_candidate_lookup.rb.erb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+class CreateIndexGoodJobJobsForCandidateLookup < ActiveRecord::Migration<%= migration_version %>
+  disable_ddl_transaction!
+
+  def change
+    reversible do |dir|
+      dir.up do
+        # Ensure this incremental update migration is idempotent
+        # with monolithic install migration.
+        return if connection.index_name_exists?(:good_jobs, :index_good_job_jobs_for_candidate_lookup)
+      end
+    end
+
+    add_index :good_jobs, [:priority, :created_at], order: { priority: "ASC NULLS LAST", created_at: :asc },
+      where: "finished_at IS NULL", name: :index_good_job_jobs_for_candidate_lookup,
+      algorithm: :concurrently
+  end
+end

data/lib/good_job/adapter.rb

@@ -49,7 +49,7 @@ module GoodJob
       active_jobs = Array(active_jobs)
       return 0 if active_jobs.empty?
 
-      Rails.application.reloader.wrap do
+      Rails.application.executor.wrap do
         current_time = Time.current
         executions = active_jobs.map do |active_job|
           GoodJob::Execution.build_for_enqueue(active_job).tap do |execution|
@@ -139,7 +139,7 @@ module GoodJob
       # job there to be enqueued using enqueue_all
       return if GoodJob::Bulk.capture(active_job, queue_adapter: self)
 
-      Rails.application.reloader.wrap do
+      Rails.application.executor.wrap do
         will_execute_inline = execute_inline? && (scheduled_at.nil? || scheduled_at <= Time.current)
         execution = GoodJob::Execution.enqueue(
           active_job,

data/lib/good_job/cli.rb

@@ -68,7 +68,7 @@ module GoodJob
     method_option :poll_interval,
                   type: :numeric,
                   banner: 'SECONDS',
-                  desc: "Interval between polls for available jobs in seconds (env var: GOOD_JOB_POLL_INTERVAL, default: 5)"
+                  desc: "Interval between polls for available jobs in seconds (env var: GOOD_JOB_POLL_INTERVAL, default: 10)"
     method_option :max_cache,
                   type: :numeric,
                   banner: 'COUNT',
@@ -94,6 +94,9 @@ module GoodJob
                   type: :numeric,
                   banner: 'PORT',
                   desc: "Port for http health check (env var: GOOD_JOB_PROBE_PORT, default: nil)"
+    method_option :probe_handler,
+                  type: :string,
+                  desc: "Use 'webrick' to use WEBrick to handle probe server requests which is Rack compliant, otherwise default server that is not Rack compliant is used. (env var: GOOD_JOB_PROBE_HANDLER, default: nil)"
     method_option :queue_select_limit,
                   type: :numeric,
                   banner: 'COUNT',
@@ -112,7 +115,7 @@ module GoodJob
       systemd.start
 
       if configuration.probe_port
-        probe_server = GoodJob::ProbeServer.new(port: configuration.probe_port)
+        probe_server = GoodJob::ProbeServer.new(app: configuration.probe_app, port: configuration.probe_port, handler: configuration.probe_handler)
         probe_server.start
       end
 

data/lib/good_job/configuration.rb

@@ -342,10 +342,26 @@ module GoodJob
     end
 
     # Port of the probe server
-    # @return [nil,Integer]
+    # @return [nil, Integer]
     def probe_port
-      options[:probe_port] ||
+      (options[:probe_port] ||
         env['GOOD_JOB_PROBE_PORT']
+      )&.to_i
+    end
+
+    # Probe server handler
+    # @return [nil, Symbol]
+    def probe_handler
+      (options[:probe_handler] ||
+        rails_config[:probe_handler] ||
+        env['GOOD_JOB_PROBE_HANDLER']
+      )&.to_sym
+    end
+
+    # Rack compliant application to be run on the ProbeServer
+    # @return [nil, Class]
+    def probe_app
+      rails_config[:probe_app]
     end
 
     def enable_listen_notify

data/lib/good_job/probe_server/healthcheck_middleware.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module GoodJob
+  class ProbeServer
+    class HealthcheckMiddleware
+      def initialize(app)
+        @app = app
+      end
+
+      def call(env)
+        case Rack::Request.new(env).path
+        when '/', '/status'
+          [200, {}, ["OK"]]
+        when '/status/started'
+          started = GoodJob::Scheduler.instances.any? && GoodJob::Scheduler.instances.all?(&:running?)
+          started ? [200, {}, ["Started"]] : [503, {}, ["Not started"]]
+        when '/status/connected'
+          connected = GoodJob::Scheduler.instances.any? && GoodJob::Scheduler.instances.all?(&:running?) &&
+                      GoodJob::Notifier.instances.any? && GoodJob::Notifier.instances.all?(&:connected?)
+          connected ? [200, {}, ["Connected"]] : [503, {}, ["Not connected"]]
+        else
+          @app.call(env)
+        end
+      end
+    end
+  end
+end

data/lib/good_job/probe_server/not_found_app.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module GoodJob
+  class ProbeServer
+    module NotFoundApp
+      def self.call(_env)
+        [404, {}, ["Not found"]]
+      end
+    end
+  end
+end

data/lib/good_job/probe_server/simple_handler.rb

@@ -0,0 +1,83 @@
+# frozen_string_literal: true
+
+module GoodJob
+  class ProbeServer
+    class SimpleHandler
+      SOCKET_READ_TIMEOUT = 5 # in seconds
+
+      def initialize(app, options = {})
+        @app    = app
+        @port   = options[:port]
+        @logger = options[:logger]
+
+        @running = Concurrent::AtomicBoolean.new(false)
+      end
+
+      def stop
+        @running.make_false
+        @server&.close
+      end
+
+      def running?
+        @running.true?
+      end
+
+      def build_future
+        Concurrent::Future.new { run }
+      end
+
+      private
+
+      def run
+        @running.make_true
+        start_server
+        handle_connections if @running.true?
+      rescue StandardError => e
+        @logger.error "Server encountered an error: #{e}"
+      ensure
+        stop
+      end
+
+      def start_server
+        @server = TCPServer.new('0.0.0.0', @port)
+      rescue StandardError => e
+        @logger.error "Failed to start server: #{e}"
+        @running.make_false
+      end
+
+      def handle_connections
+        while @running.true?
+          begin
+            ready_sockets, = IO.select([@server], nil, nil, SOCKET_READ_TIMEOUT)
+            next unless ready_sockets
+
+            client = @server.accept_nonblock
+            request = client.gets
+
+            if request
+              status, headers, body = @app.call(parse_request(request))
+              respond(client, status, headers, body)
+            end
+
+            client.close
+          rescue IO::WaitReadable, Errno::EINTR, Errno::EPIPE
+            retry
+          end
+        end
+      end
+
+      def parse_request(request)
+        method, full_path = request.split
+        path, query = full_path.split('?')
+        { 'REQUEST_METHOD' => method, 'PATH_INFO' => path, 'QUERY_STRING' => query || '' }
+      end
+
+      def respond(client, status, headers, body)
+        client.write "HTTP/1.1 #{status}\r\n"
+        headers.each { |key, value| client.write "#{key}: #{value}\r\n" }
+        client.write "\r\n"
+        body.each { |part| client.write part.to_s }
+      end
+    end
+  end
+end

data/lib/good_job/probe_server/webrick_handler.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module GoodJob
+  class ProbeServer
+    class WebrickHandler
+      def initialize(app, options = {})
+        @app    = app
+        @port   = options[:port]
+        @logger = options[:logger]
+        @handler = ::Rack::Handler.get('webrick')
+      end
+
+      def stop
+        @handler&.shutdown
+      end
+
+      def running?
+        @handler&.instance_variable_get(:@server)&.status == :Running
+      end
+
+      def build_future
+        Concurrent::Future.new(args: [@handler, @port, GoodJob.logger]) do |thr_handler, thr_port, thr_logger|
+          thr_handler.run(@app, Port: thr_port, Host: '0.0.0.0', Logger: thr_logger, AccessLog: [])
+        end
+      end
+    end
+  end
+end

data/lib/good_job/probe_server.rb

@@ -8,13 +8,20 @@ module GoodJob
       GoodJob._on_thread_error(thread_error) if thread_error
     end
 
-    def initialize(port:)
-      @port = port
+    def self.default_app
+      ::Rack::Builder.new do
+        use GoodJob::ProbeServer::HealthcheckMiddleware
+        run GoodJob::ProbeServer::NotFoundApp
+      end
+    end
+
+    def initialize(port:, handler: nil, app: nil)
+      app ||= self.class.default_app
+      @handler = build_handler(port: port, handler: handler, app: app)
     end
 
     def start
-      @handler = HttpServer.new(self, port: @port, logger: GoodJob.logger)
-      @future = Concurrent::Future.new { @handler.run }
+      @future = @handler.build_future
       @future.add_observer(self.class, :task_observer)
       @future.execute
     end
@@ -28,19 +35,17 @@ module GoodJob
       @future&.value # wait for Future to exit
     end
 
-    def call(env)
-      case Rack::Request.new(env).path
-      when '/', '/status'
-        [200, {}, ["OK"]]
-      when '/status/started'
-        started = GoodJob::Scheduler.instances.any? && GoodJob::Scheduler.instances.all?(&:running?)
-        started ? [200, {}, ["Started"]] : [503, {}, ["Not started"]]
-      when '/status/connected'
-        connected = GoodJob::Scheduler.instances.any? && GoodJob::Scheduler.instances.all?(&:running?) &&
-                    GoodJob::Notifier.instances.any? && GoodJob::Notifier.instances.all?(&:connected?)
-        connected ? [200, {}, ["Connected"]] : [503, {}, ["Not connected"]]
+    def build_handler(port:, handler:, app:)
+      if handler == :webrick
+        begin
+          require 'webrick'
+          WebrickHandler.new(app, port: port, logger: GoodJob.logger)
+        rescue LoadError
+          GoodJob.logger.warn("WEBrick was requested as the probe server handler, but it's not in the load path. GoodJob doesn't keep WEBrick as a dependency, so you'll have to make sure its added to your Gemfile to make use of it. GoodJob will fallback to its own webserver in the meantime.")
+          SimpleHandler.new(app, port: port, logger: GoodJob.logger)
+        end
       else
-        [404, {}, ["Not found"]]
+        SimpleHandler.new(app, port: port, logger: GoodJob.logger)
       end
     end
   end

data/lib/good_job/version.rb

@@ -2,7 +2,7 @@
 
 module GoodJob
   # GoodJob gem version.
-  VERSION = '3.22.0'
+  VERSION = '3.23.0'
 
   # GoodJob version as Gem::Version object
   GEM_VERSION = Gem::Version.new(VERSION)

data/lib/good_job.rb

@@ -32,8 +32,11 @@ require "good_job/log_subscriber"
 require "good_job/multi_scheduler"
 require "good_job/notifier"
 require "good_job/poller"
-require "good_job/http_server"
 require "good_job/probe_server"
+require "good_job/probe_server/healthcheck_middleware"
+require "good_job/probe_server/not_found_app"
+require "good_job/probe_server/simple_handler"
+require "good_job/probe_server/webrick_handler"
 require "good_job/scheduler"
 require "good_job/shared_executor"
 require "good_job/systemd_service"
@@ -247,7 +250,7 @@ module GoodJob
     loop do
       break if limit && iteration >= limit
 
-      result = Rails.application.reloader.wrap { job_performer.next }
+      result = Rails.application.executor.wrap { job_performer.next }
       break unless result
       raise result.unhandled_error if result.unhandled_error
 
@@ -273,7 +276,7 @@ module GoodJob
   def self.migrated?
     # Always update with the most recent migration check
     GoodJob::Execution.reset_column_information
-    GoodJob::Execution.active_job_id_index_removal_migrated?
+    GoodJob::Execution.candidate_lookup_index_migrated?
   end
 
   ActiveSupport.run_load_hooks(:good_job, self)