Issue #4763 has been updated by Matt Robinson.

Arnaud, are you sure you tested it with activerecord 3 still installed?  When I 
try James' patch allowing ActiveRecord 3.0 I run into a few problems.  The 
first is:

err: uninitialized constant Puppet::Rails::Logger -- lib/puppet/rails.rb:16:in 
`connect'

This is easily fixed by doing a require 'logger' at the beginning of the file.  
However once I get past that I run into another error.

err: undefined method `allow_concurrency=' for ActiveRecord::Base:Class -- 
lib/puppet/rails.rb:27:in `connect'

allow_concurrency is deprecated and has been removed in 3.0.  In fact if you 
google for it the first thing that comes up is a puppet patch from years ago, 
ticket 1805.  
http://groups.google.com/group/puppet-dev/browse_thread/thread/35a57560b509885. 
 DEPRECATION WARNING: ActiveRecord::Base.allow_concurrency= has been deprecated 
and no longer has any effect. Please remove all references to 
allow_concurrency=.. (called from allow_concurrency= at 
/usr/lib/ruby/gems/1.8/gems/activerecord-2.2.2/lib/active_record/connection_adapters/abstract/connection_specification.rb:96)

Unfortunately it appears we support activerecord 2.1 where allow_concurrency 
still does something, so we can't get rid of the line entirely.  If this is the 
case we need to bound on active record version a little tighter when we set 
allow_concurrency.

Once I made those changes I was able to run storeconfigs successfully.  I'll 
create a patch to send to the list.
----------------------------------------
Bug #4763: ActiveRecord detection fails
http://projects.puppetlabs.com/issues/4763

Author: arnaud chong
Status: Ready for Testing
Priority: Normal
Assignee: James Turnbull
Category: stored configuration
Target version: 2.6.2
Affected version: 2.6.1
Keywords: 
Branch: http://github.com/jamtur01/puppet/tree/tickets/2.6.x/4763


> Could not parse configuration file: StoreConfigs not supported without 
> ActiveRecord 2.1 or higher

I have active_record 3.0.0 installed. I guess it's because of 
::ActiveRecord::VERSION::MAJOR == 2 in lib/puppet/feature/rails.rb



-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to