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 retry_on_failure/sending_queue config to values.yaml #460

Merged
merged 7 commits into from
Jun 17, 2022
Merged

Add retry_on_failure/sending_queue config to values.yaml #460

merged 7 commits into from
Jun 17, 2022

Conversation

hvaghani221
Copy link
Contributor

@hvaghani221 hvaghani221 commented Jun 3, 2022

Expose the batch, retry_on_failure and sending_queue configurations so they are configurable within the values.yaml.
Users would still use default values, but this way users have full control.

@hvaghani221
Copy link
Contributor Author

As of now, I have added retry_on_failure and sending_queue to splunkPlatform only.

@hvaghani221 hvaghani221 changed the title Add retry_on_failure/sending_queue/batch config to values.yaml Add retry_on_failure/sending_queue config to values.yaml Jun 9, 2022
@@ -75,6 +75,27 @@ splunkPlatform:
# Boolean for keeping Otel convention fields after renaming it
keepOtelConvention: true

# Refer https://github.com/open-telemetry/opentelemetry-collector/blob/main/exporter/exporterhelper/README.md#configuration
Copy link
Contributor

Choose a reason for hiding this comment

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

another nit

Suggested change
# Refer https://github.com/open-telemetry/opentelemetry-collector/blob/main/exporter/exporterhelper/README.md#configuration
# Refer to https://github.com/open-telemetry/opentelemetry-collector/blob/main/exporter/exporterhelper/README.md#configuration

Copy link
Contributor

@rmfitzpatrick rmfitzpatrick left a comment

Choose a reason for hiding this comment

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

lgtm w/ doc updates

@emaderer emaderer merged commit f93c2b8 into signalfx:main Jun 17, 2022
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.

3 participants