-
Notifications
You must be signed in to change notification settings - Fork 614
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
[CORE-8999] schema_registry/protobuf: Add test for legacy_map #25001
[CORE-8999] schema_registry/protobuf: Add test for legacy_map #25001
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will need the patch for the bazel build too. There are some examples using single_version_override in MODULE.bazel
The Confluent Schema Registry client renders maps in the expanded form demonstrated by the test, which is considered invalid by libprotobuf >= v25. The patch required to fix this is: redpanda-data/vtools#3434 Signed-off-by: Ben Pope <ben@redpanda.com>
9afe3ee
to
d33def4
Compare
CI test resultstest results on build#61502
|
CI Failure is |
/backport v24.3.x |
/backport v24.2.x |
Failed to create a backport PR to v24.3.x branch. I tried:
|
Failed to create a backport PR to v24.2.x branch. I tried:
|
The Confluent Schema Registry client renders maps in the expanded form demonstrated by the test, which is considered invalid by libprotobuf >= v25.
The patch required to fix this is: https://github.com/redpanda-data/vtools/pull/3434
Backports Required
Release Notes
Bug Fixes