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

redis.json removed on purge { config => true } #461

Closed
jcochard opened this issue Jan 7, 2016 · 2 comments
Closed

redis.json removed on purge { config => true } #461

jcochard opened this issue Jan 7, 2016 · 2 comments

Comments

@jcochard
Copy link
Contributor

jcochard commented Jan 7, 2016

Testing in an ubuntu 12.04 vagrant instance that is both server/client, enterprise enabled, the redis.json config is removed when config purge is set to true.

==> default: Info: Computing checksum on file /etc/sensu/conf.d/redis.json
==> default: Info: /Stage[main]/Sensu::Redis::Config/File[/etc/sensu/conf.d/redis.json]: Filebucketed /etc/sensu/conf.d/redis.json to puppet with sum ca24608064728ad61e8b846b16671979
==> default: Notice: /Stage[main]/Sensu::Redis::Config/File[/etc/sensu/conf.d/redis.json]/ensure: removed

From redis/config.pp:

if $sensu::_purge_config and !$sensu::server and !$sensu::api {
    $ensure = 'absent'
  } else {
    $ensure = 'present'
  }

I believe the issue is that with enterprise, we have to set $sensu::server and $sensu::api = false, so likely just need to add in an and !$sensu::enterprise. I will submit a pull request shortly.

@jcochard
Copy link
Contributor Author

jcochard commented Jan 9, 2016

#462

@jlambert121
Copy link
Contributor

Closed with #462

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

No branches or pull requests

2 participants