Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unknown variable collectd_version #700

Open
fnoop opened this issue Oct 24, 2017 · 2 comments
Open

Unknown variable collectd_version #700

fnoop opened this issue Oct 24, 2017 · 2 comments
Labels
bug Something isn't working needs-feedback Further information is requested

Comments

@fnoop
Copy link

fnoop commented Oct 24, 2017

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 5.2.0
  • Ruby: 2.3.3p222
  • Distribution: Ubuntu 17.04 zesty
  • Module version: 6.0.0

How to reproduce (e.g Puppet code you use)

class { "collectd":
config_file => $config_file,
plugin_conf_dir => $plugin_conf_dir,
plugin_conf_dir_mode => "755",
purge => true,
recurse => true,
purge_config => true,
minimum_version => '5.7',
manage_package => $manage_package,
manage_repo => $manage_repo,
manage_service => true,
service_name => 'maverick-collectd',
typesdb => [$typesdb],
}

What are you seeing

Error: Evaluation Error: Unknown variable: '::collectd_version'. at /srv/maverick/software/maverick/manifests/puppet-modules/collectd/manifests/init.pp:37:33 on node maverick-ubuntu17

Any additional information you'd like to impart

puppet is being run as a local standalone process (puppet apply), not as client/server. Worked fine with version 5.3.0 of this module

@juniorsysadmin juniorsysadmin added bug Something isn't working needs-feedback Further information is requested labels Oct 24, 2017
@crigertg
Copy link

crigertg commented Nov 3, 2017

The problem occurs when strict_variables are enabled in the puppetmaster configuration: https://puppet.com/docs/puppet/5.2/configuration.html#strictvariables

When the fact is empty when collectd is not installed (that's the case on the first puppet run with this module) it fails.

@btravouillon
Copy link
Member

Hi @fnoop ,

PR #708 merged yesterday. Can you give a try in your environment?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-feedback Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants