Passing additional options to Thin

[mf...@redhat.com]

Hi,

This small patch will add possibility to pass additional options to Thin web server.
If you want let say set PID file or spawn multiple thin server you can use this syntax:

deltacloudd -i mock --thin "--pid file.pid --server 10"

To obtain full list of thin options please use 'thin --help'.
Please keep in mind that these options need to be enquoted.


  -- Michal

Re: [PATCH core] Added possibility to pass additional Thin options using --thin parameter

[David Lutterkort <lu...@redhat.com>]

On Fri, 2011-04-08 at 14:10 +0200, mfojtik@redhat.com wrote:
> From: Michal Fojtik <mf...@redhat.com>
> 
> ---
>  server/bin/deltacloudd |    8 ++++++--
>  1 files changed, 6 insertions(+), 2 deletions(-)

ACK

[PATCH core] Added possibility to pass additional Thin options using --thin parameter

[mf...@redhat.com]

From: Michal Fojtik <mf...@redhat.com>

---
 server/bin/deltacloudd |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/server/bin/deltacloudd b/server/bin/deltacloudd
index addb1f5..8c15d1c 100755
--- a/server/bin/deltacloudd
+++ b/server/bin/deltacloudd
@@ -40,6 +40,7 @@ BANNER
     ENV['API_PROVIDER'] = provider
   end
   opts.on( '-e', '--env ENV', 'Environment (default: "development")') { |env| options[:env] = env }
+  opts.on( '-t', '--thin OPTIONS', 'Additional thin options ("--ssl --pid FILENAME")') { |opt| options[:thin] = opt}
   opts.on( '-h', '--help', '') { options[:help] = true }
 end
 
@@ -106,10 +107,13 @@ else
   argv_opts << ['--chdir', dirname ]
   argv_opts << ['-e', options[:env] ]
   argv_opts << ['--timeout', '60']
-  argv_opts << ['--threaded', '-D', '--stats', '/stats']
-
+  argv_opts << ['--threaded', '-D']
   argv_opts.flatten!
 
+  if options[:thin]
+    argv_opts << "#{options[:thin]}"
+  end
+
   if have_rerun && options[:env] == "development"
     argv_opts.unshift "thin"
     command = argv_opts.join(" ")
-- 
1.7.4.1