From bde50da4084d7d15516de03b542b20cbf3683fa0 Mon Sep 17 00:00:00 2001
From: Yuta Iwama <ganmacs@gmail.com>
Date: Mon, 3 Feb 2020 16:40:34 +0900
Subject: [PATCH] Extract as method

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
---
 lib/fluent/supervisor.rb | 81 +++++++++++++++++++++-------------------
 1 file changed, 43 insertions(+), 38 deletions(-)

diff --git a/lib/fluent/supervisor.rb b/lib/fluent/supervisor.rb
index 16b02aadbc..047a2c70ee 100644
--- a/lib/fluent/supervisor.rb
+++ b/lib/fluent/supervisor.rb
@@ -660,44 +660,7 @@ def supervise
       Process.setproctitle("supervisor:#{@system_config.process_name}") if @system_config.process_name
       $log.info "starting fluentd-#{Fluent::VERSION}", pid: Process.pid, ruby: RUBY_VERSION
 
-      rubyopt = ENV["RUBYOPT"]
-      fluentd_spawn_cmd = [ServerEngine.ruby_bin_path]
-
-      if rubyopt
-        encodes, others = rubyopt.split(' ').partition { |e| e.match?('\A-E') }
-        fluentd_spawn_cmd.concat(others)
-
-        adopted_encode =
-          if encodes.size >= 1
-            r = encodes.pop
-            unless encodes.empty?
-              drop = encodes.join(', ')
-              $log.warn("Encoding option #{drop} is(are) ignored")
-            end
-
-            r
-          else
-            '-Eascii-8bit:ascii-8bit'
-          end
-
-        fluentd_spawn_cmd << adopted_encode
-      else
-        fluentd_spawn_cmd << '-Eascii-8bit:ascii-8bit'
-      end
-
-      # Adding `-h` so that it can avoid ruby's command blocking
-      # e.g. `ruby -Eascii-8bit:ascii-8bit` will block. but `ruby -Eascii-8bit:ascii-8bit -h` won't.
-      cmd = fluentd_spawn_cmd.join(' ')
-      _, e, s = Open3.capture3("#{cmd} -h")
-      if s.exitstatus != 0
-        $log.error('Invalid option is passed to RUBYOPT', command: cmd, error: e)
-        exit s.exitstatus
-      end
-
-      fluentd_spawn_cmd << $0
-      fluentd_spawn_cmd += $fluentdargv
-      fluentd_spawn_cmd << "--under-supervisor"
-
+      fluentd_spawn_cmd = build_spawn_command
       $log.info "spawn command to main: ", cmdline: fluentd_spawn_cmd
 
       params = {
@@ -904,5 +867,47 @@ def build_system_config(conf)
       system_config.overwrite_variables(**opt)
       system_config
     end
+
+    def build_spawn_command
+      fluentd_spawn_cmd = [ServerEngine.ruby_bin_path]
+
+      rubyopt = ENV['RUBYOPT']
+      if rubyopt
+        encodes, others = rubyopt.split(' ').partition { |e| e.match?('\A-E') }
+        fluentd_spawn_cmd.concat(others)
+
+        adopted_encode =
+          if encodes.size >= 1
+            r = encodes.pop
+            unless encodes.empty?
+              drop = encodes.join(', ')
+              $log.warn("Encoding option #{drop} is(are) ignored")
+            end
+
+            r
+          else
+            '-Eascii-8bit:ascii-8bit'
+          end
+
+        fluentd_spawn_cmd << adopted_encode
+      else
+        fluentd_spawn_cmd << '-Eascii-8bit:ascii-8bit'
+      end
+
+      # Adding `-h` so that it can avoid ruby's command blocking
+      # e.g. `ruby -Eascii-8bit:ascii-8bit` will block. but `ruby -Eascii-8bit:ascii-8bit -h` won't.
+      cmd = fluentd_spawn_cmd.join(' ')
+      _, e, s = Open3.capture3("#{cmd} -h")
+      if s.exitstatus != 0
+        $log.error('Invalid option is passed to RUBYOPT', command: cmd, error: e)
+        exit s.exitstatus
+      end
+
+      fluentd_spawn_cmd << $0
+      fluentd_spawn_cmd += $fluentdargv
+      fluentd_spawn_cmd << '--under-supervisor'
+
+      fluentd_spawn_cmd
+    end
   end
 end