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

feat: add logLevel option to ingestConsumerAttachments #1468

Merged

Conversation

patsevanton
Copy link
Contributor

Add logLevel option to ingestConsumerAttachments

This PR introduces a new logLevel option for the ingestConsumerAttachments configuration in the Sentry Helm chart. This allows users to control the logging level of the ingest consumer attachments process.

Changes:

  • Added logLevel parameter to values.yaml.
  • Updated deployment-sentry-ingest-consumer-attachments.yaml to include the new logLevel option in the command arguments.

@patsevanton
Copy link
Contributor Author

patsevanton commented Sep 18, 2024

test with logLevel: debug

k get pod -n test | grep sentry-ingest-consumer-attachments

sentry-ingest-consumer-attachments-6b687d867b-qhmzt               1/1     Running   0             27m

k logs -n test sentry-ingest-consumer-attachments-6b687d867b-qhmzt

00:55:40 [DEBUG] arroyo.processing.processor: Starting
00:55:46 [INFO] arroyo.processing.processor: New partitions assigned: {Partition(topic=Topic(name='ingest-attachments'), index=0): 0}
00:55:46 [DEBUG] arroyo.processing.processor: Initialized processing strategy: <arroyo.processing.strategies.healthcheck.Healthcheck object at 0x7f774adfaf10>
00:55:47 [DEBUG] arroyo.processing.processor: Waited 0.0001 seconds for offsets to be committed to <arroyo.backends.kafka.consumer.KafkaConsumer object at 0x7f774ad83f50>.

@Mokto Mokto merged commit 8005f0f into sentry-kubernetes:develop Sep 18, 2024
2 checks passed
@Mokto Mokto mentioned this pull request Sep 18, 2024
@patsevanton patsevanton deleted the ingestConsumerAttachments branch September 18, 2024 13:58
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