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.

Sign up for GitHub

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

Jump to bottom

fixes splunk/ansible-role-for-splunk#49 #130

Merged
merged 2 commits into from
Aug 24, 2022
Merged

fixes splunk/ansible-role-for-splunk#49 #130

merged 2 commits into from
Aug 24, 2022

Conversation

jewnix
Copy link
Collaborator

@jewnix jewnix commented Aug 23, 2022

This allows you to have multiple apps in a single repo, and rsync the apps in the correct destination directory.

jewnix
jewnix commented Aug 23, 2022
View reviewed changes
roles/splunk/tasks/install_apps.yml
@@ -90,7 +90,7 @@

- name: Ensure correct permissions are set
file:
path: "{{ splunk_home }}/{{ app_dest }}/{{ item.name }}"
path: "{{ splunk_home }}/{{ app_dest }}"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

If I did not remove this,Ansible will create a empty directory in splunk_app_deploy_path with the app name.
This will should have no impact, since the directory where the apps are should be owned by the user splunk is running as.

@dtwersky dtwersky merged commit 852d71f into splunk:master Aug 24, 2022
zyphermonkey pushed a commit to zyphermonkey/ansible-role-for-splunk that referenced this pull request Sep 30, 2022
Squashed commit of the following:
ef9b325 
commit 852d71f
Author: Jewnix <[email protected]>
Date:   Wed Aug 24 12:46:39 2022 -0400

    fixes splunk#49 (splunk#130)

    * fixes splunk#49

commit c3c66a5
Author: Jewnix <[email protected]>
Date:   Tue Aug 23 14:26:44 2022 -0400

    configure restartSplunkd on serverclass level (splunk#129)

commit 7da7c25
Author: Jewnix <[email protected]>
Date:   Mon Aug 22 13:24:34 2022 -0400

    fixewd missing quotes (splunk#128)

commit e4c120b
Author: Jewnix <[email protected]>
Date:   Fri Aug 19 10:23:10 2022 -0400

    fixes splunk#75 (splunk#127)

commit 41b2a8e
Author: zyphermonkey <[email protected]>
Date:   Tue Aug 16 21:05:19 2022 -0400

    feat: add option to create polkit rules file for splunk (splunk#115)

    Co-authored-by: Merrel Raney III <[email protected]>
    Co-authored-by: dtwersky <[email protected]>

commit 9276f51
Author: arcsector <[email protected]>
Date:   Tue Aug 16 17:54:09 2022 -0700

    Additional OOTB Cluster components (splunk#125)

    * Fixing splunk#123

    * Adding DMC Playbook and Task

    * DMC playbook merge with task - update README

    Co-authored-by: haraksin <[email protected]>

commit 5e9c8db
Author: Jewnix <[email protected]>
Date:   Tue Aug 16 20:49:41 2022 -0400

    Multiple updates (splunk#122)

    * added license files and test environments to .gitignore

    * Added more license configuration options

    * Disable THP using tuned

    * new variables for version and build. updated to 9.0.0.1

    * check if splunk_package_version is newer. regex gets full version

    * Fixed many boot-start issues

commit a8d2df5
Author: arcsector <[email protected]>
Date:   Wed Aug 3 13:02:23 2022 -0700

    Fixing splunk#123 (splunk#124)

    Co-authored-by: haraksin <[email protected]>

commit 13b7ca9
Author: Jewnix <[email protected]>
Date:   Wed Jun 1 22:43:10 2022 -0400

    Added sysctl to allow read of dmesg (splunk#119)

    * Added sysctl to allow read of dmesg

commit 2a962d1
Merge: 11f3922 e6a32d5
Author: dtwersky <[email protected]>
Date:   Mon Mar 28 11:10:44 2022 -0400

    Merge pull request splunk#117 from jewnix/splunk_version

    updated splunk package to 8.2.5

commit e6a32d5
Author: jewnix <[email protected]>
Date:   Mon Mar 28 10:30:55 2022 -0400

    updated splunk package to 8.2.5

commit 11f3922
Merge: 9ff2b92 62972ef
Author: dtwersky <[email protected]>
Date:   Thu Mar 24 16:59:14 2022 -0400

    Merge pull request splunk#116 from jewnix/fix_thp_disable

    disable-thp.service should be enabled and started

commit 62972ef
Author: jewnix <[email protected]>
Date:   Thu Mar 24 16:18:02 2022 -0400

    disable-thp.service should be enabled and started
@jewnix jewnix deleted the ds_bundled_app branch December 21, 2022 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dtwersky dtwersky dtwersky approved these changes

@anelson-splunk anelson-splunk anelson-splunk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jewnix @dtwersky @anelson-splunk