puppet-splunk
puppet-splunk copied to clipboard
WIP: Fix issues 230, 232, 235, and 249
- Moved defaults to module data, and removed the params class
- Privatized all
splunk::enterprise::andsplunk::forwarder::install, config, and service classes - Added a
$releaseparam, which replaces the$versionparam- For ensurable package_providers, the release is used as the Splunk package ensure, if specified
- The release no longer defaults to a specific version and build, instead, the Splunk package resource defaults ensure to 'installed'
- Added a Splunk::Release type
- Added a service_ensure param, per #249
- Modified splunk*_version facts to be part of splunkforwarder and splunkenterprise fact hashes
- Removed init.pp, which only served to confuse
$[enterprise,forwarder]_package_srcand$package_sourceparams renamed to$managed_package_sourceand$unmanaged_package_source, for clarity- Fixed: enterprise and forwarder password classes cross-referenced params
Fixed #230 Fixed #232 Fixed #235 Fixed #249
The bulk of the changes are done, but here are a few notes about the initial commit:
- This commit finalizes what the original refactor should have been. The module much more closely follows the module best practices, and the refactor was necessary to fix a number of the issues reported by users.
- Still need to expand upon the README and update REFERENCES.md.
- I've been experiencing an issue with splunk_config redefining types and throwing errors during puppet-applys, for instance:
Info: Redefining splunk_distsearch in Puppet::Type
Error: Could not autoload puppet/type/splunk_config: Attempt to redefine entity 'http://puppet.com/2016.1/runtime/type/splunk_distsearch'. Originally set at file:/etc/puppetlabs/code/modules/splunk/lib/puppet/type/splunk_distsearch.rb?line=3.
Error: Evaluation Error: Error while evaluating a Resource Statement, Could not autoload puppet/type/splunk_config: Attempt to redefine entity 'http://puppet.com/2016.1/runtime/type/splunk_distsearch'. Originally set at file:/etc/puppetlabs/code/modules/splunk/lib/puppet/type/splunk_distsearch.rb?line=3. (file: /etc/puppetlabs/code/modules/splunk/manifests/enterprise.pp, line: 311, column: 3) on node centos7-64-1.home
@ody Thank you for the review! I've really dragged my feet digging into this PR after submitting. By now it'll probably need a nasty rebase, but I'll try to get the feedback addressed and get things rolling again.
Dear @nick-markowski, thanks for the PR!
This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?
You can find my sourcecode at voxpupuli/vox-pupuli-tasks
Dear @nick-markowski, thanks for the PR!
This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?
You can find my sourcecode at voxpupuli/vox-pupuli-tasks
Dear @nick-markowski, thanks for the PR!
This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?
You can find my sourcecode at voxpupuli/vox-pupuli-tasks
Dear @nick-markowski, thanks for the PR!
This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?
You can find my sourcecode at voxpupuli/vox-pupuli-tasks
Dear @nick-markowski, thanks for the PR!
This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?
You can find my sourcecode at voxpupuli/vox-pupuli-tasks
Dear @nick-markowski, thanks for the PR!
This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs? If you need any help, you can reach out to us on our IRC channel voxpupuli on Freenode or our Slack channel voxpupuli at slack.puppet.com. You can find my sourcecode at voxpupuli/vox-pupuli-tasks