fix(kafka): fix crash from _get_cluster_id#18236
fix(kafka): fix crash from _get_cluster_id#18236gh-worker-dd-mergequeue-cf854d[bot] merged 3 commits into
_get_cluster_id#18236Conversation
|
BenchmarksBenchmark execution time: 2026-05-22 11:42:28 Comparing candidate commit 7c03fe2 in PR branch Found 0 performance improvements and 3 performance regressions! Performance is the same for 559 metrics, 9 unstable metrics. scenario:iastaspects-lstrip_aspect
scenario:iastaspectsospath-ospathbasename_aspect
scenario:telemetryaddmetric-1-count-metric-1-times
|
7c03fe2 to
55ba2b7
Compare
Codeowners resolved as |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 55ba2b7886
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
quinna-h
left a comment
There was a problem hiding this comment.
LGTM, do we need to add a small unit test?
i.e. something in tests/contrib/kafka/test_kafka that calls _get_cluster_id and asserts that 1) result is "", 2) list_topics is never invoked
## Description This fixes a crash that would occur when using the Kafka integration. **Python stacks** ``` Thread 0x00007f2e5affd6c0 (most recent call first): File "/usr/lib/python3.12/threading.py", line 355 in wait File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/_queue.py", line 242 in get File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/worker.py", line 120 in _target File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2e797fa6c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2e7bfff6c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2ed9ffb6c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2ef0d016c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2ed97fa6c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap Thread 0x00007f2edbfff6c0 (most recent call first): File "/opt/venv/lib/python3.12/site-packages/ddtrace/contrib/internal/kafka/patch.py", line 232 in traced_poll_or_consume File "/opt/app/pkgs/extensions/kafka/confluent_kafka_consumer.py", line 136 in consume File "/opt/app/suma/kafka/consumer/runner.py", line 158 in consume_messages File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 245 in poll_messages File "/opt/app/suma/kafka/consumer/runner.py", line 333 in _run File "/opt/venv/lib/python3.12/site-packages/ddtrace/_trace/tracer.py", line 860 in func_wrapper File "/opt/app/suma/kafka/consumer/runner.py", line 325 in run File "/opt/app/suma/kafka/consumer_cmd.py", line 156 in run_consumer_thread File "/usr/lib/python3.12/threading.py", line 1012 in run File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 133 in _run_old_run_func File "/opt/venv/lib/python3.12/site-packages/sentry_sdk/integrations/threading.py", line 140 in run File "/usr/lib/python3.12/threading.py", line 1075 in _bootstrap_inner File "/usr/lib/python3.12/threading.py", line 1032 in _bootstrap ``` **Native stack** ``` Error UnixSignal: Process terminated with SEGV_MAPERR (SIGSEGV) #0 0x00007f3058686017 rd_kafka_q_serve_rkmessages #1 0x00007f30586480bc rd_kafka_consume_batch_queue #2 0x00007f306d783cda cfunction_call #3 0x00007f306d796f02 _PyObject_Call.localalias #4 0x00007f306d76804a _PyEval_EvalFrameDefault.localalias #5 0x00007f306d783a43 object_vacall #6 0x00007f306d827040 PyObject_CallFunctionObjArgs #7 0x00007f306d757ccc _PyObject_MakeTpCall.localalias #8 0x00007f306d762ec4 _PyEval_EvalFrameDefault.localalias #9 0x00007f306d7b220e _PyObject_VectorcallTstate.lto_priv.14 #10 0x00007f306d7b1d6f method_vectorcall #11 0x00007f306d8a3afd thread_run #12 0x00007f306d89018c pythread_wrapper #13 0x00007f306d3e73c6 start_thread #14 0x00007f306d46c1cc __clone3 ``` Co-authored-by: thomas.kowalski <thomas.kowalski@datadoghq.com>
Description
This fixes a crash that would occur when using the Kafka integration.
Python stacks
Native stack