Skip to content
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

[ja] update content/ja/docs/concepts/context-propagation page #6283

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Msksgm
Copy link
Contributor

@Msksgm Msksgm commented Feb 11, 2025

I updated content/ja/docs/concepts/context-propagation page because it is outdated.
I just fix outdated links in this page.

This issue is related to #6172.

> npm run check:i18n -- -d content/ja/docs/concepts/context-propagation.md

> check:i18n
> scripts/check-i18n.sh -d content/ja/docs/concepts/context-propagation.md

Processing paths: content/ja/docs/concepts/context-propagation.md
diff --git a/content/en/docs/concepts/context-propagation.md b/content/en/docs/concepts/context-propagation.md
index 31537ffd..d7134048 100644
--- a/content/en/docs/concepts/context-propagation.md
+++ b/content/en/docs/concepts/context-propagation.md
@@ -4,11 +4,11 @@ weight: 10
 description: Learn about the concept that enables Distributed Tracing.
 ---

-With context propagation, [Signals](/docs/concepts/signals) can be correlated
-with each other, regardless of where they are generated. Although not limited to
-tracing, context propagation allows [traces](/docs/concepts/signals/traces) to
-build causal information about a system across services that are arbitrarily
-distributed across process and network boundaries.
+With context propagation, [signals](../signals/) can be correlated with each
+other, regardless of where they are generated. Although not limited to tracing,
+context propagation allows [traces](../signals/traces/) to build causal
+information about a system across services that are arbitrarily distributed
+across process and network boundaries.

 To understand context propagation, you need to understand two separate concepts:
 context and propagation.
@@ -33,7 +33,7 @@ information to be propagated from one service to another.

 Propagation is usually handled by instrumentation libraries and is transparent
 to the user. In the event that you need to manually propagate context, you can
-use Propagation APIs.
+use the [Propagators API](/docs/specs/otel/context/api-propagators/).

 OpenTelemetry maintains several official propagators. The default propagator is
 using the headers specified by theDRIFTED files: 1 out of 1

@opentelemetrybot opentelemetrybot requested review from a team February 11, 2025 07:25
@Msksgm Msksgm marked this pull request as ready for review February 11, 2025 07:29
@Msksgm Msksgm requested a review from a team as a code owner February 11, 2025 07:29
Copy link
Contributor

@ymotongpoo ymotongpoo left a comment

Choose a reason for hiding this comment

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

LGTM

c.f. #6172

@ymotongpoo ymotongpoo added the docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer label Feb 13, 2025
@ymotongpoo
Copy link
Contributor

@Msksgm thank you for your contribution. if you are to contribute more for the fix of obsolete docs, can you add checks on the list on #6172

@ymotongpoo ymotongpoo force-pushed the update-#6172/ja/docs/concepts/context-propagation branch from 51ca55d to a48a152 Compare February 13, 2025 07:02
@opentelemetrybot opentelemetrybot requested a review from a team February 13, 2025 07:03
@Msksgm
Copy link
Contributor Author

Msksgm commented Feb 13, 2025

@Msksgm thank you for your contribution. if you are to contribute more for the fix of obsolete docs, can you add checks on the list on #6172

@ymotongpoo

Thank you for a lot of your review.
I would like to want to do other obsolete documentations.
However, I cannot add checks on the list on #6172 due to the permissions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer lang:ja
Projects
Development

Successfully merging this pull request may close these issues.

2 participants