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

module doesn't notify splunk restart when changing an input #63

Closed
adamstrawson opened this issue Oct 20, 2016 · 4 comments
Closed

module doesn't notify splunk restart when changing an input #63

adamstrawson opened this issue Oct 20, 2016 · 4 comments

Comments

@adamstrawson
Copy link

adamstrawson commented Oct 20, 2016

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 3.8.5
  • Ruby: ruby 2.3.1p112 (2016-04-26) [x86_64-linux-gnu]
  • Distribution: Ubuntu 16.04.1 LTS
  • Module version: 4.0.0

How to reproduce

class { '::splunk::forwarder': }

@splunkforwarder_input { 'source_type_example':
    section => "monitor://$logs_path/*.log",
    setting => 'sourcetype',
    value   => $name,
    tag     => "splunk_forwarder"
  }

What are you seeing

When applying a change to an instance that has already had puppet applied to it (eg. splunkforwarder already installed, then editing 'source_type_example' and running again), splunkforwarder is not restarted.

What behaviour did you expect instead

When making a chance to the resource type for splunk, splunk should be restarted

Output log

Notice: /Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/inputs.conf]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Splunkforwarder_input[default_host]/ensure: created

No mention of a restart in the output, when running splunk list monitors the new input doesn't exist. Once I manually restart splunk splunk restart, the monitor then shows.

@Vincent--
Copy link

Seems to work properly in version 5.0.2
Might be due to this fix: f520a18

Could you try again with this version of the puppet-splunk module?

@crayfishx
Copy link
Contributor

Yes, I think this was the exact problem that #50 fixed

@adamstrawson
Copy link
Author

Perfect, I'll test this tomorrow when I'm back in the office 👍

@adamstrawson
Copy link
Author

Version 5.0.2 does indeed fix this issue. Thanks!

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

3 participants