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.

Sign up for GitHub

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

Jump to bottom

Add more OTEL context to GQL resolver errors #1193

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Add more OTEL context to GQL resolver errors #1193

merged 1 commit into from
Nov 5, 2024

Conversation

myieye
Copy link
Contributor

@myieye myieye commented Oct 31, 2024 •
edited
Loading

I ran into errors (I can't remember which ones 😬) where I wanted more context, so I think this might help in the future.

@myieye Graphite App
Copy link
Contributor Author

myieye commented Oct 31, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @myieye and the rest of your teammates on Graphite Graphite

@myieye myieye marked this pull request as ready for review October 31, 2024 10:55
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 31, 2024 •
edited
Loading

C# Unit Tests

75 tests  ±0   75 ✅ ±0   5s ⏱️ ±0s
13 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 39de4be. ± Comparison against base commit ad076a1.

♻️ This comment has been updated with latest results.

@myieye myieye force-pushed the gt/10-31-add_more_otel_context_to_gql_resolver_errors branch from 6c2b5a6 to 39de4be Compare November 5, 2024 09:42
@myieye Graphite App
Copy link
Contributor Author

myieye commented Nov 5, 2024 •
edited
Loading

Merge activity

  • Nov 5, 4:42 AM EST: Graphite rebased this pull request as part of a merge.
  • Nov 5, 4:46 AM EST: A user merged this pull request with Graphite.

@myieye myieye merged commit b5803ed into develop Nov 5, 2024
8 checks passed
@myieye myieye deleted the gt/10-31-add_more_otel_context_to_gql_resolver_errors branch November 5, 2024 09:46
@myieye myieye mentioned this pull request Nov 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hahn-kev hahn-kev hahn-kev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@myieye @hahn-kev