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

zabbix::userparameters has to require zabbix::agent #539

Closed
baurmatt opened this issue Sep 17, 2018 · 0 comments · Fixed by #540
Closed

zabbix::userparameters has to require zabbix::agent #539

baurmatt opened this issue Sep 17, 2018 · 0 comments · Fixed by #540

Comments

@baurmatt
Copy link
Contributor

baurmatt commented Sep 17, 2018

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: All
  • Ruby: All
  • Distribution: All
  • Module version: Current master

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

zabbix::userparameters { 'mysql':
  content => 'foo',
}
...
include zabbix::agent

What are you seeing

$include_dir          = getvar('::zabbix::agent::include_dir')
$zabbix_package_agent = getvar('::zabbix::agent::zabbix_package_agent')
$agent_config_owner   = getvar('::zabbix::agent::agent_config_owner')
$agent_config_group   = getvar('::zabbix::agent::agent_config_group')

==> Resolve to undef

What behaviour did you expect instead

zabbix::userparameters uses the correct variables.

Output log

Any additional information you'd like to impart

baurmatt added a commit to syseleven/puppet-zabbix that referenced this issue Sep 17, 2018
Before this commit it was possible that the configuration variables were
undef because of unincluded subclasses. This commits adds a include
directive for zabbix::userparameters which ensures that this can't
happen.

Fixes voxpupuli#539.
baurmatt added a commit to syseleven/puppet-zabbix that referenced this issue Sep 24, 2018
Before this commit it was possible that the configuration variables were
undef because of unincluded subclasses. This commits adds a include
directive for zabbix::userparameters which ensures that this can't
happen.

Fixes voxpupuli#539.
baurmatt added a commit to syseleven/puppet-zabbix that referenced this issue Nov 2, 2018
Before this commit it was possible that the configuration variables were
undef because of unincluded subclasses. This commits adds a include
directive for zabbix::userparameters which ensures that this can't
happen.

Fixes voxpupuli#539.
baurmatt added a commit to syseleven/puppet-zabbix that referenced this issue Nov 2, 2018
Before this commit it was possible that the configuration variables were
undef because of unincluded subclasses. This commits adds a include
directive for zabbix::userparameters which ensures that this can't
happen.

Fixes voxpupuli#539.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant