update confluent kafka to 2.13.0#22509
Merged
steveny91 merged 2 commits intopiotr.wolski/update-confluent-kafkafrom Feb 11, 2026
Merged
update confluent kafka to 2.13.0#22509steveny91 merged 2 commits intopiotr.wolski/update-confluent-kafkafrom
steveny91 merged 2 commits intopiotr.wolski/update-confluent-kafkafrom
Conversation
1233b63 to
778328a
Compare
This comment has been minimized.
This comment has been minimized.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files🚀 New features to boost your workflow:
|
778328a to
c9939f8
Compare
56cffe7 to
45487a6
Compare
steveny91
previously approved these changes
Feb 9, 2026
Review from steveny91 is dismissed. Related teams and files:
- agent-integrations
- .builders/images/windows-x86_64/build_script.ps1
- agent_requirements.in
- kafka_actions/pyproject.toml
- kafka_consumer/pyproject.toml
steveny91
approved these changes
Feb 9, 2026
Contributor
❤️ |
rdesgroppes
approved these changes
Feb 9, 2026
piochelepiotr
approved these changes
Feb 11, 2026
4a3bbfa
into
piotr.wolski/update-confluent-kafka
75 of 83 checks passed
steveny91
added a commit
that referenced
this pull request
Feb 13, 2026
* update confluent kafka to 2.13.0 * move changelog * Patch librdkafka to avoid build error due to missing header (#22509) Co-authored-by: Steven Yuen <steven.yuen@datadoghq.com> * changelogs * remove stale changelogs --------- Co-authored-by: Piotr Wolski <piotr.wolski42@gmail.com> Co-authored-by: Alex Lopez <alex.lopez.zorzano@gmail.com>
lukepatrick
pushed a commit
to lukepatrick/integrations-core
that referenced
this pull request
Mar 17, 2026
* update confluent kafka to 2.13.0 * move changelog * Patch librdkafka to avoid build error due to missing header (DataDog#22509) Co-authored-by: Steven Yuen <steven.yuen@datadoghq.com> * changelogs * remove stale changelogs --------- Co-authored-by: Piotr Wolski <piotr.wolski42@gmail.com> Co-authored-by: Alex Lopez <alex.lopez.zorzano@gmail.com> Signed-off-by: lukepatrick <lukephilips@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR applies a patch to librdkafka to let us build it in our runners, such that we don't sacrifice glibc compatibility in the Agent (glibc 2.17 for linux amd64, glibc 2.23 for linux arm64).
This change in librdkafka adds usage of the
getentropyfunction, declared in the<sys/random.h>header. This function was introduced with glibc 2.25, which is above our compatibility threshold. This header is not in our builders, causing the build to fail.However, the usage of
getentropyon that change is gated by a configure-time check that sets theHAVE_GETENTROPYpreprocessor variable. Therefore, we can simply skip the offending includes based on this variable for no apparent harm. I will try to upstream this patch, as it seems to be generally correct.Post merge note: I realized this had already been reported upstream: confluentinc/librdkafka#5283
Motivation
Making it possible to support the confluent-kafka update to 2.13.0 as #22446 intended.
Review checklist (to be filled by reviewers)
qa/skip-qalabel if the PR doesn't need to be tested during QA.backport/<branch-name>label to the PR and it will automatically open a backport PR once this one is merged