docs: update observability/telemetry overview pages #6563
Conversation
✅ Docs preview readyThe preview is ready to be viewed. View the preview File Changes 1 new, 0 changed, 0 removedBuild ID: 54a6911a7d270c0c410cfb53 URL: https://www.apollographql.com/docs/deploy-preview/54a6911a7d270c0c410cfb53 |
|
@Meschreiber, please consider creating a changeset entry in |
|
!docs set-base-branch main |
Meschreiber
changed the title
|
!docs set-base-branch main |
Co-authored-by: Edward Huang <[email protected]>
Co-authored-by: Edward Huang <[email protected]>
| Map GraphQL error data to OTel errors with the following configuration: | ||
|
|
||
| ```yaml title="router.yaml" showLineNumbers=false | ||
| telemetry: |
There was a problem hiding this comment.
Hmm, it seems the commit that adds these configs is causing Circle CI tests that validate configs to fail.
When testing them in VS Code with router config awareness, they didn't throw any issues though. 🤔
There was a problem hiding this comment.
How sensitive are the tests to whitespace? For example, this bit:
static: error
condition:
eq:
- true
- on_graphql_error: true
looks indented a bit more than the rest
|
@Mergifyio backport 1.x |
✅ Backports have been created
|
Co-authored-by: Edward Huang <[email protected]> Co-authored-by: Shane Myrick <[email protected]> (cherry picked from commit e5359ca) # Conflicts: # docs/source/reference/router/telemetry/instrumentation/standard-instruments.mdx
…6750) Co-authored-by: Maria Elisabeth Schreiber <[email protected]>
This PR: