From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS14383 205.234.109.0/24 X-Spam-Status: No, score=-0.5 required=5.0 tests=AWL,MSGID_FROM_MTA_HEADER, RP_MATCHES_RCVD shortcircuit=no autolearn=unavailable version=3.3.2 Path: news.gmane.org!not-for-mail From: Eric Wong Newsgroups: gmane.comp.lang.ruby.unicorn.general Subject: Re: Pidfiles and cwd? Date: Sat, 5 Sep 2009 18:38:49 -0700 Message-ID: <20090906013849.GC28829@dcvr.yhbt.net> References: <8b73aaca0909042020w73fb03dfpf6c77c85c1c486ad@mail.gmail.com> <8b73aaca0909042021v25d5f0eajd926250f71623042@mail.gmail.com> <20090905042806.GA9507@dcvr.yhbt.net> <8b73aaca0909051750h1520fa88ub42bcd4cd3944dbc@mail.gmail.com> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1252201141 23305 80.91.229.12 (6 Sep 2009 01:39:01 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 6 Sep 2009 01:39:01 +0000 (UTC) Cc: mongrel-unicorn@rubyforge.org To: Chris Wanstrath Original-X-From: mongrel-unicorn-bounces@rubyforge.org Sun Sep 06 03:38:54 2009 Return-path: Envelope-to: gclrug-mongrel-unicorn@m.gmane.org X-Original-To: mongrel-unicorn@rubyforge.org Delivered-To: mongrel-unicorn@rubyforge.org Content-Disposition: inline In-Reply-To: <8b73aaca0909051750h1520fa88ub42bcd4cd3944dbc@mail.gmail.com> User-Agent: Mutt/1.5.18 (2008-05-17) X-BeenThere: mongrel-unicorn@rubyforge.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: mongrel-unicorn-bounces@rubyforge.org Errors-To: mongrel-unicorn-bounces@rubyforge.org Xref: news.gmane.org gmane.comp.lang.ruby.unicorn.general:19 Archived-At: Received: from rubyforge.org ([205.234.109.19]) by lo.gmane.org with esmtp (Exim 4.50) id 1Mk6iU-0001l4-6e for gclrug-mongrel-unicorn@m.gmane.org; Sun, 06 Sep 2009 03:38:54 +0200 Received: from rubyforge.org (rubyforge.org [127.0.0.1]) by rubyforge.org (Postfix) with ESMTP id 963D118580E0; Sat, 5 Sep 2009 21:38:53 -0400 (EDT) Received: from dcvr.yhbt.net (dcvr.yhbt.net [64.71.152.64]) by rubyforge.org (Postfix) with ESMTP id 5B8D118580E0 for ; Sat, 5 Sep 2009 21:38:50 -0400 (EDT) Received: from localhost (unknown [127.0.2.5]) by dcvr.yhbt.net (Postfix) with ESMTP id BD7741F794; Sun, 6 Sep 2009 01:38:49 +0000 (UTC) Chris Wanstrath wrote: > On Fri, Sep 4, 2009 at 9:28 PM, Eric Wong wrote: > = > >> A) Is there a reason `unicorn` allows you to specify the pidfile's > >> location but `unicorn_rails` does not? > > > > `unicorn` was designed to mimic `rackup` in terms of command line > > options and `unicorn_rails` was designed to mimic `script/server` in > > Rails. > > > > I really didn't know what I was doing with the command-line options for > > this, so I decided to steal from others :) > > > > For long-running servers, I'm not a fan of command-line options in > > general because they're easy to forget, so `unicorn` only supports it > > because `rackup` does it (so the embedded CLI options in config.ru can > > be shared). > > > > For `unicorn_rails`, --daemonize already sets a default PID path in > > RAILS_ROOT/tmp/pids/unicorn.pid whereas `script/server` chooses > > RAILS_ROOT/tmp/pids/server.pid. =A0Since Rails values "convention over > > configuration", =A0I figured I might as well hard code it... > > > > Additionally, the "-P" parameter used by unicorn_rails and script/server > > is used to set RAILS_RELATIVE_URL_ROOT so it conflicts with the short > > option used by rackup/unicorn. > = > I'm 100% on board with your reasoning here, but I don't understand why > we can't set the pid in the config file. Am I reading this wrong? > (Very possibly.) You can definitely set the pid in the config file, let me know if you're having issues... > Doesn't this line mean that any `pid` setting in the configurator is igno= red: > = > http://git.bogomips.org/cgit/unicorn.git/tree/lib/unicorn.rb#n83 It's only ignored at that time, the pid is dropped later in the start method: http://git.bogomips.org/cgit/unicorn.git/tree/lib/unicorn.rb#n114 self.pid =3D config[:pid] So I don't drop the pid file until listeners are bound. I can't remember why I drop the pid late, however :) I know I don't bind the listeners right away because I try inheriting them first. It might be to avoid issues with the pid clobbering in the parent, or that some health checkers rely on both the pid and listen socket or something... > If so, how would you suggest setting the pid at runtime? I don't mean > to be difficult, we just want to keep everything in /var/run The config file really should work, but be sure to let me know if it's still not working for you. I just tested unicorn_rails with a test app and it seems to work here. Perhaps you're hitting permissions problems? As far as permissions, I'm considering adding user switching, but I'm afraid it could lead to more problems/support issues since I don't know anybody who runs Mongrel/Thin as root. However I've already written a huge comment/example in the Configurator RDoc for it :) > >> B) Is there a reason `unicorn_rails` must start in the app root and > >> doesn't allow it as a config option? > > > > Since the config file is just Ruby, you can just Dir.chdir inside it. > > And since the chdir is done when the config file is evaluated, the > > chdir can be done across restarts/reloads (you can point it to a > > symlinked directory) to pick up new code/releases. > > > > If you do that, I would initially start Unicorn in "/" or some other > > directory that won't get deleted so you'll be safe for upgrades. > > > > If you managed to forget that, you can set the following in your > > Unicorn config: > > > > =A0Unicorn::HttpServer::START_CTX[:cwd] =3D "/" > > > > And then HUP the process before doing the USR2+QUIT to reexec. > > Subsequent Unicorns will always start in "/" and then you can > > Dir.chdir to wherever you run your app. > > > > Hopefully that makes sense, one thing I've been trying to avoid with the > > configuration is having too many ways to do the same thing. > = > Sounds good to me. Unicorn has quite completely convinced me that > pure-Ruby config files are the way to go. I've already done some crazy > stuff in there during our migration that a YAML file (or whatever) > just wouldn't have been able to handle. Cool, thanks for the feedback. I've had lots of issues dealing with ugly/limited config files and seen too many ways of templating them to count. The after_fork, before_fork, and before_exec hooks sealed the deal for Ruby and Ezra / Rack::Builder helped push me in the right direction with the DSL. It was originally "write a small ruby script to run this thing" :) -- = Eric Wong