Skip to content

Conversation

@arcsector
Copy link
Contributor

@arcsector arcsector commented Aug 28, 2023

Summary

Fixes an issue where the DMC is not automatically configured in Distributed mode because of the lack of the local directory referenced for the file {{ splunk_home }}/etc/apps/splunk_monitoring_console/local/splunk_monitoring_console_assets.conf in configure_dmc.yml. This fix makes sure the directory exists before creating the file, ensuring that the subsequent task doesn't fail.

Also fixes an issue where Splunk returns a non-zero exit code when you add a peer to the DMC as a search peer which is already there (rc 24), and logs as such. This no longer fails the task, and instead just doesn't log a change to the ansible worker.

Also fixes issue #197

Changed

  • roles/splunk/tasks/configure_dmc.yml

@arcsector arcsector changed the title Creates local DMC dir if nonexistent Fixes issues with configure_dmc.yml Aug 28, 2023
@ChrisHills463
Copy link
Contributor

I ran into this issue with configure_dmc.yml today, is there any likelihood this will get reviewed and merged any time soon?

Copy link
Collaborator

@dtwersky dtwersky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just the fqcn

---
- name: Check splunk
include_task: check_splunk.yml
include_tasks: check_splunk.yml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
include_tasks: check_splunk.yml
ansible.builtin.include_tasks: check_splunk.yml

include_tasks: check_splunk.yml

- name: Configure DMC as a IDXC member
include_tasks: configure_idxc_sh.yml
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
include_tasks: configure_idxc_sh.yml
ansible.builtin.include_tasks: configure_idxc_sh.yml

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 this pull request may close these issues.

3 participants