This repository was archived by the owner on Nov 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Telemetry per node #7463
Merged
Merged
Telemetry per node #7463
Changes from 47 commits
Commits
Show all changes
277 commits
Select commit
Hold shift + click to select a range
59429ab
WIP
cecton c7e824a
Serialize instead of stringify
cecton 28a79a0
WIP
cecton 6ac4ea8
WIP
cecton 6d6da6d
CLEANUP
cecton e1bd592
CLEANUP
cecton 389da31
CLEANUP
cecton 4602bcf
Re-using telemetry nodes
cecton aadae1c
Refactoring guarding object
cecton 520c38b
Inject span id in the json (not elegant)
cecton fe28d76
Merge commit 3664cdbc92b13d7197fbf2a7fa2b0cd6201eba09 (no conflict)
cecton ab089f9
Merge commit cab986549f964a081343336797bb6cf6b3526335 (conflicts)
cecton 7385c62
Fix passing transport argument
cecton cc41d72
CLEANUP
cecton a5c64fd
Spawn 2 nodes instead of one
cecton 79f05ac
WIP
cecton 24f3c0d
WIP
cecton 1476571
WIP
cecton 295a18e
Use proper FormatEvent in the browser
cecton 2f40937
Smol fix
cecton b17aa9d
CLEANUP
cecton a8c6e8c
Removed Arc<Mutex<...>> in Telemetries
cecton 1b83b6f
CLEANUP
cecton 66a2db9
browser: share transport instance
cecton 0320c95
Telemetries still need to be clonable to pass it to other nodes
cecton 76f12bb
Move logging.rs to its own crate sc-logging
cecton 12b7d52
Split module to pieces RIP
cecton e49e181
rustfmt
cecton 6416651
Fix tests
cecton 00f34bd
Revert "Spawn 2 nodes instead of one"
cecton 84c3fac
Merge commit 095d0f06a42b66a92f28d097499a69e3f170d59d (no conflict)
cecton 6960f4f
Doc
cecton 0708147
Replace Vec by Iterator
cecton 4b6091c
Doc
cecton fa4265e
Revert "Replace Vec by Iterator"
cecton 33268f1
Solving weird bug
cecton 4522330
Clever trick taken from jsonrpc-core to circumvent the deserializing …
cecton dc5cfb0
Same trick to fix serialization issue in tests
cecton 326feb5
Fixing again
cecton d095426
Merge commit e26a46a805d68ce55457d67552ba71b76421f708 (no conflict)
cecton 2ecb656
Merge commit 02771e146f126a9759a62948ec8a2edd6ad47ee9 (conflicts)
cecton bda2398
Merge commit 99602cda4e1c06d66a40aa933789308d4ca9a7ce (no conflict)
cecton 15a1b9b
Merge commit 08b217a0650635f8430655979c581f0cebe4bed3 (conflicts)
cecton fe918f5
Merge commit 407cd3af06b03201a506b645c9ee587e5d4748bb (no conflict)
cecton deb7c42
Cargo.lock
cecton 6fc89f9
doc
cecton 2f9fcb0
Move sc-cli-proc-macro to sc-logging-proc-macro
cecton bed3d31
Make prefix_logs_with accessible also from sc-cli
cecton 64a7627
Doc
cecton b5420e6
doc
cecton 388bafd
README.md
cecton a177df4
Removed some not working logs (not too important either)
cecton 53f91ba
Use tracing Id instead of u64
cecton ad70639
Doc
cecton c83849d
Doc and rename init function
cecton af78842
Clean-up & better error code
cecton b110d38
CLEANUP
cecton f26db51
Renamed invalid name for creating Telemetry
cecton d791b48
Move stuff around
cecton 0fbf1a9
Comment
cecton 647b4a4
Doc
cecton 448c8a1
Unneeded clone
cecton aa5fdbe
Renamed message_verbosity to verbosity
cecton db42f20
Change match to if let
cecton 98c791f
Unneeded clone
cecton 537d882
Fix doc
cecton 5ec5792
Apply suggestions from code review
cecton a0907d5
Merge commit c180950de63dbdfaa11053a8a56f35e694ed9e36 (no conflict)
cecton 5f68c2b
Merge remote-tracking branch 'origin/master' into cecton-telemetry-wi…
cecton 2ce02f8
Unused variable in test
cecton 8e1a83e
Renamed init_logger to init_logging_and_telemetry
cecton 75672d9
English fixes
cecton 1fe185b
Improve error message
cecton efbbf9e
Removed unused_crate_dependencies
cecton de15a04
Move sc-logging to a tracing module
cecton e1650bf
Apply suggestion
cecton 96730ca
Apply suggestions
cecton 86929c3
Apply suggestion
cecton e10fbc2
Apply suggestion
cecton 162bf74
Apply suggestion
cecton 7b1b5ac
More explanation
cecton 5afe364
Fix doc comment
cecton f3c2786
Fixes
cecton ca23c7a
Fix test
cecton 717e2cd
Missing dependency
cecton 39306ca
Fix wrong path
cecton f87ed55
Write to stderr without panicking
cecton 00b7015
Merge commit 1c122c6d138f7f58f1ed7ce1414cf3dd283cd4ce (no conflict)
cecton e8e0d23
Merge commit a0e8b7ecd1d1d44bfdad1e67ef4028121f7d9d82 (conflicts)
cecton de0e378
Merge commit 0840c58849bc84e7fd72627745bda243741adcd8 (no conflict)
cecton 5bf08e2
Merge commit d021078200021b1fcae1aa8434282edd9e6a8f4b (conflicts)
cecton 70f016f
Merge commit 7e94f01af67c498605eb0206b7fca261f12a278c (no conflict)
cecton d9f4ef6
Merge commit 31733d5075bdba62df00975df0463a56fadbcaa1 (conflicts)
cecton efd8a3b
Merge commit 1ec8a7f57ba5faac74c09296eab9977fb331c386 (no conflict)
cecton 02d196e
Fix another issue with arbitrary_precision
cecton 21c8e05
Fixed test to use sc_cli
cecton 22f72fa
Update README.md
cecton f44dd19
Remove arbitrary_precision
cecton d208832
Do not panic if serialization fail during runtime
cecton b99af5c
Fix for lack of arbitrary_precision in serde_json
cecton 5a0e055
Fix incorrect doc
cecton 2bc432e
Rework doc
cecton b47aaf9
Spelling and small change
cecton 887a526
Improve doc again
cecton b11d95e
Drop unused "level" in payload
cecton e5c74e5
WIP Refactor to single worker
cecton f36606d
WIP
cecton fc96779
WIP
cecton 89d6bd2
WIP
cecton ee63974
WIP
cecton df1778d
WIP
cecton b67ff7d
WIP
cecton 3f6f8eb
WIP
cecton 20e0f42
WIP
cecton 5cf613f
Using Arc<Mutex<>>
cecton 8addedd
Revert "Using Arc<Mutex<>>"
cecton 1b4762a
WIP
cecton bec2d67
WIP
cecton 2b49cab
CLEANUP
cecton 64b5a6e
Apply suggestions from code review
cecton d2a8d42
Merge commit de44c00109ea19de13ef9219f8888f968d484b55 (no conflict)
cecton 486012a
Merge commit d428fc79a52fce1255442ed84ec42415aea487bf (conflicts)
cecton cec8774
Merge commit 72cfd777812d1c374265d17524bec63ee5f07d69 (no conflict)
cecton 9102901
Merge commit 9450a9d29156a80f48947fdf650396f488ca1c32 (conflicts)
cecton 0f8cff5
Merge commit ace7f47735a6cb87580513d4eaa6bc7c7c398bd4 (no conflict)
cecton a3a72de
Merge commit b63b8643e00214f1cbe28b6ca4e804af656c9038 (conflicts)
cecton b699ea0
Merge commit 77f1089942b7f1190652887abb1246ea09dff894 (no conflict)
cecton afd7b36
Merge commit f4d4244ed6f09675f09c6e1afa96c2b595a689bf (conflicts)
cecton 8748b46
Merge commit 675cbe9ac507a4f75bd0105ce02b00b772140c6b (no conflict)
cecton 8e8bc3a
Merge commit 0dfdc328fa2b9514847b354e6968553a9e29d61e (conflicts)
cecton ca3da6d
WIP
cecton 0817b94
Merge commit f73ac5eb0df68b72d42d25faf4da411f2d3581b6 (no conflict)
cecton 0a8d616
WIP
cecton 68be858
Debug
cecton 07d8ae5
WIP
cecton 42e0d9a
WIP
cecton 5fe0fe2
WIP
cecton 2ec34b0
CLEANUP
cecton 7f19841
CLEANUP
cecton f433698
CLEANUP
cecton 8dc614b
CLEANUP
cecton ddb71ef
WIP
cecton 49e9434
WIP
cecton ba4274f
CLEANUP
cecton b49a0f4
CLEANUP
cecton eb9aed3
Revert "Make prefix_logs_with accessible also from sc-cli"
cecton 6cf9af2
Add check for first character in json attribute
cecton ef26df6
WIP
cecton 32b0b14
Remove passing around of WASM transport
cecton 3c12ba7
Fix previous commit
cecton 834eac4
Remove passing around of WASM transport (sc-network)
cecton a63bb25
Removed useless proof
cecton 2c24ccb
Revert "Remove passing around of WASM transport (sc-network)"
cecton bd687ef
Revert "Fix previous commit"
cecton f7e88fa
Revert "Remove passing around of WASM transport"
cecton 5140912
Re-create transport instead of passing it around
cecton 0c41445
WIP
cecton 0551a6b
Merge commit 4b8c862ae3b0d34ecdc351888469f81704df68e2 (no conflict)
cecton f0b69ed
Merge commit 9599f39a15007eba3bcc38d0eddb8af08de2b2b9 (conflicts)
cecton 33156f8
Merge commit be8fd0588013a3b8eab5d5c761fe1c55bdd60dbb (no conflict)
cecton 9ed09d2
Merge commit 661bbc9a73107d144aaea80e4ed0e4b54a5b2a96 (conflicts)
cecton 9eb8b53
Merge commit b1ef77071f9b085777b4cef08a2348b762131003 (no conflict)
cecton 48b2c77
Merge commit 2a7a985d34fea5db68f12e9c1fbde8142de27cdc (conflicts)
cecton 2f54638
Merge commit 987a9723920217917f2708388d150add5ef52ef7 (no conflict)
cecton 12740fe
CLEANUP
cecton a04451d
Renamed Telemetries to TelemetryWorker
cecton 7b3cdb3
doc
cecton 7ef81dc
Renamed TelemetryConnectionSinks to TelemetryConnectionNotifier
cecton dfa0dab
Refactor initialization and doc
cecton e044c0e
Re-organize
cecton af28034
doc
cecton 281cacb
Fix, clean-up & re-organize
cecton 566d95a
Use a builder pattern for global logger and remove tracing dependencies
cecton bb926c0
Rename to GlobalLoggerBuilder
cecton fe160aa
Refactor a bit
cecton f2667be
Handle errors better
cecton 4e1de2a
CLEANUP
cecton 074fa7e
CLEANUP
cecton 70e1ac5
CLEANUP
cecton 6130ee3
Refactor to remove Arc Mutex
cecton 7a3fb6e
WIP
cecton 1c5860d
formatting
cecton acdf211
Fix tests
cecton fc39a08
Fix failing test
cecton 9be9560
Re-export global logger builder in sc-cli
cecton 28916cb
Fix ci issue?
cecton 787ecba
Update doc
cecton d2fcfcf
Simplify code a tad bit
cecton 8e7d3cc
Update README.md
cecton 4d52c2e
Add comment
cecton d670efe
CLEANUP
cecton 9c26aa9
Apply suggestions from code review
cecton f4e4749
Fix after suggestion
cecton 9ba4bff
Group hashmaps
cecton a6fce44
Indent
cecton 6521813
Moving down
cecton b1b5545
Simplify json object creation
cecton 37e045e
Doc
cecton 79a04e2
Ordering stuff
cecton fa5ea74
Indent
cecton 0a8ca91
Remove TelemetryError
cecton 8dbc29f
Merge commit f1d8be84f443b9c6198d8c788118e30ef6e2a646 (no conflict)
cecton e2b4ac1
Merge commit 6abbbd639d07f041255fb326491fea27ece8a490 (conflicts)
cecton 025c2a7
Merge commit 5215cd71aed415465e4d25926d426900a0b1f29a (no conflict)
cecton d8902f2
Merge commit 9b08105b8c7106d723c4f470304ad9e2868569d9 (conflicts)
cecton 4fb6dbe
Merge commit 89275433863532d797318b75bb5321af098fea7c (no conflict)
cecton eaed136
Merge commit 762f4b0242bb9d14fa0df5b6a1a3a64b005268e4 (no conflict)
cecton 50a59f3
Merge commit e5e7606fa54175dfc424b7ca90d60b07a3c3f026 (conflicts)
cecton 0db22eb
Merge commit b27503591d019b94a0eea7510578dadc5ad3196c (no conflict)
cecton ddb53d0
New payload format & remove ts
cecton 966fd4c
Replace dispatcher with a simple loop
cecton 5c733cc
Re-introduce ts field as suggested
cecton f6fa4a1
Separate span from TelemetryHandle so we can re-enter at will
cecton 494c024
Fix: replace std Instant not working in WASM
cecton 280a5b6
Revert test on browser
cecton d4d1135
Revert "Revert "Spawn 2 nodes instead of one""
cecton 351e3f1
Allows getting a new telemetry handle from the runner
cecton 47132f9
Fix test code
cecton 8295bc1
Revert test code
cecton 8f77aa0
Fix tests
cecton 0bd4ce7
Fix more tests
cecton 5eb2753
Merge commit c9d93653e567f10867273b0171f3025419795c37 (no conflict)
cecton 2e95044
Merge commit 5f3db3359ae822a1ea610059e93976c6b3dfcdde (no conflict)
cecton c34c3db
Merge commit fa404167bbfe7bfd459689d25fc19b86da698367 (conflicts)
cecton f7750a6
Doc & warn missing doc
cecton 7770d46
Update client/tracing/src/logging/directives.rs
cecton b33628b
Specialized struct for the connection message
cecton 853eb42
Update client/telemetry/src/lib.rs
cecton a6fdb28
Update client/telemetry/src/lib.rs
cecton ab0e78d
Update client/telemetry/src/lib.rs
cecton 007a41a
Update client/telemetry/src/lib.rs
cecton 3679140
Removed unneeded code
cecton 0cbf0b8
Revert "Update client/telemetry/src/lib.rs"
cecton f1aa1a4
CLEANUP
cecton a626cec
Simplify stream
cecton 09be76a
WIP
cecton 4501e8b
Test code
cecton 3d35251
Doc & cleanup
cecton eb866c1
Revert "Test code"
cecton 2698ee4
Merge commit f1c67253bb563708c90cec6c49ebcebef4aebb3a (no conflict)
cecton 842c4d7
Merge commit 8ee55dde2eb62f85aa64b54a0814529ea2b8368d (conflicts)
cecton cd716e4
Fix line width
cecton 36ec61b
Added missing code adaptation
cecton def0202
Merge commit 946c0e1749b359ba3889a169079a6296b0717b0f (no conflict)
cecton fbc58c4
Merge commit b64ec19641fea539d56b81466e77552157afe716 (no conflict)
cecton 24596be
Move inner functions out of run function
cecton f6ec36d
Fixed incorrect return of function
cecton f462418
Change error to expect
cecton acf1ec8
Another attempt to have a higher level doc
cecton bc36136
More doc
cecton 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
#!/usr/bin/env sh | ||
cargo +nightly build --release -p node-cli --target wasm32-unknown-unknown --no-default-features --features browser -Z features=itarget | ||
wasm-bindgen ../../../../target/wasm32-unknown-unknown/release/node_cli.wasm --out-dir pkg --target web | ||
python -m http.server 8000 | ||
python -m http.server 8080 | ||
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.
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.
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.
Oops I did it again