puma 2.8.0-java → 2.8.1-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 604797b820e7795890f76c88044863ef3a43d0e8
-  data.tar.gz: 393dcdd1212faebe9ec608666cb6050efe2f596b
+  metadata.gz: 4a4ca5639981db20a9ea606243a312d164f63784
+  data.tar.gz: c0a3a97591d9e3d52ebab9e91541437c7a739a86
 SHA512:
-  metadata.gz: 680abe37fe6c5c7052d67306501c8a2f0a0a0c35716d2a5efb181f09ed3b1c0793e5f37344fb3236a2334171814ce7acb2421526fd9d93778a71d4e800475e24
-  data.tar.gz: 413b9efbec3292e0c7d87933bb0bd568c8f3afea0e8207e3e92d14d6e785de2ef1f6565d659fd282c32eef60319a3e9b3b33c9c7b19ade607b0ccd86a97b6852
+  metadata.gz: 940eb6f43f830a7c6247d7a0c14b5e30839365b884c78356a8f2c42f49db8e572f59a78ec4845ec4fc34188eee775e3cd3b87c566f44c6bf3f2cb4ed6ff2981f
+  data.tar.gz: bf9ec90368f7b010753c05f665cbbd3b6afa6fbac8cc4430babcba247c5d64de5646840a483648db42b095fbc66af66f20778b18cfaf1b77365ee460ec7bca5f

data/DEPLOYMENT.md

@@ -0,0 +1,92 @@
+# Deployment engineering for puma
+
+Puma is software that is expected to be run in a deployed environment eventually.
+You can centainly use it as your dev server only, but most people look to use
+it in their production deployments as well.
+
+To that end, this is meant to serve as a foundation of wisdom how to do that
+in a way that increases happiness and decreases downtime.
+
+## Specifying puma
+
+Most people want to do this by putting `gem "puma"` into their Gemfile, so we'll
+go ahead and assume that. Go add it now... we'll wait.
+
+
+Welcome back!
+
+## Single vs Cluster mode
+
+Puma was originally concieved as a thread-only webserver, but grew the ability to
+also use processes in version 2.
+
+Here are some rules of thumb:
+
+### MRI
+
+* Use cluster mode and set the number of workers to 1.5x the number of cpu cores
+  in the machine, minimum 2.
+* Set the number of threads to desired concurrent requests / number of workers.
+  Puma defaults to 8 and thats a decent number.
+
+#### Migrating from Unicorn
+
+* If you're migrating from unicorn though, here are some settings to start with:
+  * Set workers to half the number of unicorn workers you're using
+  * Set threads to 2
+  * Enjoy 50% memory savings
+* As you grow more confident in the thread safety of your app, you can tune the
+  workers down and the threads up.
+
+#### Worker utilization
+
+**How do you know if you're got enough (or too many workers)?**
+
+A good question. Due to MRI's GIL, only one thread can be executing at a time.
+But since so many apps are waiting on IO from DBs, etc., they can utilize threads
+to make better use of the process.
+
+The rule of thumb is you never want processes that are pegged all the time. This
+means that there is more work to do that the process can get through. On the other
+hand, if you have processes that sit around doing nothing, then they're just eating
+up resources.
+
+Watching your CPU utilization over time and aim for about 70% on average. This means
+you've got capacity still but aren't starving threads.
+
+## Daemonizing
+
+I prefer to not daemonize my servers and use something like `runit` or `upstrart` to
+monitor them as child processes. This gives them fast response to crashes and
+makes it easy to figure out what is going on. Additionally, unlike `unicorn`,
+puma does not require daemonization to do zero-downtime restarts.
+
+I see people using daemonization because they start puma directly via capistrano
+task and thus want it to live on past the `cap deploy`. To this people I said:
+You need to be using a process monitor. Nothing is making sure puma stays up in
+this scenario! You're just waiting for something weird to happen, puma to die,
+and to get paged at 3am. Do yourself a favor, at least the process monitoring
+your OS comes with, be it `sysvinit`, `upstart`, or `systemd`. Or branch out
+and use `runit` or hell, even `monit`.
+
+## Restarting
+
+You probably will want to deploy some new code at some point, and you'd like
+puma to start running that new code. Minimizing the amount of time the server
+is unavailable would be nice as well. Here's how to do it:
+
+1. Don't use `preload!`. This dirties the master process and means it will have
+to shutdown all the workers and re-exec itself to get your new code, which means
+much higher waiting around for things to load.
+
+1. Use `prune_bundler`. This makes it so that the cluster master will detach itself
+from a Bundler context on start. This allows the cluster workers to load your app
+and start a brand new Bundler context within the worker only. This means your
+master remains pristine and can live on between new releases of your code.
+
+1. Use phased-restart (`SIGUSR1` or `pumactl phased-restart`). This tells the master
+to kill off one worker at a time and restart them in your new code. This minimizes
+downtime and staggers the restart nicely. **WARNING** This means that both your
+old code and your new code will be running concurrently. Most deployment solutions
+already cause that, but it's worth warning you about it again. Be careful with your
+migrations, etc!

