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

localhost on agent #3732

Merged
merged 4 commits into from
Oct 7, 2023
Merged

localhost on agent #3732

merged 4 commits into from
Oct 7, 2023

Conversation

rmfitzpatrick
Copy link
Contributor

Description:
These changes update the SPLUNK_LISTEN_INTERFACE environment variable to be 127.0.0.1 for the default agent config locations where it was otherwise 0.0.0.0 in all cases. They also update the installers to only forward the env var to the target service if configured by the user.

supersedes #3639

Testing:
Updated unit and integration tests.

Documentation:
Updated readmes

@rmfitzpatrick rmfitzpatrick requested review from a team as code owners October 5, 2023 20:18
@rmfitzpatrick rmfitzpatrick marked this pull request as draft October 5, 2023 20:20
@rmfitzpatrick rmfitzpatrick force-pushed the localhostonagent branch 4 times, most recently from c509a75 to 34a7c86 Compare October 5, 2023 21:38
@rmfitzpatrick rmfitzpatrick marked this pull request as ready for review October 6, 2023 14:25
@rmfitzpatrick rmfitzpatrick force-pushed the localhostonagent branch 2 times, most recently from fa4375b to d01ed1a Compare October 6, 2023 15:39
Copy link
Contributor

@hughesjj hughesjj left a comment

Choose a reason for hiding this comment

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

Looks great to me, approved pending all test coverage (ansible) passing

internal/settings/settings.go Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@jeffreyc-splunk
Copy link
Contributor

Our salt module should also be updated: https://github.com/signalfx/splunk-otel-collector/blob/main/deployments/salt/splunk-otel-collector/collector_config.sls#L29

* Add chef tests for listen interface

* Add puppet tests for listen interface

* Add salt tests for listen interface
Copy link
Contributor

@jeffreyc-splunk jeffreyc-splunk left a comment

Choose a reason for hiding this comment

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

Installer changes LGTM.

@rmfitzpatrick rmfitzpatrick merged commit c901f74 into main Oct 7, 2023
361 checks passed
@delete-merged-branch delete-merged-branch bot deleted the localhostonagent branch October 7, 2023 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants