m4ce / puppet-kapacitor

Puppet type and providers for managing Kapacitor
4 stars 3 forks source link

expects a value for parameter 'x' #1

Open druchoo opened 7 years ago

druchoo commented 7 years ago

OS: CentOS Linux release 7.2.1511 (Core) Puppet: 4.8.1 Facter: 3.5.0 Ruby: 2.0.0 Hiera: 3.2.2 m4ce-kapacitor: 0.1.0 kapacitor-ruby : 0.0.13

Are defaults not set or am I doing something wrong here?

manifest:

include 'kapacitor'
$ puppet agent -t
Notice: Local environment: 'production' doesn't match server specified node environment 'feature_influxdb', switching agent to 'feature_influxdb'.
Info: Retrieving pluginfacts
Info: Retrieving plugin
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, Class[Kapacitor]:
  expects a value for parameter 'templates'
  expects a value for parameter 'tasks'
  expects a value for parameter 'data_dir'
  expects a value for parameter 'opts'
  expects a value for parameter 'config_dir'
  expects a value for parameter 'config_file'
  expects a value for parameter 'config_file_manage'
  expects a value for parameter 'packages'
  expects a value for parameter 'gem_dependencies'
  expects a value for parameter 'service_provider'
  expects a value for parameter 'service_opts'
  expects a value for parameter 'service_image'
  expects a value for parameter 'service_name'
  expects a value for parameter 'service_manage'
  expects a value for parameter 'service_ensure'
  expects a value for parameter 'service_enable' at /etc/puppetlabs/code/environments/feature_influxdb/site/profiles/manifests/influxdb.pp:52:3 on node influxdb1
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run
m4ce commented 7 years ago

I migrated the module to Hiera 5 - You need Puppet >= 4.10.0. Else, use 0.0.16.

druchoo commented 7 years ago

Ok I see. Note that everything works if i set all those parameters manually. Not sure when we'll upgrade to Puppet >= 4.10.0 so i'll just set parameters for the time being. Hopefully that will work going forward so I take advantage of any future update.

Thanks for the module!

druchoo commented 7 years ago

@m4ce, would it be possible to support older versions of Hiera?

m4ce commented 7 years ago

@druchoo, use 0.0.16, that will do. There only change between 0.0.16 and 0.1.0 is support for Hiera 5.

druchoo commented 7 years ago

Yes understood about versions 0.0.16 and 0.1.0 and sorry I didn't state my question clearly...

For future versions, would you consider supporting older versions of Hiera?

m4ce commented 7 years ago

Let me see if I can do that for the next release 👍