-
Notifications
You must be signed in to change notification settings - Fork 292
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
Fix module name detection for headless sessions #7779
Merged
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/module-name-fix
Oct 18, 2024
Merged
Fix module name detection for headless sessions #7779
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/module-name-fix
Oct 18, 2024
Conversation
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
nikita-tkachenko-datadog
added
type: bug
comp: ci visibility
Continuous Integration Visibility
labels
Oct 16, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 9 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.072 s) : 0, 1071754
Total [baseline] (10.399 s) : 0, 10399273
Agent [candidate] (1.077 s) : 0, 1077066
Total [candidate] (10.383 s) : 0, 10382878
section appsec
Agent [baseline] (1.211 s) : 0, 1210920
Total [baseline] (10.608 s) : 0, 10607616
Agent [candidate] (1.207 s) : 0, 1207389
Total [candidate] (10.631 s) : 0, 10630977
section iast
Agent [baseline] (1.197 s) : 0, 1196513
Total [baseline] (10.871 s) : 0, 10870559
Agent [candidate] (1.207 s) : 0, 1207133
Total [candidate] (10.877 s) : 0, 10877140
section profiling
Agent [baseline] (1.27 s) : 0, 1269611
Total [baseline] (10.752 s) : 0, 10751550
Agent [candidate] (1.27 s) : 0, 1270024
Total [candidate] (10.777 s) : 0, 10777037
gantt
title petclinic - break down per module: candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.907 ms) : 0, 682907
BytebuddyAgent [candidate] (687.208 ms) : 0, 687208
GlobalTracer [baseline] (313.175 ms) : 0, 313175
GlobalTracer [candidate] (314.294 ms) : 0, 314294
AppSec [baseline] (53.933 ms) : 0, 53933
AppSec [candidate] (53.78 ms) : 0, 53780
Remote Config [baseline] (664.49 µs) : 0, 664
Remote Config [candidate] (651.255 µs) : 0, 651
Telemetry [baseline] (7.457 ms) : 0, 7457
Telemetry [candidate] (7.438 ms) : 0, 7438
section appsec
BytebuddyAgent [baseline] (702.641 ms) : 0, 702641
BytebuddyAgent [candidate] (700.658 ms) : 0, 700658
GlobalTracer [baseline] (310.968 ms) : 0, 310968
GlobalTracer [candidate] (311.373 ms) : 0, 311373
AppSec [baseline] (162.426 ms) : 0, 162426
AppSec [candidate] (163.029 ms) : 0, 163029
Remote Config [baseline] (640.656 µs) : 0, 641
Remote Config [candidate] (645.268 µs) : 0, 645
Telemetry [baseline] (10.396 ms) : 0, 10396
Telemetry [candidate] (7.845 ms) : 0, 7845
IAST [baseline] (20.338 ms) : 0, 20338
IAST [candidate] (20.369 ms) : 0, 20369
section iast
BytebuddyAgent [baseline] (795.874 ms) : 0, 795874
BytebuddyAgent [candidate] (803.788 ms) : 0, 803788
GlobalTracer [baseline] (301.142 ms) : 0, 301142
GlobalTracer [candidate] (303.697 ms) : 0, 303697
AppSec [baseline] (54.604 ms) : 0, 54604
AppSec [candidate] (56.953 ms) : 0, 56953
Remote Config [baseline] (601.963 µs) : 0, 602
Remote Config [candidate] (600.156 µs) : 0, 600
Telemetry [baseline] (7.03 ms) : 0, 7030
Telemetry [candidate] (7.05 ms) : 0, 7050
IAST [baseline] (23.672 ms) : 0, 23672
IAST [candidate] (21.275 ms) : 0, 21275
section profiling
BytebuddyAgent [baseline] (676.156 ms) : 0, 676156
BytebuddyAgent [candidate] (678.517 ms) : 0, 678517
GlobalTracer [baseline] (395.245 ms) : 0, 395245
GlobalTracer [candidate] (394.211 ms) : 0, 394211
AppSec [baseline] (54.87 ms) : 0, 54870
AppSec [candidate] (54.347 ms) : 0, 54347
Remote Config [baseline] (662.536 µs) : 0, 663
Remote Config [candidate] (645.442 µs) : 0, 645
Telemetry [baseline] (7.512 ms) : 0, 7512
Telemetry [candidate] (7.427 ms) : 0, 7427
ProfilingAgent [baseline] (96.58 ms) : 0, 96580
ProfilingAgent [candidate] (96.075 ms) : 0, 96075
Profiling [baseline] (96.604 ms) : 0, 96604
Profiling [candidate] (96.099 ms) : 0, 96099
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.075 s) : 0, 1075323
Total [baseline] (8.626 s) : 0, 8626244
Agent [candidate] (1.081 s) : 0, 1080620
Total [candidate] (8.613 s) : 0, 8612506
section iast
Agent [baseline] (1.199 s) : 0, 1199051
Total [baseline] (9.164 s) : 0, 9164214
Agent [candidate] (1.206 s) : 0, 1205766
Total [candidate] (9.148 s) : 0, 9148402
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.195 s) : 0, 1195425
Total [baseline] (9.12 s) : 0, 9120090
Agent [candidate] (1.198 s) : 0, 1197749
Total [candidate] (9.106 s) : 0, 9105597
section iast_TELEMETRY_OFF
Agent [baseline] (1.194 s) : 0, 1194471
Total [baseline] (9.089 s) : 0, 9089000
Agent [candidate] (1.194 s) : 0, 1194311
Total [candidate] (9.078 s) : 0, 9078320
gantt
title insecure-bank - break down per module: candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.638 ms) : 0, 683638
BytebuddyAgent [candidate] (690.442 ms) : 0, 690442
GlobalTracer [baseline] (315.618 ms) : 0, 315618
GlobalTracer [candidate] (314.221 ms) : 0, 314221
AppSec [baseline] (54.224 ms) : 0, 54224
AppSec [candidate] (54.043 ms) : 0, 54043
Remote Config [baseline] (669.526 µs) : 0, 670
Remote Config [candidate] (670.332 µs) : 0, 670
Telemetry [baseline] (7.565 ms) : 0, 7565
Telemetry [candidate] (7.482 ms) : 0, 7482
section iast
BytebuddyAgent [baseline] (797.337 ms) : 0, 797337
BytebuddyAgent [candidate] (802.382 ms) : 0, 802382
GlobalTracer [baseline] (301.09 ms) : 0, 301090
GlobalTracer [candidate] (302.888 ms) : 0, 302888
AppSec [baseline] (57.613 ms) : 0, 57613
AppSec [candidate] (56.828 ms) : 0, 56828
IAST [baseline] (21.632 ms) : 0, 21632
IAST [candidate] (21.418 ms) : 0, 21418
Remote Config [baseline] (610.698 µs) : 0, 611
Remote Config [candidate] (603.723 µs) : 0, 604
Telemetry [baseline] (7.133 ms) : 0, 7133
Telemetry [candidate] (7.888 ms) : 0, 7888
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (794.838 ms) : 0, 794838
BytebuddyAgent [candidate] (796.685 ms) : 0, 796685
GlobalTracer [baseline] (300.989 ms) : 0, 300989
GlobalTracer [candidate] (301.114 ms) : 0, 301114
AppSec [baseline] (57.22 ms) : 0, 57220
AppSec [candidate] (55.74 ms) : 0, 55740
IAST [baseline] (21.168 ms) : 0, 21168
IAST [candidate] (22.846 ms) : 0, 22846
Remote Config [baseline] (594.463 µs) : 0, 594
Remote Config [candidate] (628.622 µs) : 0, 629
Telemetry [baseline] (7.001 ms) : 0, 7001
Telemetry [candidate] (7.071 ms) : 0, 7071
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (794.022 ms) : 0, 794022
BytebuddyAgent [candidate] (794.277 ms) : 0, 794277
GlobalTracer [baseline] (301.615 ms) : 0, 301615
GlobalTracer [candidate] (301.339 ms) : 0, 301339
AppSec [baseline] (54.846 ms) : 0, 54846
AppSec [candidate] (56.011 ms) : 0, 56011
IAST [baseline] (22.797 ms) : 0, 22797
IAST [candidate] (21.443 ms) : 0, 21443
Remote Config [baseline] (595.099 µs) : 0, 595
Remote Config [candidate] (611.663 µs) : 0, 612
Telemetry [baseline] (6.94 ms) : 0, 6940
Telemetry [candidate] (6.984 ms) : 0, 6984
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section baseline
no_agent (374.857 µs) : 355, 394
. : milestone, 375,
iast (491.622 µs) : 470, 514
. : milestone, 492,
iast_FULL (559.896 µs) : 539, 581
. : milestone, 560,
iast_GLOBAL (507.586 µs) : 486, 529
. : milestone, 508,
iast_HARDCODED_SECRET_DISABLED (487.905 µs) : 467, 509
. : milestone, 488,
iast_INACTIVE (461.913 µs) : 441, 483
. : milestone, 462,
iast_TELEMETRY_OFF (473.811 µs) : 452, 495
. : milestone, 474,
tracing (449.983 µs) : 429, 471
. : milestone, 450,
section candidate
no_agent (375.226 µs) : 354, 396
. : milestone, 375,
iast (485.164 µs) : 464, 506
. : milestone, 485,
iast_FULL (558.789 µs) : 538, 580
. : milestone, 559,
iast_GLOBAL (508.378 µs) : 487, 530
. : milestone, 508,
iast_HARDCODED_SECRET_DISABLED (491.059 µs) : 470, 512
. : milestone, 491,
iast_INACTIVE (445.098 µs) : 425, 466
. : milestone, 445,
iast_TELEMETRY_OFF (474.803 µs) : 454, 496
. : milestone, 475,
tracing (448.873 µs) : 428, 470
. : milestone, 449,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section baseline
no_agent (1.327 ms) : 1308, 1346
. : milestone, 1327,
appsec (1.711 ms) : 1686, 1736
. : milestone, 1711,
appsec_no_iast (1.716 ms) : 1691, 1740
. : milestone, 1716,
iast (1.466 ms) : 1442, 1489
. : milestone, 1466,
profiling (1.492 ms) : 1469, 1514
. : milestone, 1492,
tracing (1.493 ms) : 1469, 1517
. : milestone, 1493,
section candidate
no_agent (1.323 ms) : 1304, 1343
. : milestone, 1323,
appsec (1.731 ms) : 1706, 1755
. : milestone, 1731,
appsec_no_iast (1.752 ms) : 1728, 1776
. : milestone, 1752,
iast (1.477 ms) : 1454, 1499
. : milestone, 1477,
profiling (1.492 ms) : 1468, 1515
. : milestone, 1492,
tracing (1.482 ms) : 1458, 1506
. : milestone, 1482,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section baseline
no_agent (15.088 s) : 15088000, 15088000
. : milestone, 15088000,
appsec (15.223 s) : 15223000, 15223000
. : milestone, 15223000,
iast (18.772 s) : 18772000, 18772000
. : milestone, 18772000,
iast_GLOBAL (18.141 s) : 18141000, 18141000
. : milestone, 18141000,
profiling (15.777 s) : 15777000, 15777000
. : milestone, 15777000,
tracing (15.369 s) : 15369000, 15369000
. : milestone, 15369000,
section candidate
no_agent (14.946 s) : 14946000, 14946000
. : milestone, 14946000,
appsec (15.002 s) : 15002000, 15002000
. : milestone, 15002000,
iast (19.078 s) : 19078000, 19078000
. : milestone, 19078000,
iast_GLOBAL (17.587 s) : 17587000, 17587000
. : milestone, 17587000,
profiling (15.4 s) : 15400000, 15400000
. : milestone, 15400000,
tracing (15.393 s) : 15393000, 15393000
. : milestone, 15393000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~185d67e2c7, baseline=1.41.0-SNAPSHOT~3f89f0321b
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1458, 1481
. : milestone, 1469,
appsec (2.326 ms) : 2285, 2368
. : milestone, 2326,
iast (2.082 ms) : 2029, 2135
. : milestone, 2082,
iast_GLOBAL (2.113 ms) : 2061, 2165
. : milestone, 2113,
profiling (1.957 ms) : 1913, 2000
. : milestone, 1957,
tracing (1.919 ms) : 1879, 1959
. : milestone, 1919,
section candidate
no_agent (1.459 ms) : 1448, 1470
. : milestone, 1459,
appsec (2.337 ms) : 2296, 2379
. : milestone, 2337,
iast (2.074 ms) : 2022, 2126
. : milestone, 2074,
iast_GLOBAL (2.119 ms) : 2066, 2171
. : milestone, 2119,
profiling (1.94 ms) : 1899, 1982
. : milestone, 1940,
tracing (1.917 ms) : 1877, 1957
. : milestone, 1917,
|
anmarchenko
approved these changes
Oct 18, 2024
This was referenced Oct 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What Does This Do
Fixes module name detection for headless test sessions (i.e. sessions for JVMs that are not part of an instrumented Maven/Gradle build).
When a JVM is a part of an instrumented Maven/Gradle build, the parent process provides module names to the children processes via system properties.
For headless sessions the module name is not provided, so there is a fallback that tries to derive module name from current path and repository root (e.g. if repo root is
/my/repo/root
and current path is/my/repo/root/submodule
, then the module name is set tosubmodule
).This fallback logic works incorrectly when repo root is
/my/repo/root
and the current path is/my/repo/root/
: in this case module name is empty, which is not allowed.The fix detects these cases, and uses service name as the last fallback if module name cannot be derived from current path.
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: SDTEST-1110