-
Notifications
You must be signed in to change notification settings - Fork 580
feat(koa): Adds support to ignore a span by its layer name #2028
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
base: main
Are you sure you want to change the base?
Conversation
|
4623e18
to
47bed61
Compare
f41b872
to
a5b3a6e
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2028 +/- ##
==========================================
+ Coverage 89.68% 89.70% +0.01%
==========================================
Files 187 187
Lines 9059 9082 +23
Branches 1858 1865 +7
==========================================
+ Hits 8125 8147 +22
- Misses 934 935 +1
🚀 New features to boost your workflow:
|
882914e
to
57512e3
Compare
57512e3
to
d5e6c1d
Compare
d5e6c1d
to
5cb51b8
Compare
8e3b4df
to
fe52615
Compare
f0e8dd2
to
e2b7711
Compare
ff73355
to
1dce93d
Compare
882121f
to
f7565e7
Compare
f7565e7
to
8dfc847
Compare
f7079e3
to
a7ac489
Compare
c2119dc
to
1bc39fb
Compare
4ebd167
to
7d620ea
Compare
a4ff4a8
to
9dd90be
Compare
9dd90be
to
a8f600a
Compare
a8f600a
to
20aed9b
Compare
0d877ed
to
887f208
Compare
e197f74
to
edbaf52
Compare
edbaf52
to
f542500
Compare
f542500
to
002a7f5
Compare
002a7f5
to
b93b68e
Compare
@trentm - I saw you reviewed this PR before - however this PR is targeting an unmaintained component. Are you sponsoring this feature? 🤔 If yes, please apply the |
ac13a3c
to
a474f5c
Compare
2894823
to
50ed50a
Compare
50ed50a
to
7f9737f
Compare
Which problem is this PR solving?
ignoreLayersName
option to instrumentation constructor config to allow layers to be ignored by its name.ignoreLayersType
config, this new one will run after this check.Short description of the changes
KoaInstrumentationConfig
withignoreLayersName
which is an array ofstring
(layer names):isLayerNameIgnored
to check if a layer should be ignored by the name_patchLayer
to check if the layer is ignored by the config