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

Move /usage/distributed-tracing to /tracing #9572

Closed
wants to merge 7 commits into from

Conversation

stephanie-anderson
Copy link
Contributor

@stephanie-anderson stephanie-anderson commented Mar 28, 2024

  • changes wording on all platform pages "distributed tracing" -> "tracing"
  • moves contents from /usage/distributed-tracing to /tracing
  • adds redirects where necessary
  • modifies the platform sidebar to render a dedicated section for tracing

ref: #9570

@stephanie-anderson stephanie-anderson requested a review from a team as a code owner March 28, 2024 10:31
Copy link

vercel bot commented Mar 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 28, 2024 3:33pm

Copy link

codecov bot commented Mar 28, 2024

Bundle Report

Changes will decrease total bundle size by 4.05kB ⬇️

Bundle name Size Change
sentry-docs-server 6.11MB 4.04kB ⬇️
sentry-docs-client 5.52MB 6 bytes ⬇️
sentry-docs-edge-server 618.05kB 3 bytes ⬇️

Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

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

Great first step! I am in favor of this - I think we still need to fine tune/change parts where we emphasize the "Distributed" part of Tracing too much.
There are paragraphs where it makes sense, but generally we should just talk about Tracing.

Copy link
Contributor

@lizokm lizokm left a comment

Choose a reason for hiding this comment

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

Made some language suggestions, but THANK YOU so much for making these changes!!

@@ -0,0 +1,34 @@
---
title: Tracing
description: "Learn how to connect events across applications/services."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
description: "Learn how to connect events across applications/services."
description: "Learn how to connect events across multiple applications and services using tracing."

sidebar_order: 40
---

If the overall application landscape that you want to observe with Sentry consists of more than just a single service or application, distributed tracing can add a lot of value.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If the overall application landscape that you want to observe with Sentry consists of more than just a single service or application, distributed tracing can add a lot of value.
Tracing allows you to see interactions across multiple systems. If your application relies on more than one service, setting up tracing can help catch issues earlier and faster by providing a holistic view of your application landscape.


If the overall application landscape that you want to observe with Sentry consists of more than just a single service or application, distributed tracing can add a lot of value.

## What is Tracing?
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## What is Tracing?
## What Is Tracing?

Comment on lines +10 to +14

In the context of a distributed system, tracing acts as a powerful debugging tool. Imagine your application as a vast network of interconnected parts. For example, your system might be spread across different servers or your application might split into different backend and frontend services, each potentially having their own technology stack.

When an error or performance issue occurs, it can be challenging to pinpoint the root cause due to the complexity of such a system. Tracing helps you follow the path of an event as it travels through this intricate web, recording every step it takes. By examining these traces, you can reconstruct the sequence of events leading up to the event of interest, identify the specific components involved, and understand their interactions. This detailed visibility enables you to diagnose and resolve issues more effectively, ultimately improving the reliability and performance of your distributed system.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
In the context of a distributed system, tracing acts as a powerful debugging tool. Imagine your application as a vast network of interconnected parts. For example, your system might be spread across different servers or your application might split into different backend and frontend services, each potentially having their own technology stack.
When an error or performance issue occurs, it can be challenging to pinpoint the root cause due to the complexity of such a system. Tracing helps you follow the path of an event as it travels through this intricate web, recording every step it takes. By examining these traces, you can reconstruct the sequence of events leading up to the event of interest, identify the specific components involved, and understand their interactions. This detailed visibility enables you to diagnose and resolve issues more effectively, ultimately improving the reliability and performance of your distributed system.
If you're working within a distributed system where your application is a vast network of interconnected parts, tracing can be a powerful debugging tool.
Trying to pinpoint the root cause of an error or performance issue in a system that's spread across different servers, or split into different backend and frontend services (each with their own technology stack), can be challenging.
Tracing helps you follow the path of an event as it travels through this intricate web, recording every step it takes. By looking at traces, you can reconstruct the sequence of events that lead up to the event of interest, identify the components involved, and understand their interactions. This detailed visibility lets you diagnose and resolve issues more effectively and improve the overall reliability and performance of your distributed system.


