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

Solaris support #333

Closed
wants to merge 9 commits into from
Closed

Conversation

davemcdonnell
Copy link
Contributor

Pull Request (PR) description

Fixes for permission startup when running as root user.
Removal of net-tools due to this incorrectly being specified for SunOS.

This Pull Request (PR) fixes the following issues

Fixes #310 - it appears the default user for splunk has changed from root to splunk at some point. The default being root meant not explicitly specifying the user still worked. The resulted in the module config having root, file ownership having root, but the service starting as splunk due to it not being explicitly specified.

Partially fixes #272 - Net-tools removed. On the other issue reported, using the .pkg file, it appears Splunk no longer ship a .pkg format installer. Only .p5p and .Z. I had issues getting .p5p support working because I'm using Solaris zones, and this is incompatible with the way the installer works (Solaris zones need a pkg provider defined in the global zone, and cant install .p5p files without this).

My workaround for the missing .pkg download was to build a .pkg file myself using pkgbuild. Due to licensing I dont think I'm able to distribute the build package but I've put pkgbuild spec file and some instructions in this git repo:
https://github.com/davemcdonnell/splunk-solarispkg

With all of the above I've got splunk forwarder running on Solaris 11.4.45 with splunkforwarder 9.0.0.

@davemcdonnell
Copy link
Contributor Author

Test failing, i'll review, and separate out the fixes in separate commits and make sure tests are passing

@davemcdonnell
Copy link
Contributor Author

I believe I've fixed the tests but unable to 100% verify locally due to some issues getting tests to run (the relevant test to this pull req are passing now).

I've run out of time to get the commits separated out for each individual issue. I wanted to reopen this PR to see if tests pass, and maybe someone can finish it up.

@davemcdonnell davemcdonnell reopened this Jun 24, 2022
@davemcdonnell
Copy link
Contributor Author

Replaced by:
#334
#335
#336

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants