example42 / puppi

Puppet module to manage applications deployments and servers local management
http://www.example42.com
Other
142 stars 84 forks source link

`puppi` 2.2.9 breaks catalogues with `instance?' for String:Class error #155

Open ElvenSpellmaker opened 5 years ago

ElvenSpellmaker commented 5 years ago

Expected Behavior

Catalogues to compile

Actual Behavior

Info: Loading facts
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Evaluation Error: Error while evaluating a Function Call, undefined method `instance?' for String:Class at /etc/puppetlabs/code/modules/yum/manifests/init.pp:167:27 at /etc/puppetlabs/code/modules/yum/manifests/repo/epel.pp:64 on node foo.com

Steps to Reproduce the Problem

  1. Use an example42 module without pinning puppi in your main Puppetfile to get the latest 2.2.9 version of puppi.

Specifications

Please add this info:

  1. Output of facter -p on the failing node (at least the OS related facts): N/A (all CentOS nodes we have fail)
  2. Version of Puppet and of the module
    • Puppet: 4.10.0
    • puppi: 2.2.9
  3. The relevant Puppet code and eventually Hiera data: example42/yum in init.pp, when calling params_lookup.

This is what changed when I pinned back to 2.2.6: image

hawkeye-7 commented 5 years ago

I have the same problem.

alvagante commented 5 years ago

2.2.7 introduces the change, it would be very useful the log of the puppetserver when failing to compile.

alvagante commented 5 years ago

I was able to reproduce, for the moment I reverted that change, introduced to remove some warnings.