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.

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

Add the code for the "Manager" option; have both old and new option r… #135

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

schneewe
Copy link
Contributor

@schneewe schneewe commented Oct 8, 2022

The README stated, that it is possible to have manager as value, but for me I didn't see any code for the Manager.
So I added the code for the manager and set the pass4SymmKey.

Additionally I changed the code, to remove the manager_uri, not even depending on the current splunk version.
No both the old and the new options were removed, in such cases were wee would change the license group, and splunk upgrade from version <9.0 to >= 9.x

@dtwersky
Copy link
Collaborator

dtwersky commented Oct 9, 2022

@schneewe That is an error in the README.md. When the instance is a LM, you just set the splunk_license_group to Enterprise.

@dtwersky dtwersky closed this Oct 9, 2022
@schneewe
Copy link
Contributor Author

schneewe commented Oct 9, 2022

@dtwersky
Yeah I see that only the Enterprise value is needed, but nothing in the role sets the pass4SymmKey in the general stanza for the LM. So it never activate the License Manager. That why I wrote the additional code and for me then it makes sense.
I also find a variable in the default vars "splunk_general_key" which is for the option, but again, nothing is using this.

@dtwersky
Copy link
Collaborator

dtwersky commented Oct 9, 2022

@schneewe you are correct about the pass4SymmKey. Strange that the variable does not exist anywhere.

Maybe we should add a var called splunk_lm_key just like there is splunk_ds_key, and this task should have something like:
pass4SymmKey: "{{ splunk_lm_key }}", and set the same key for the LM and the Peers?

When splunk starts up, it will automatically set the general pass4SymmKey to changeme, so we should have something somewhere the option to set it, and be able to override it in this task. I'm also thinking that maybe we should set the general pass4SymmKey in the configure_splunk_secret.yml task when it's called by install_splunk.yml?

I'm thinking if we should get rid of the splunk_general_key variable altogether, since it is not referenced anywhere, and replace it with a generic pass4SymmKey.

In any case, the README should be updated.

@dtwersky dtwersky reopened this Oct 9, 2022
@dtwersky dtwersky self-assigned this Oct 9, 2022
@schneewe
Copy link
Contributor Author

I will update the README and try to make a suggestion to solve this.
Honestly I would make the change in the configure_license.yml and check if the host is in the licensemaster group. Because not everyone is setting a global splunk_secret and so the task would not run.
I use this role on many of my customers and never used the configure_secret task, because it's not necessary.

@dtwersky dtwersky self-requested a review November 10, 2022 14:24
@dtwersky dtwersky added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants