Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

Pr44 copy #48

Open
wants to merge 56 commits into
base: main
Choose a base branch
from
Open

Pr44 copy #48

wants to merge 56 commits into from

Conversation

sshvans
Copy link
Contributor

@sshvans sshvans commented Aug 27, 2022

Description of changes:
Copy of #44 with resolved merge conflicts and build issues

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Bill Bartlett and others added 30 commits August 14, 2019 15:37
initial commit of user_data.sh -- testing viability of using external shell scripts for user data instead of doing everything in-line with the template.
missed updating the creationpolicy timeout on the previous commit.  fixing that, and lowering the timeout to 10 minutes instead of 15.
10 minutes was not enough for the CM to boot before timeout.  changed back to 15.
changed systemctl commands to use splunk binary
moved variables up in user data
removed the cloud init files for splunk ansible before ansible is ran
without something in the functions, bash will error out on a syntax error.  added echo statements as placeholders.
Prior to this change, when the script was ran without any arguments, it would still try to execute the base install of Splunk.  This change was made so that if someone executes the script to see usage, it won't try to execute the install before showing the usage.
previously, it wrote to /etc/splunk/splunk.license and then the user_data script moved it.  that seems unnecessary, so I'm having cloudformation write directly to the appropriate directory.
the minimum number of indexers in an AZ must be at least as large as the replication factor.  since we're using RF = 2, and a default minimum of 2 AZ, there must be at least 4 indexers in total - 2 in each AZ.
added sleep in the deployer after restart.  this allows for splunk to be fully restarted before issuing splunk cli commands that would fail because the REST endpoint wasn't yet responding.
converted the JSON templates to YAML, which is now required for the quickstart repo.
unfinished docs, but commiting so that they may be shared.
Bill Bartlett and others added 26 commits February 12, 2021 09:56
in order to use the automated testing that the AWS Quickstart team provides, the templates need to be named *template.
… .yaml, the

corresponding template URL was not updated.
successfully tested taskcat on us-west-2 with these settings, where bracketed sample values were substituted with real data.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants