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: Update README.md with Latest Configuration Values #1665

Conversation

patsevanton
Copy link
Contributor

This PR updates the README.md file to reflect the latest configuration values for the Sentry Helm chart. The changes include updates to various sections of the configuration table to ensure it accurately represents the current state of the Helm chart.

  1. Clickhouse Configuration:

    • Removed clickhouse.clickhouse.imageVersion as it is no longer needed.
    • Added clickhouse.nodeSelector to allow for node selection configuration.
  2. Kafka Configuration:

    • Simplified and reorganized Kafka-related configurations.
    • Removed redundant Kafka topic configurations and added new topics for better event handling.
    • Updated Kafka topic names and configurations to align with the latest requirements.
  3. Global Configuration:

    • Added global.nodeSelector, global.sidecars, global.tolerations, and global.volumes for more flexible deployment configurations.
  4. Memcached and Redis:

    • Added nodeSelector options for both Memcached and Redis to support node-specific deployments.
  5. Metrics and Nginx:

    • Added sidecars and volumes configurations for metrics.
    • Added nodeSelector for Nginx to allow for node-specific deployments.
  6. PostgreSQL and PgBouncer:

    • Added PgBouncer configuration options for better connection pooling and management.
    • Updated PostgreSQL configurations for improved performance and reliability.
  7. Sentry Web and Worker:

    • Updated existingSecretEnv from a list to a string for consistency and simplicity.
  8. Snuba Consumers:

    • Removed autoOffsetReset configurations from various Snuba consumers as they are no longer needed.
    • Added sidecars configurations for profiling consumers.
  9. Zookeeper:

    • Added nodeSelector for Zookeeper to support node-specific deployments.

These changes aim to improve the flexibility, reliability, and performance of the Sentry Helm chart. Please review the changes and provide feedback.

Reviewers:

Thank you for your time and review!

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.

1 participant