data/History.txt

@@ -1,3 +1,19 @@
+=== 2.8.1 / 2014-03-06
+
+* 1 bug fixes:
+  * Run puma-wild with proper deps for prune_bundler
+
+* 2 doc changes:
+  * Described the configuration file finding behavior added in 2.8.0 and how to disable it.
+  * Start the deployment doc
+
+* 6 PRs merged:
+  * Merge pull request #471 from arthurnn/fix_test
+  * Merge pull request #485 from joneslee85/patch-9
+  * Merge pull request #486 from joshwlewis/patch-1
+  * Merge pull request #490 from tobinibot/patch-1
+  * Merge pull request #491 from brianknight10/clarify-no-config
+
 === 2.8.0 / 2014-02-28
 
 * 8 minor features:

data/Manifest.txt

@@ -1,4 +1,5 @@
 COPYING
+DEPLOYMENT.md
 Gemfile
 History.txt
 LICENSE
@@ -6,6 +7,7 @@ Manifest.txt
 README.md
 Rakefile
 bin/puma
+bin/puma-wild
 bin/pumactl
 docs/config.md
 docs/nginx.md

data/README.md

@@ -149,6 +149,12 @@ You can also provide a configuration file which Puma will use with the `-C` (or
 
     $ puma -C /path/to/config
 
+By default, if no configuration file is specifed, Puma will look for a configuration file at config/puma.rb. If an environment is specified, either via the `-e` and `--environment` flags, or through the `RACK_ENV` environment variable, the default file location will be config/puma/environment_name.rb.
+
+If you want to prevent Puma from looking for a configuration file in those locations, provide a dash as the argument to the `-C` (or `--config`) flag:
+
+    $ puma -C "-"
+
 Take the following [sample configuration](https://github.com/puma/puma/blob/master/examples/config.rb) as inspiration or check out [configuration.rb](https://github.com/puma/puma/blob/master/lib/puma/configuration.rb) to see all available options.
 
 ## Restart

data/bin/puma-wild

@@ -0,0 +1,17 @@
+#!/usr/bin/env ruby
+#
+# Copyright (c) 2014 Evan Phoenix
+#
+
+require 'rubygems'
+
+deps = ARGV.shift.split(",").each do |s|
+  name, ver = s.split(":",2)
+  gem name, ver
+end
+
+require 'puma/cli'
+
+cli = Puma::CLI.new ARGV
+
+cli.run

data/lib/puma/cli.rb

@@ -456,14 +456,27 @@ module Puma
       end
 
       if prune_bundler? && defined?(Bundler)
-        puma_lib_dir = $:.detect { |d| File.exist? File.join(d, "puma", "const.rb") }
+        puma = Bundler.rubygems.loaded_specs("puma")
+
+        dirs = puma.require_paths.map { |x| File.join(puma.full_gem_path, x) }
+
+        puma_lib_dir = dirs.detect { |x| File.exist? File.join(x, "../bin/puma-wild") }
+
+        deps = puma.runtime_dependencies.map { |d|
+          spec = Bundler.rubygems.loaded_specs(d.name)
+          "#{d.name}:#{spec.version.to_s}"
+        }.join(",")
 
         if puma_lib_dir
           log "* Pruning Bundler environment"
           Bundler.with_clean_env do
-            Kernel.exec(Gem.ruby, "-I", puma_lib_dir,
-                        File.expand_path(puma_lib_dir + "/../bin/puma"),
-                        *@original_argv)
+
+            wild = File.expand_path(File.join(puma_lib_dir, "../bin/puma-wild"))
+
+            args = [Gem.ruby] + dirs.map { |x| ["-I", x] }.flatten +
+                   [wild, deps] + @original_argv
+
+            Kernel.exec(*args)
           end
         end
 

data/lib/puma/const.rb

@@ -28,8 +28,8 @@ module Puma
   # too taxing on performance.
   module Const
 
-    PUMA_VERSION = VERSION = "2.8.0".freeze
-    CODE_NAME = "Sir Edmund Percival Hillary"
+    PUMA_VERSION = VERSION = "2.8.1".freeze
+    CODE_NAME = "Sir Edmund Percival Hillary".freeze
 
     FAST_TRACK_KA_TIMEOUT = 0.2
 
@@ -58,7 +58,7 @@ module Puma
     PUMA_TMP_BASE = "puma".freeze
 
     # Indicate that we couldn't parse the request
-    ERROR_400_RESPONSE = "HTTP/1.1 400 Bad Request\r\n\r\n"
+    ERROR_400_RESPONSE = "HTTP/1.1 400 Bad Request\r\n\r\n".freeze
 
     # The standard empty 404 response for bad requests.  Use Error4040Handler for custom stuff.
     ERROR_404_RESPONSE = "HTTP/1.1 404 Not Found\r\nConnection: close\r\nServer: Puma #{PUMA_VERSION}\r\n\r\nNOT FOUND".freeze
@@ -69,7 +69,7 @@ module Puma
     CONTENT_LENGTH = "CONTENT_LENGTH".freeze
 
     # Indicate that there was an internal error, obviously.
-    ERROR_500_RESPONSE = "HTTP/1.1 500 Internal Server Error\r\n\r\n"
+    ERROR_500_RESPONSE = "HTTP/1.1 500 Internal Server Error\r\n\r\n".freeze
 
     # A common header for indicating the server is too busy.  Not used yet.
     ERROR_503_RESPONSE = "HTTP/1.1 503 Service Unavailable\r\n\r\nBUSY".freeze

data/test/test_http11.rb

@@ -1,7 +1,7 @@
 # Copyright (c) 2011 Evan Phoenix
-# Copyright (c) 2005 Zed A. Shaw 
+# Copyright (c) 2005 Zed A. Shaw
 
-require 'test/testhelp'
+require 'testhelp'
 
 include Puma
 
@@ -24,11 +24,11 @@ class Http11ParserTest < Test::Unit::TestCase
     assert_equal 'GET', req['REQUEST_METHOD']
     assert_nil req['FRAGMENT']
     assert_nil req['QUERY_STRING']
-    
+
     parser.reset
     assert parser.nread == 0, "Number read after reset should be 0"
   end
- 
+
   def test_parse_dumbfuck_headers
     parser = HttpParser.new
     req = {}
@@ -38,7 +38,7 @@ class Http11ParserTest < Test::Unit::TestCase
     assert parser.finished?
     assert !parser.error?
   end
-  
+
   def test_parse_error
     parser = HttpParser.new
     req = {}
@@ -72,7 +72,7 @@ class Http11ParserTest < Test::Unit::TestCase
   def rand_data(min, max, readable=true)
     count = min + ((rand(max)+1) *10).to_i
     res = count.to_s + "/"
-    
+
     if readable
       res << Digest::SHA1.hexdigest(rand(count * 100).to_s) * (count / 40)
     else
@@ -142,4 +142,3 @@ class Http11ParserTest < Test::Unit::TestCase
 
   end
 end
-

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puma
 version: !ruby/object:Gem::Version
-  version: 2.8.0
+  version: 2.8.1
 platform: java
 authors:
 - Evan Phoenix
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2014-02-28 00:00:00.000000000 Z
+date: 2014-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -80,6 +80,7 @@ executables:
 - pumactl
 extensions: []
 extra_rdoc_files:
+- DEPLOYMENT.md
 - History.txt
 - Manifest.txt
 - README.md
@@ -91,6 +92,7 @@ extra_rdoc_files:
 - tools/jungle/upstart/README.md
 files:
 - COPYING
+- DEPLOYMENT.md
 - Gemfile
 - History.txt
 - LICENSE
@@ -98,6 +100,7 @@ files:
 - README.md
 - Rakefile
 - bin/puma
+- bin/puma-wild
 - bin/pumactl
 - docs/config.md
 - docs/nginx.md