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

fix: del --max-batch-time-ms and enable maxBatchTimeMs in values #1412

Merged
merged 1 commit into from
Sep 3, 2024
Merged

fix: del --max-batch-time-ms and enable maxBatchTimeMs in values #1412

merged 1 commit into from
Sep 3, 2024

Conversation

patsevanton
Copy link
Contributor

@patsevanton patsevanton commented Sep 3, 2024

Description:

In this pull request, changes have been made to the Snuba configuration to enhance flexibility and simplify management of the --max-batch-time-ms parameter. Previously, this parameter was hardcoded in several deployment templates, limiting the configurability for users.

Key Changes:

  • Removed Hardcoded Parameter: The --max-batch-time-ms parameter has been removed from all deployment templates where it was previously hardcoded.
  • Added Configuration to values.yaml: Introduced the maxBatchTimeMs parameter in the values.yaml file for all relevant Snuba consumers. This allows users to configure the --max-batch-time-ms value directly through the Helm chart values, enhancing flexibility and ease of management.

These changes are aimed at improving the manageability and configurability of the Snuba configuration, making it more adaptable to various deployment scenarios and user preferences.

Please review and provide feedback. Thank you!

@Mokto Mokto merged commit 086b477 into sentry-kubernetes:develop Sep 3, 2024
2 checks passed
@Mokto Mokto mentioned this pull request Sep 3, 2024
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.

2 participants