Skip to content

docs: document usage statistics reporting, including K8s deployment metadata (DOC-1910) - #1885

Open
JakeSCahill wants to merge 2 commits into
mainfrom
DOC-1910-k8s-telemetry
Open

docs: document usage statistics reporting, including K8s deployment metadata (DOC-1910)#1885
JakeSCahill wants to merge 2 commits into
mainfrom
DOC-1910-k8s-telemetry

Conversation

@JakeSCahill

@JakeSCahill JakeSCahill commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Resolves DOC-1910: documents the Kubernetes deployment telemetry that shipped in Operator/chart v26.1.1 (ENG-358). Until now the K8s docs had zero coverage of usage statistics reporting - the chart's own values.yaml still links to a stats-reporting docs anchor that no longer exists.

Changes

Adds a Usage statistics reporting section to the shared monitor-redpanda.adoc partial, so it renders on both the Linux and Kubernetes monitoring pages with platform-scoped content:

  • Both platforms: what the 24-hour report to Redpanda Data contains (aggregate counts, versions, configuration - not topic names, message data, or client identifiers), and the enable_metrics_reporter / metrics_reporter_report_interval properties that control it.
  • Kubernetes only (ifdef::env-kubernetes): the six REDPANDA_METRICS_K8S_* environment variables the chart and operator inject since v26.1.1, including the detection caveats (no Azure/AKS detection, cluster ID requires reading the kube-system namespace), a note that Console reports separately, a kubectl exec command to inspect what a deployment reports, and Operator/Helm tabs for disabling.
  • The docs distinguish the two opt-outs, which prior internal notes conflated: config.cluster.enable_metrics_reporter: false disables all reporting, while logging.usageStats.enabled: false omits only the Kubernetes metadata.

Verification

Everything was verified against source rather than the ticket's April research report:

  • charts/redpanda/metrics.go on redpanda-operator main: env var names, both gating conditions, GKE/EKS detection, kube-system UID lookup.
  • src/v/cluster/metrics_reporter.cc at redpanda v26.2.1: all six env vars are read into the report payload; report contents confirmed from the payload serializer.
  • Introduced-in version confirmed from git history: metrics.go first shipped in operator/v26.1.1.
  • Rendered the published 26.2.1 chart (helm template): default injection on the redpanda container, GCP detection with a GKE kube-version and no ENVIRONMENT var with a plain one, and both documented opt-outs remove the variables. CLUSTER_ID is absent in offline renders only because lookup needs a live cluster.
  • Full local Antora build passes. Rendered HTML spot checks: the section renders on both pages, the K8s table and tabs appear only on the K8s page, the rpk command only on the Linux page, and the property xrefs resolve.

🤖 Generated with Claude Code

…etadata (DOC-1910)

Add a usage statistics reporting section to the shared monitoring partial.
It covers what the 24-hour report to Redpanda Data contains, the Kubernetes
deployment metadata that chart and operator versions 26.1.1 and later inject
through environment variables, and how to disable reporting on each platform.

Verified against redpanda-operator main (charts/redpanda/metrics.go) and
redpanda v26.2.1 (src/v/cluster/metrics_reporter.cc). Both opt-outs and the
default injection validated by rendering the published 26.2.1 chart.
@JakeSCahill
JakeSCahill requested a review from a team as a code owner August 6, 2026 08:05
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit f6e20ed
🔍 Latest deploy log https://app.netlify.com/projects/redpanda-docs-preview/deploys/6a778b3671b7d8000806def9
😎 Deploy Preview https://deploy-preview-1885--redpanda-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ff8cfef3-5356-4f28-a99b-6257a484b5f6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Added a table-of-contents link and a “Usage statistics reporting” section. The documentation describes the default reporting interval, report contents, configuration properties, Kubernetes metadata, inspection commands, and reporting controls. It includes Operator, Helm, and rpk procedures for disabling reporting or excluding Kubernetes metadata while retaining base reporting.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Suggested reviewers: feediver1, kbatuigas, michelerp, vbotbuildovich

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the documentation change and its focus on usage statistics reporting and Kubernetes deployment metadata.
Description check ✅ Passed The description explains the purpose, ticket, detailed changes, verification steps, and affected platforms, but omits template sections for review deadline, page previews, and checks.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch DOC-1910-k8s-telemetry

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@modules/manage/partials/monitor-redpanda.adoc`:
- Around line 282-286: Update the introductory sentence above the kubectl
command to describe it as inspecting the Kubernetes metadata environment
variables configured for the Redpanda Pod, rather than claiming it shows exactly
what the deployment reports. Do not imply that the command displays reporter
state, usage reports, or data sent to Redpanda Data.
🪄 Autofix

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 Plus

Run ID: ecf691b8-f2f5-478b-b6e2-e69cca1990f4

📥 Commits

Reviewing files that changed from the base of the PR and between 62d6494 and 3cab18b.

📒 Files selected for processing (1)
  • modules/manage/partials/monitor-redpanda.adoc

Comment thread modules/manage/partials/monitor-redpanda.adoc Outdated
Review finding: "To inspect exactly what your deployment reports"
overclaimed. The kubectl exec command greps only the
REDPANDA_METRICS_K8S_* environment variables on the redpanda container;
it does not show the assembled usage report or the metrics reporter
state. Reworded to describe the environment-variable inspection.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

1 participant