[Pkg-puppet-devel] Bug#774643: verify_active_connections is not present in ruby-activerecord 4.1.8

[bertagaz]

Hi,

> > => the best way to help get this fixed is probably to test the
> > proposed patches and to report back here :)

> By the way, we're running a production puppetmaster with the patches and 
> everything (db creation, stored configs and exported resource 
> collection) works fine. I'll commit the patch to the repository.

I've just build the puppet* packages from Jessie with the last patch from this
thread, then installed it as well as the ruby dependencies on our (Jessie)
puppetmaster.

So far it seems to work correctly (which was not the case with the first
proposed patch). The exported resources seem to be collected and deployed, e.g
the puppet-sshkeys module which uses storeconfig works now and correctly deploy
its fancy stuffs.

So I think we're in the good direction to solve this issue. Still I'd prefer to
wait for Intrigeri to review my tests before adopting this patch. If any other
can test it, please do so.

> By the way, I wonder if puppetmaster-common should Recommend (or at 
> least Suggest) ruby-activerecord and 
> ruby-activerecord-deprecated-finders. If not, we should at least add a 
> notice in README.Debian about what is needed to enable stored configs.

Yes, I believe it should Suggest them, and *also* deserve some explanations in
the README. There's no dependencies between
ruby-activerecord-deprecated-finders and ruby-activerecord, so we should make
it clear one has to install the former for the storeconfig option to work.

Also I'm not sure the patch should remove the deprecation warnings.

Thanks for your work on that!

bert.