Optimize deployment setup time #795
Conversation
| @@ -75,16 +75,3 @@ | |||
| retries: "{{ shc_sync_retry_num }}" | |||
| delay: "{{ retry_delay }}" | |||
| no_log: "{{ hide_password }}" | |||
|
|
|||
| - name: Destructive sync search head | |||
There was a problem hiding this comment.
should we instead make it optional? would it cause any issues while performing upgrades?
There was a problem hiding this comment.
This is one of those tasks that took almost 5 minutes to complete. According to this doc https://docs.splunk.com/Documentation/Splunk/9.2.0/DistSearch/HowconfrepoworksinSHC#Why_a_recovering_member_might_need_to_resync_manually we shouldn't need to perform a destructive sync unless the cluster is recovering a member, and then it's only necessary on that one instance. We always could leave this off by default and have some flag to run it if possible.
| - include_tasks: set_certificate_prefix.yml | ||
| when: splunk.splunk_http_enabled|bool == true | ||
|
|
||
| - include_tasks: clean_user_seed.yml |
There was a problem hiding this comment.
do we need to remove this playbook from the repository? it's not used anywhere else.
There was a problem hiding this comment.
I don'r recall exactly, but I think the user seed is used to determine if this is the first time Splunk is bootstrapped or not. I remember running into a problem when doing upgrade and when using local dev build because of it
There was a problem hiding this comment.
Yeah this one I don't have strong preferences about, I couldn't find anything in the docs about what happens if we keep this around.
| @@ -122,14 +122,9 @@ | |||
| - include_tasks: enable_dsp.yml | |||
| when: "'dsp' in splunk and 'enable' in splunk.dsp and splunk.dsp.enable" | |||
|
|
|||
| - include_tasks: start_splunk.yml | |||
|
|
|||
| - include_tasks: set_certificate_prefix.yml | |||
There was a problem hiding this comment.
also this playbook?
There was a problem hiding this comment.
This playbook, set_certificate_prefix I essentially incorporated into roles/splunk_common/tasks/enable_splunkd_ssl.yml. If we turn on ssl we might as well set the certificate prefix in the same file instead of having two different steps.
Removed multiple restarts that were slowing down deployment setup as well as other tasks that are no longer recommended per Splunk Docs.