This trace shows a Vue app's `pageload` making a request to a Python backend, which then calls the `/api` endpoint of a Ruby microservice.

What happens in the background is that Sentry uses reads and further propagates two HTTP headers between your applications:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
What happens in the background is that Sentry uses reads and further propagates two HTTP headers between your applications:
In the background, Sentry uses reads and further propagates two HTTP headers between your applications:

@@ -72,7 +72,7 @@ When using custom instrumentation to create a transaction, you can add data to t

Whatever a transaction's sampling decision, that decision will be passed to its child spans and from there to any transactions they subsequently cause in other services.

(See <PlatformLink to="/usage/distributed-tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Tracing</PlatformLink> for more about how that propagation is done.)

@@ -22,7 +22,7 @@ The Cocoa SDK uses [swizzling](https://nshipster.com/method-swizzling/) to provi
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#network-tracking">Auto instrumentation for HTTP requests</PlatformLink>
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#file-io-tracing">Auto instrumentation for File I/O operations</PlatformLink>
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#core-data-tracing">Auto instrumentation for Core Data operations</PlatformLink>
- <PlatformLink to="/usage/distributed-tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for tracing</PlatformLink>

@@ -12,7 +12,7 @@ The Cocoa SDK uses [swizzling](https://nshipster.com/method-swizzling/) to provi
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#network-tracking">Auto instrumentation for HTTP requests</PlatformLink>
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#file-io-tracing">Auto instrumentation for File I/O operations</PlatformLink>
- <PlatformLink to="/performance/instrumentation/automatic-instrumentation/#core-data-tracing">Auto instrumentation for Core Data operations</PlatformLink>
- <PlatformLink to="/usage/distributed-tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for distributed tracing</PlatformLink>
- <PlatformLink to="/tracing/">Automatically added sentry-trace header to HTTP requests for tracing</PlatformLink>

@@ -72,7 +72,7 @@ When using custom instrumentation to create a transaction, you can add data to t

Whatever a transaction's sampling decision, that decision will be passed to its child spans and from there to any transactions they subsequently cause in other services.

(See <PlatformLink to="/usage/distributed-tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Tracing</PlatformLink> for more about how that propagation is done.)

@@ -72,7 +72,7 @@ When using custom instrumentation to create a transaction, you can add data to t

Whatever a transaction's sampling decision, that decision will be passed to its child spans and from there to any transactions they subsequently cause in other services.

(See <PlatformLink to="/usage/distributed-tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
(See <PlatformLink to="/tracing/">Distributed Tracing</PlatformLink> for more about how that propagation is done.)
(See <PlatformLink to="/tracing/">Tracing</PlatformLink> for more about how that propagation is done.)

@HazAT
Copy link
Member

HazAT commented Apr 2, 2024

Sorry, I changed my mind on this - at least dropping the "Distributed" part.
Ultimately, for me personally, I do see Tracing & Distributed Tracing to be the same thing, but I want that if we decide to drop the "Distributed" part, it should be a deliberate global decision.

Some evidence there are lot of pages outside of docs that still describe it as such

https://www.google.com/search?q=sentry+distributed+tracing&oq=sentry+distributed+tracing&gs_lcrp=EgZjaHJvbWUyBggAEEUYOTIGCAEQRRhAMgYIAhBFGDwyBggDEEUYPDIGCAQQRRg80gEINTMwN2owa
jeoAgCwAgA&sourceid=chrome&ie=UTF-8

image (34)

Also most of our competitors call it "Distributed Tracing"

@stephanie-anderson
Copy link
Contributor Author

This PR will be replaced by #9604

@github-actions github-actions bot locked and limited conversation to collaborators Apr 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants