Skip to content

docs: update zipkin plugin documentation#13277

Merged
Yilialinn merged 2 commits into
apache:masterfrom
Yilialinn:docs/update-zipkin
Apr 29, 2026
Merged

docs: update zipkin plugin documentation#13277
Yilialinn merged 2 commits into
apache:masterfrom
Yilialinn:docs/update-zipkin

Conversation

@Yilialinn
Copy link
Copy Markdown
Contributor

Summary

  • Fix typos in the EN doc where the plugin name zipkin was accidentally truncated to kin in the "Send Traces to Jaeger" and "Using Trace Variables in Logging" sections (route ID, plugin key, section header, and variable names)
  • Add admin_key note block before the first Admin API curl example in both EN and ZH docs

- Fix typos where 'zipkin' was truncated to 'kin' in EN doc (route ID,
  plugin key, section heading, and variable names)
- Add admin_key note block before first curl example in EN and ZH docs
@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. doc Documentation things labels Apr 21, 2026
@Yilialinn Yilialinn requested a review from Copilot April 22, 2026 06:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Zipkin plugin documentation to correct references that were mistakenly truncated and to make the Admin API examples easier to follow by explicitly showing how to obtain admin_key.

Changes:

  • Added an admin_key note block before the first Admin API curl example in both EN and ZH Zipkin plugin docs.
  • Fixed truncated kin references in the EN Zipkin doc (route id, plugin key, section wording, and trace variable names).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/en/latest/plugins/zipkin.md Adds admin_key note and corrects kinzipkin references in Jaeger/logging sections.
docs/zh/latest/plugins/zipkin.md Adds admin_key note before the first Admin API example.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/en/latest/plugins/zipkin.md
@Yilialinn Yilialinn requested a review from kayx23 April 22, 2026 08:32
@Yilialinn Yilialinn merged commit 39fa390 into apache:master Apr 29, 2026
7 checks passed
@Yilialinn Yilialinn deleted the docs/update-zipkin branch April 29, 2026 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Documentation things size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants