This repository was archived by the owner on Mar 31, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 34
[Tracing] Support Jaeger tracing in NCM and TC #707
Merged
Merged
Changes from all commits
Commits
Show all changes
136 commits
Select commit
Hold shift + click to select a range
47caabf
Tried to make TC's create container process concurrent
zzxgzgz 6a29451
Added back background ping
zzxgzgz 6adc8e0
Changed fixed time sleep to a library(await) that waits until GSs are…
zzxgzgz fbe57f1
Try to add code to send GSV1 to ACA for testing
zzxgzgz 62992b3
Try to add code to send GSV1 to ACA for testing
zzxgzgz f4df134
Sending only gsv1, to see if it works
zzxgzgz 3402d92
Sending only gsv1, to see if it works
zzxgzgz 1378f7a
Sending only gsv1, to see if it works
zzxgzgz 25507e5
Enable gsv2 sending for testing.
zzxgzgz 8918995
Enable gsv2 sending for testing.
zzxgzgz 5fdc054
Enable gsv2 sending for testing.
zzxgzgz 1d4d1c0
Enable gsv2 sending for testing.
zzxgzgz 88d18bc
Test sending v1 and v2 together
zzxgzgz 210ccda
Comment out sending GSV1 code in TC for on-demand performance testing
zzxgzgz c146da5
Start to change TC, and try to figure out why two hosts have differen…
zzxgzgz c2aaab5
Try to make port amount balanced
zzxgzgz 32be97d
Try to add support for different number of ports on two hosts
zzxgzgz d2c5b50
Try to add support for different number of ports on two hosts
zzxgzgz 556217a
Try to add support for different number of ports on two hosts
zzxgzgz e51bc76
Try to add support for different number of ports on two hosts
zzxgzgz 50a13b4
Try to add support for different number of ports on two hosts
zzxgzgz 4afbbe3
Disabling container creating and pings, to focus on testing ACA's abi…
zzxgzgz cf4f34f
Disabling gsv2 printout, also added time for waiting response of gRPC…
zzxgzgz cb97d94
Tried to add router states in TC
zzxgzgz f959c07
Fixed set/add function mismatches
zzxgzgz a17ae63
Added setRevisionNumber for router configuration
zzxgzgz 05c17ad
Added gateway info in the subnet state
zzxgzgz 5358b73
Make host_vdr_mac_address consistent for routers on the same host
zzxgzgz 53f2d9f
Try to make neighbors L3, and added routers to both hosts' GSs
zzxgzgz 3b82784
Added another subnet, so there are now two subnets
zzxgzgz 827b0ab
Added another subnet, so there are now two subnets
zzxgzgz a69951d
Added another subnet, so there are now two subnets
zzxgzgz f604cf8
Change neighbor type to l2 for testing
zzxgzgz ff9626f
Enable container creating and ping for testing
zzxgzgz b39014c
Try to remove extra subnet/router, and focus on testing pings
zzxgzgz 823f79f
Added more wait time for container creating
zzxgzgz d4b9626
Merge branch 'master' into tc_create_containers_concurrently
zzxgzgz 3a3f124
Try to use countdown latch, instead of simply sleep
zzxgzgz a3da687
Changed wait() to await(); fixed typo in comment; deleted unused code
zzxgzgz 8960b45
Removed extra files
zzxgzgz 8b0832c
Added param to let user decide whether to create container and ping
zzxgzgz 0ad576c
Added param to let user decide whether to create container and ping
zzxgzgz 8bad8f4
Created a constant variable to represent the default vlan_id, 1
zzxgzgz 39601b8
Saving code before pulling upstream master
zzxgzgz e7d2bdf
Merge branch 'master' into onNext_slow
zzxgzgz 5bf1618
Added logs to furthur breakdown the time spent in doSendGoalState
zzxgzgz 639c66a
Try to reuse StreamObserver s for gRPC request and reply, to see if i…
zzxgzgz 1a2fa08
Added mutex.unlock() when error happens calling onNext
zzxgzgz c93bb12
Commented out mutexes for debugging
zzxgzgz aff8ac9
Reverted back stream oberver design, added more logs to breakdown the…
zzxgzgz 415c12a
Chnaged implementation for sendGoalStates so that it doesn't wait for…
zzxgzgz 0c55954
Tried to make warmup calls continously for 5 seconds
zzxgzgz 0e2874d
Revert channel warmup method
zzxgzgz 8cfb15d
Revert channel warmup method
zzxgzgz 7d0727a
Investigate why only 10 warmup gs were sent, when 1000 should be sent
zzxgzgz 777a6a3
Tried to use fixedThreadPool
zzxgzgz 92dfb19
Try to make flow control window bigger
zzxgzgz f9208dd
Investigate why multiple warmups can't be received by ACA
zzxgzgz d7a7a50
Try to change default value of numberOfWarmupsPerChannel
zzxgzgz 0e0b13c
Set different version number of warmup gs, to see if it makes a diffe…
zzxgzgz e452f6d
Tried to sleep after onNext for warmups
zzxgzgz 3e5a21f
Sleep doesn't work, try to use old channel builder
zzxgzgz 85e6810
Try to init one request and response observer for each onNext
zzxgzgz bea19eb
Tried to warmup each channel for five seconds
zzxgzgz 3eb10ea
Try to warmup channels at the same time
zzxgzgz 52af123
Added back future to future list
zzxgzgz 83f27d3
Change back to use NettyChannelBuilder, to see if it brings better perf
zzxgzgz 72f976e
Try to trace grpc client for pusing goalstates
zzxgzgz 609f9bd
Investigate on why it complains service name is null/empty
zzxgzgz 318c663
Investigate on why it complains service name is null/empty
zzxgzgz a7398fb
Try to get tracer from env
zzxgzgz e9b9306
Try to add spring.application.name
zzxgzgz 70413cf
Use tracr resolver again
zzxgzgz 46dbb97
Use global tracer
zzxgzgz 19dce9d
Tried to add spans to sendGoalStates
zzxgzgz 52680d9
Tried to add jaeger tracer/span to tc and the grpc server in NCM
zzxgzgz 93fd820
Temperarily execute the ping using ssh
zzxgzgz 10c2787
Try to use different way to get parent span
zzxgzgz 0814050
Added logs to see what kind of tracer and span we got here
zzxgzgz 4ad7610
Try to see what tracer other services have
zzxgzgz a3738f6
Try to get tracer from TracerResolver, and see what is in there
zzxgzgz 4af5450
Removed duplicated dependency
zzxgzgz 75508a0
Disabled grpc and enable rest in NCM
zzxgzgz 2738f5b
Continue investigating the jaeger on NCM
zzxgzgz e7fbee5
Removed default values
zzxgzgz 085f52a
Added spaces around = sign
zzxgzgz 7ae558d
Added more logs
zzxgzgz 9fa6c13
Check if all stuffs in applications.properties doesn't get reflected
zzxgzgz fa3c2d2
Is none of the @Value working
zzxgzgz 2f1a09d
Changed position of @Component
zzxgzgz 4428f6f
Try to get param from ENV
zzxgzgz d03391f
Try to log @Value after construction
zzxgzgz d9487b6
Try to log @Value after construction
zzxgzgz 928be6d
Try to add a tracer after construction
zzxgzgz 86e1c5a
Used configuration to mannually create and register a tracer for NCM
zzxgzgz 81eca68
Added more spans
zzxgzgz 49a7edd
Added span for on-demand reply
zzxgzgz 2a56581
Changed tracer in pseudo controller, also added spans in NCM pushdown…
zzxgzgz 8ae52f7
Changed tracer in pseudo controller, also added spans in NCM pushdown…
zzxgzgz 49a1c74
Changed tracer in pseudo controller, also added spans in NCM pushdown…
zzxgzgz f33b2c1
Added some DurationStatistics
zzxgzgz e78c87b
Added printouts for tracer configs
zzxgzgz 83d1f5f
Added to string
zzxgzgz bd81ece
testing with verbosity for interceptors
zzxgzgz af4ed99
testing with verbosity for interceptors
zzxgzgz 73cd267
testing with streaming for interceptors
zzxgzgz 7237c8c
Try to add scopes for spans
zzxgzgz ac35258
manually added span log for experiement
zzxgzgz bedee17
Added env var JAEGER_SERVICE_NAME in order to let TracerResolver.reso…
zzxgzgz 6d8d34e
Adjusted locations of spans
zzxgzgz 56312a1
Try to make gRPC client use the same tracer as gRPC server
zzxgzgz 3a6dc7c
adjust scope position for ncm's gRPC client
zzxgzgz 1f40ff4
Changed how requestGoalStates get the parent span
zzxgzgz 760a93e
Reset parent span to null
zzxgzgz 0921f44
Reset parent span to null
zzxgzgz d0ae523
Reset parent span to null
zzxgzgz 157b73b
Reset parent span to null
zzxgzgz 5709a6b
Corrected wrong parent spans
zzxgzgz 483fc8c
get back parent span, since it is not the cause of the chained spans
zzxgzgz 07090d7
set parent span to null for requestGoalStates
zzxgzgz cc212dc
get parent spans for requestGoalStates
zzxgzgz e4bee84
Investigate why all requestGoalStates related spans are connected tog…
zzxgzgz 0a8ad33
Added parent span context printing for request goal states
zzxgzgz 97c12ee
Changed to print parent span's tracer id
zzxgzgz 4785a6b
Added span id printing
zzxgzgz 838bca7
Try to move server and tracer initialization back to the constructor
zzxgzgz 737de02
Removed un needed code
zzxgzgz 96979b8
Try to merge master
zzxgzgz a7f7c38
Added back missing onNext(reply)
zzxgzgz 7964bae
Removed unwanted files and code
zzxgzgz 4c99f65
Try to make test controller's tracer report to the jaeger on another …
zzxgzgz 00cee48
Added null check and removed unused code
zzxgzgz bd30110
Added VPC states into cache during pushGoalStatesStream
zzxgzgz 7efc558
Tried to merge from master
zzxgzgz c82daab
Removed local file
zzxgzgz 562e44e
Made span names a part of the application.properties
zzxgzgz 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 hidden or 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
This file contains hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.