Skip to content

Update splunk-otel-collector.conf.erb #5891

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Update splunk-otel-collector.conf.erb #5891

wants to merge 5 commits into from

Conversation

aknarts
Copy link
Contributor

@aknarts aknarts commented Feb 11, 2025

  • Allow HEC token to be in run_state
  • Fix access to additional env_vars

Description:

  • Making run_state the default when getting tokens, makes sense as that storage is empty by default and if not that value should be used
  • Allowing HEC token to be also in run_state makes sense from security standpoint
  • the @node variable does not exist, it is not passed through the template resource, using the same as any above references.

Testing:

Documentation:

@aknarts
Update splunk-otel-collector.conf.erb
97f1fa0 
 * Allow HEC token to be in run_state
 * Fix access to additional env_vars
@aknarts aknarts requested review from a team as code owners February 11, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hughesjj hughesjj hughesjj approved these changes

@jinja2 jinja2 jinja2 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aknarts @hughesjj @jinja2