-
Notifications
You must be signed in to change notification settings - Fork 283
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
initial studio agent, client awareness and operation reporting #309
Merged
Merged
Changes from 78 commits
Commits
Show all changes
105 commits
Select commit
Hold shift + click to select a range
f4d71ae
Start exploring options around a studio agent
garypen b03ead8
Merge branch 'main' into usage-agent
garypen a1b6439
Experiment some more with various tracing options
garypen 50a24d9
Merge branch 'main' into usage-agent
garypen 083ca02
Some hackery to figure out what is happening
garypen 4822655
Start integrating apollo telemetry client with router
garypen e5b8f2a
Add enough functionality to automate sending some trace data
garypen 6ff030d
Focus efforts on the OpenTelemetry approach
garypen faa3272
Merge branch 'main' into usage-agent
garypen a63748f
Clean up integration of Apollo OT tracing
garypen 85da9eb
Add usage_agent dependency to github repo
garypen ceff8dc
Revise git dependency for usage_agent
garypen 5c03ffc
Make cargo deny happy
garypen 06a6117
Add a usage server to the router
garypen f84cdb1
Update versions to pickup usage-agent changes
garypen c839ec9
Migrate usage-agent to router repo
garypen 9280c35
Try to fix licensing complaints
garypen 8def90a
Merge branch 'main' into usage-agent
garypen d60aaea
Pin encoding_rs to 0.8.29 and add usage_agent to deny.toml
garypen de28a3b
Further investigation of work for studio agent functionality
garypen 8abaebc
Fix some complaints about licensing and clippy
garypen de78fa0
Merge branch 'main' into garypen/66-usage-agent
garypen 24836db
Move telemetry from prepare to execute
garypen de66553
Throw away the custom logging investigation
garypen 1e374b3
Start thinking about how to propagate client information
garypen 9715096
Clean up some of the logging and fake statistics
garypen d0b0d45
Replace use of libflate with flate2
garypen 96cea73
Make sure to seek before truncating
garypen 31e2520
Add some sleeping to see if that helps the linux tests to pass
garypen 1a339dc
Make clear what the call to set_global_tracer() is returning
garypen 29a8f42
Merge branch 'main' into garypen/66-usage-agent
garypen 4c370f3
Add support for configurable collectors
garypen 348cb81
Merge branch 'main' into garypen/66-usage-agent
garypen b33fb06
Do some snapshot updates and tweaking to help cargo xtask all to pass
garypen 4548890
Enable client detail recording
garypen 084f55c
More cleaning up of code and make agent work with jaeger
garypen 63391e4
Re-structure code in bid to figure out what is causing hangs
garypen 8a049fa
resolve issues with configuration change responses
garypen b00bdda
some polishing
garypen 7e76cf0
Merge branch 'main' into garypen/66-usage-agent
garypen 91d6231
Improve internal server robustness
garypen f8bcdc9
Add "uname" support for windows
garypen 52c7e09
pull in code review changes for 331/337
garypen fd82b60
Merge branch 'main' into garypen/66-usage-agent
garypen ca8f210
Remove hard-coded addresses and improve configurability
garypen eaa97d8
Add support for studio integration into router configuration
garypen 297d8d1
flush opentelemtry events and clean up some code names
garypen 346cc77
relayer not required if we don't have graph configuration
garypen 244f7a9
add support for retries and triggering transfers
garypen 663ce7b
rename usage-agent to apollo-relay
garypen e208be1
format the changes I made
garypen 5cefecf
Fix enough stuff to keep xtask all happy
garypen 8cd1ee4
rename server to relay
garypen ad289e3
Commit to renaming agent/server to relay
garypen 9819840
Improve performance by lowering synchronisation
garypen d8d630b
Cleanup reqwest interaction a little
garypen e705d6f
Merge branch 'main' into garypen/66-usage-agent
garypen c0fafa9
Add support for overriding apollo ingress
garypen b99d796
code review feedback on backoff Duration
garypen eee04d9
Introduce DurationHistory struct to help with bucketing
garypen 8cd56ec
port existing TS tests for DurationHistogram
garypen f0e4bd8
Merge branch 'main' into garypen/66-usage-agent
garypen 99526d0
Improve normalization error reporting
garypen 87314b7
fix bug where usage data is double transferred
garypen b2a5c28
rename relay to spaceport
garypen 949bf03
improve comments and clean up some redundant cloning
garypen aaf8ba0
Improve performance and scalability of telemetry
garypen 9e73d28
Merge branch 'main' into garypen/66-usage-agent
garypen 3ffff59
remove comment and stop producing json for tracing subscribers
garypen 8a86a88
move many spans from info to trace
garypen b87ff86
Move the telemetry recording a bit further up the stack
garypen 3f873b6
Merge branch 'main' into garypen/66-usage-agent
garypen b2afdc0
Odd, merged with main but Cargo.lock hasn't been updated
garypen 0d86b50
Odd, why is licenses.html not updated
garypen 92988c6
don't report studio keys on debug output
garypen a2bb359
Final round of cleanup before starting first code review
garypen 7026a33
Merge branch 'main' into garypen/66-usage-agent
garypen 27caf08
some small changes to fit in with code review comments
garypen 25ad732
review comment
garypen 6c5be65
more review comments
garypen 6ff86de
review comments
garypen a7d7a59
code review comments lead to spaceport re-structure
garypen 220ccaa
address snapshot review comment
garypen 9f2e421
Merge branch 'main' into garypen/66-usage-agent
garypen fd7f130
reverse out snapshot level change (back to DEBUG)
garypen 6da5f92
use the same cargo versioning approach as other components
garypen 7da9dcc
remember to fix licenses.html after changing cargo.toml
garypen 78d2004
bryn's code review comment about coud location
garypen 07001d2
improve transfer triggering
garypen 1b15dad
Merge branch 'main' into garypen/66-usage-agent
garypen f1a78a9
remove snapshots which aren't on main
garypen 13565b6
remove all snapshots not on main
garypen 1f6ef44
merged with main so update licenses.html
garypen 03d809b
review comments
garypen d5b2d7f
more review comments
garypen 2b01014
code review comments
garypen 2b78f00
add support for apollo telemetry when OTLP tracing enabled
garypen af58cb9
fix various strange errors relating to insta snapshots
garypen 92b6df9
review comments
garypen d6fb85e
review comments
garypen 2e3efba
remove op_name from normalized query cache
garypen adb2e7b
review comment to remove Ingress from a comment
garypen 042f837
review comments
garypen b9ed1b8
address review comments
garypen 2c83393
update comments around set global tracer
garypen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| rust 1.57.0 | ||
| rust 1.58.1 | ||
|
|
||
| nodejs 16.9.1 |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the name <3 🚀