DOC-1790: Schema Registry Metadata Properties#1690
Conversation
Adds a new section covering metadata properties support in Schema Registry, including API usage, rpk flags, inheritance behavior, and Confluent Data Contracts compatibility notes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
✅ Deploy Preview for redpanda-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughAdded a new documentation section describing how Schema Registry supports arbitrary key-value metadata properties alongside schemas. The section covers request and response formats for registering and retrieving schemas with metadata, provides CLI examples using the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
modules/manage/pages/schema-reg/schema-reg-overview.adoc (2)
166-178: ⚡ Quick winSeparate endpoint text from the JSON payload example.
The current block is tagged as JSON but includes an HTTP request line, which hurts syntax accuracy and copy/paste usability.
Proposed edit
-[source,json] ----- -POST /subjects/my-topic-value/versions -{ +POST `/subjects/my-topic-value/versions` + +[source,json] +---- +{ "schema": "{\"type\":\"string\"}", "metadata": { "properties": { "owner": "platform-team", "application.version": "2.1.0" } } } ----🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@modules/manage/pages/schema-reg/schema-reg-overview.adoc` around lines 166 - 178, The example mixes an HTTP request line with a JSON code block, so split them: move the "POST /subjects/my-topic-value/versions" request line out of the [source,json] block and place it in its own code block (e.g., [source,http] or a plain example line) and leave only the JSON payload inside the [source,json] block; update the surrounding text to reference the two blocks so the JSON payload and the HTTP request line are separately selectable and properly syntax-highlighted.
191-191: ⚡ Quick winLink
{ui}to the Schema Registry UI page.Making this a cross-reference keeps navigation consistent and improves discoverability.
Proposed edit
-To view metadata on an existing schema, add `--print-metadata` to `rpk registry schema get`. You can also view metadata properties in {ui}. +To view metadata on an existing schema, add `--print-metadata` to `rpk registry schema get`. You can also view metadata properties in xref:manage:schema-reg/schema-reg-ui.adoc[].Based on learnings: AsciiDoc linking should prefer
xref:...[]so target titles render automatically rather than hard-coded link text.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@modules/manage/pages/schema-reg/schema-reg-overview.adoc` at line 191, The AsciiDoc uses the placeholder {ui} in the sentence "You can also view metadata properties in {ui}." — replace that placeholder with an AsciiDoc cross-reference using the xref: syntax pointing to the Schema Registry UI page (e.g., xref:Schema-Registry-UI-Page[]), so the link renders the target title automatically; update the text in schema-reg-overview.adoc where {ui} appears to use xref:...[] instead of the raw variable.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@modules/manage/pages/schema-reg/schema-reg-overview.adoc`:
- Around line 180-189: The example shows use of unsupported rpk flags
(--metadata-properties and -p) when creating a schema; verify against the
current rpk implementation and either remove or replace them with supported
alternatives. Specifically, check the rpk CLI implementation or docs for the
commands used in the schema example and update the sample in
schema-reg-overview.adoc so it no longer references --metadata-properties or -p
(or annotate that metadata is unsupported), and ensure the surrounding text
about metadata on GET /subjects/{subject}/versions/{version} and GET
/schemas/ids/{id} remains accurate.
---
Nitpick comments:
In `@modules/manage/pages/schema-reg/schema-reg-overview.adoc`:
- Around line 166-178: The example mixes an HTTP request line with a JSON code
block, so split them: move the "POST /subjects/my-topic-value/versions" request
line out of the [source,json] block and place it in its own code block (e.g.,
[source,http] or a plain example line) and leave only the JSON payload inside
the [source,json] block; update the surrounding text to reference the two blocks
so the JSON payload and the HTTP request line are separately selectable and
properly syntax-highlighted.
- Line 191: The AsciiDoc uses the placeholder {ui} in the sentence "You can also
view metadata properties in {ui}." — replace that placeholder with an AsciiDoc
cross-reference using the xref: syntax pointing to the Schema Registry UI page
(e.g., xref:Schema-Registry-UI-Page[]), so the link renders the target title
automatically; update the text in schema-reg-overview.adoc where {ui} appears to
use xref:...[] instead of the raw variable.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 72001e41-62a3-461b-b5bd-051e338fbb01
📒 Files selected for processing (1)
modules/manage/pages/schema-reg/schema-reg-overview.adoc
-p is a shorthand for --metadata-properties (single flag, not two). The flag accepts key=value or JSON, and can be passed multiple times. Uses StringArrayVar to avoid comma conflicts with JSON values. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Joyce Fee <102751339+Feediver1@users.noreply.github.com>
…rence Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Use [,json]/[,bash] shorthand consistent with file conventions - Move POST method/path to prose; JSON block now contains body only - Fix dangling participial: "Using rpk" → "To set metadata using rpk" - Fix NOTE: remove double blank line, use * list markers, fix trailing whitespace, distinguish schema-level features from config options, add link to Confluent Data Contracts spec - Add line continuation to rpk reference examples for readability Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Gellért Peresztegi-Nagy <gellert.nagy@redpanda.com>
Co-authored-by: Gellért Peresztegi-Nagy <gellert.nagy@redpanda.com>
Co-authored-by: Gellért Peresztegi-Nagy <gellert.nagy@redpanda.com>
kbatuigas
left a comment
kbatuigas
left a comment
There was a problem hiding this comment.
-
Do we need to add to What's New?
-
Wouldn't most of this new content be a better fit for https://deploy-preview-1690--redpanda-docs-preview.netlify.app/current/manage/schema-reg/schema-reg-api/ ? Although that page is also getting quite long and I'm not sure how helpful it is to keep adding on.
| --metadata-properties '{"owner":"platform-team","application.version":"2.1.0"}' | ||
| ---- | ||
|
|
||
| To view metadata on an existing schema, add `--print-metadata` to `rpk registry schema get`. You can also view metadata properties in {ui}. |
There was a problem hiding this comment.
Should the rpk registry schema get page also be updated with the new flag?
There was a problem hiding this comment.
I checked with @r-vasquez on this:
Hi Joyce, this feature is only relevant to schema create and get and you won't see it at the root level, here are some examples in the original PR description: redpanda-data/redpanda#29431
Co-authored-by: Kat Batuigas <36839689+kbatuigas@users.noreply.github.com>
Co-authored-by: Kat Batuigas <36839689+kbatuigas@users.noreply.github.com>
…lents Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Yes on first bullet, no on second--don't want users to have to leave the metadata context, especially if the other page is too long/unwieldy. |
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The schema-reg-contexts page referenced rpk-registry-context-list and rpk-registry-context-delete, which didn't exist, causing xref errors during the Netlify build. Adds the three context command pages and registers them in the nav. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add explicit [#metadata-properties] ID to the heading so the xref resolves to the correct section rather than the top of the page. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
The only thing I'll raise is that the reference for rpk registry schema get still doesn't list --metadata-properties as an available flag even though we mention using it in the main doc. Otherwise, no blockers
4 participants
Summary
== Metadata propertiessection toschema-reg-overview.adoc, positioned after== JSON Schemaand before== Next stepsmetadata.properties--metadata-properties(key=value) and-p(JSON string) flags, plus--print-metadatafor viewingnullto clear) and versioning behavior (different metadata = new version)Preview
In Cloud: https://deploy-preview-1690--redpanda-docs-preview.netlify.app/redpanda-cloud/manage/schema-reg/schema-reg-overview/#metadata-properties
Self-Managed: https://deploy-preview-1690--redpanda-docs-preview.netlify.app/current/manage/schema-reg/schema-reg-overview/#metadata-properties
What's New
Test plan
{ui}attribute resolves in both cloud and self-hosted builds-pflag name is correct in rpk CLI reference🤖 Generated with Claude Code