On Tue, Jul 17, 2012 at 12:17:26PM -0300, Nicolás Sanguinetti wrote:
> On Tue, Jul 17, 2012 at 9:59 AM, David Rubin <[email protected]> wrote:
> > HI Guys,
> >
> > I recently tried VERY hard to override the database configuration bassed on
> > ENV vars or actual API calls. This task is almost next to impossible.
> >
> > I know that the database.yml file is parsed as ERB but that is sane for
> > simple ENV replacement if you want to actually change which database adaptor
> > it is (requires extra config/params) it becomes messy.
> >
> > I found https://github.com/rails/rails/issues/5297  which explains the
> > issue/solution far better then I could.
> >
> > Active record supports this natively, I can't see why rails would force the
> > requirement of a database.yml and for it to have a "production" section. see
> > http://apidock.com/rails/ActiveRecord/Base/establish_connection/class URL
> > section.
> 
> As I mentioned on the ticket, I would rather have ENV["DATABASE_URL"]
> supported as well, but I can't see how it's "VERY hard to override".
> See https://github.com/rails/rails/issues/5297#issuecomment-7037942
> 
> I've been using variations of that for a long time, and it works pretty well.

3.2+ should have support for a DATABASE_URL environment variable:

  
https://github.com/rails/rails/blob/master/activerecord/lib/active_record/connection_handling.rb#L39

Which you should be able to use like this:

  $ DATABASE_URL='sqlite3:///db/development.sqlite3' rake do:whatever

But that doesn't help the problem in #5297.  The problem in #5297 is
that Heroku calculates it's own parameters to pass in for the URL.
AFAIK, there is no way to augment the connection parameters that Heroku
sends.  But this seems like an issue with Heroku rather than Rails.

If you find entry points where the environment variable *isn't* used,
then we should consider it a bug.  For example, I just discovered that
`rake db:schema:dump` will not use the environment variable:

  
https://github.com/rails/rails/blob/master/activerecord/lib/active_record/railties/databases.rake#L229

So someone should send a patch for that. :-)

-- 
Aaron Patterson
http://tenderlovemaking.com/

Attachment: pgpXAhTQ2cH1A.pgp
Description: PGP signature

Reply via email to