-
Notifications
You must be signed in to change notification settings - Fork 318
Bump spotbugs gradle plugin to 6.4.5 #9748
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: master
Are you sure you want to change the base?
Conversation
…internal-api Also this forces JDK 11 to be the minimum.
# Conflicts: # communication/src/main/java/datadog/communication/ddagent/SharedCommunicationObjects.java # utils/test-utils/src/main/groovy/datadog/trace/test/util/DDSpecification.groovy
Debugger benchmarksParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 6 metrics, 6 unstable metrics.
See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (322.865 µs) : 284, 362
. : milestone, 323,
basic (292.34 µs) : 286, 299
. : milestone, 292,
loop (8.956 ms) : 8953, 8959
. : milestone, 8956,
section candidate
noprobe (331.706 µs) : 280, 383
. : milestone, 332,
basic (279.828 µs) : 273, 286
. : milestone, 280,
loop (8.956 ms) : 8950, 8962
. : milestone, 8956,
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 61 metrics, 4 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.101 s) : 0, 1101113
Total [baseline] (10.771 s) : 0, 10771439
Agent [candidate] (1.11 s) : 0, 1110025
Total [candidate] (10.829 s) : 0, 10829073
section appsec
Agent [baseline] (1.29 s) : 0, 1290019
Total [baseline] (11.098 s) : 0, 11098426
Agent [candidate] (1.291 s) : 0, 1291255
Total [candidate] (11.105 s) : 0, 11104704
section iast
Agent [baseline] (1.241 s) : 0, 1241111
Total [baseline] (11.233 s) : 0, 11233484
Agent [candidate] (1.251 s) : 0, 1250527
Total [candidate] (11.166 s) : 0, 11166461
section profiling
Agent [baseline] (1.234 s) : 0, 1234271
Total [baseline] (11.132 s) : 0, 11131579
Agent [candidate] (1.233 s) : 0, 1232840
Total [candidate] (11.081 s) : 0, 11081449
gantt
title petclinic - break down per module: candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.478 ms) : 0, 1478
crashtracking [candidate] (1.485 ms) : 0, 1485
BytebuddyAgent [baseline] (708.062 ms) : 0, 708062
BytebuddyAgent [candidate] (713.204 ms) : 0, 713204
GlobalTracer [baseline] (248.461 ms) : 0, 248461
GlobalTracer [candidate] (250.786 ms) : 0, 250786
AppSec [baseline] (31.773 ms) : 0, 31773
AppSec [candidate] (32.423 ms) : 0, 32423
Debugger [baseline] (64.038 ms) : 0, 64038
Debugger [candidate] (64.482 ms) : 0, 64482
Remote Config [baseline] (623.057 µs) : 0, 623
Remote Config [candidate] (633.803 µs) : 0, 634
Telemetry [baseline] (8.196 ms) : 0, 8196
Telemetry [candidate] (8.335 ms) : 0, 8335
Flare Poller [baseline] (3.658 ms) : 0, 3658
Flare Poller [candidate] (3.725 ms) : 0, 3725
section appsec
crashtracking [baseline] (1.483 ms) : 0, 1483
crashtracking [candidate] (1.486 ms) : 0, 1486
BytebuddyAgent [baseline] (737.711 ms) : 0, 737711
BytebuddyAgent [candidate] (737.684 ms) : 0, 737684
GlobalTracer [baseline] (242.564 ms) : 0, 242564
GlobalTracer [candidate] (242.817 ms) : 0, 242817
AppSec [baseline] (173.787 ms) : 0, 173787
AppSec [candidate] (174.333 ms) : 0, 174333
Debugger [baseline] (61.468 ms) : 0, 61468
Debugger [candidate] (61.827 ms) : 0, 61827
Remote Config [baseline] (682.388 µs) : 0, 682
Remote Config [candidate] (681.719 µs) : 0, 682
Telemetry [baseline] (8.261 ms) : 0, 8261
Telemetry [candidate] (8.258 ms) : 0, 8258
Flare Poller [baseline] (4.013 ms) : 0, 4013
Flare Poller [candidate] (3.951 ms) : 0, 3951
IAST [baseline] (24.97 ms) : 0, 24970
IAST [candidate] (25.227 ms) : 0, 25227
section iast
crashtracking [baseline] (1.5 ms) : 0, 1500
crashtracking [candidate] (1.498 ms) : 0, 1498
BytebuddyAgent [baseline] (833.182 ms) : 0, 833182
BytebuddyAgent [candidate] (840.005 ms) : 0, 840005
GlobalTracer [baseline] (237.608 ms) : 0, 237608
GlobalTracer [candidate] (238.951 ms) : 0, 238951
AppSec [baseline] (33.209 ms) : 0, 33209
AppSec [candidate] (33.41 ms) : 0, 33410
Debugger [baseline] (60.587 ms) : 0, 60587
Debugger [candidate] (61.041 ms) : 0, 61041
Remote Config [baseline] (549.862 µs) : 0, 550
Remote Config [candidate] (546.659 µs) : 0, 547
Telemetry [baseline] (7.645 ms) : 0, 7645
Telemetry [candidate] (7.662 ms) : 0, 7662
Flare Poller [baseline] (3.555 ms) : 0, 3555
Flare Poller [candidate] (3.494 ms) : 0, 3494
IAST [baseline] (28.505 ms) : 0, 28505
IAST [candidate] (28.842 ms) : 0, 28842
section profiling
crashtracking [baseline] (1.432 ms) : 0, 1432
crashtracking [candidate] (1.431 ms) : 0, 1431
BytebuddyAgent [baseline] (735.428 ms) : 0, 735428
BytebuddyAgent [candidate] (737.388 ms) : 0, 737388
GlobalTracer [baseline] (223.124 ms) : 0, 223124
GlobalTracer [candidate] (222.042 ms) : 0, 222042
AppSec [baseline] (32.426 ms) : 0, 32426
AppSec [candidate] (32.132 ms) : 0, 32132
Debugger [baseline] (63.054 ms) : 0, 63054
Debugger [candidate] (62.704 ms) : 0, 62704
Remote Config [baseline] (642.453 µs) : 0, 642
Remote Config [candidate] (649.437 µs) : 0, 649
Telemetry [baseline] (8.015 ms) : 0, 8015
Telemetry [candidate] (8.004 ms) : 0, 8004
Flare Poller [baseline] (3.774 ms) : 0, 3774
Flare Poller [candidate] (3.71 ms) : 0, 3710
ProfilingAgent [baseline] (97.232 ms) : 0, 97232
ProfilingAgent [candidate] (95.837 ms) : 0, 95837
Profiling [baseline] (97.831 ms) : 0, 97831
Profiling [candidate] (96.418 ms) : 0, 96418
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.105 s) : 0, 1104882
Total [baseline] (8.854 s) : 0, 8853892
Agent [candidate] (1.111 s) : 0, 1111214
Total [candidate] (8.839 s) : 0, 8839387
section iast
Agent [baseline] (1.248 s) : 0, 1248214
Total [baseline] (9.559 s) : 0, 9559135
Agent [candidate] (1.237 s) : 0, 1237146
Total [candidate] (9.542 s) : 0, 9542090
gantt
title insecure-bank - break down per module: candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.5 ms) : 0, 1500
crashtracking [candidate] (1.486 ms) : 0, 1486
BytebuddyAgent [baseline] (712.204 ms) : 0, 712204
BytebuddyAgent [candidate] (715.511 ms) : 0, 715511
GlobalTracer [baseline] (248.655 ms) : 0, 248655
GlobalTracer [candidate] (251.337 ms) : 0, 251337
AppSec [baseline] (31.946 ms) : 0, 31946
AppSec [candidate] (32.272 ms) : 0, 32272
Debugger [baseline] (63.273 ms) : 0, 63273
Debugger [candidate] (63.108 ms) : 0, 63108
Remote Config [baseline] (625.018 µs) : 0, 625
Remote Config [candidate] (630.326 µs) : 0, 630
Telemetry [baseline] (8.162 ms) : 0, 8162
Telemetry [candidate] (8.26 ms) : 0, 8260
Flare Poller [baseline] (3.668 ms) : 0, 3668
Flare Poller [candidate] (3.693 ms) : 0, 3693
section iast
crashtracking [baseline] (1.496 ms) : 0, 1496
crashtracking [candidate] (1.463 ms) : 0, 1463
BytebuddyAgent [baseline] (839.293 ms) : 0, 839293
BytebuddyAgent [candidate] (831.335 ms) : 0, 831335
GlobalTracer [baseline] (238.541 ms) : 0, 238541
GlobalTracer [candidate] (236.991 ms) : 0, 236991
AppSec [baseline] (33.32 ms) : 0, 33320
AppSec [candidate] (33.019 ms) : 0, 33019
Debugger [baseline] (60.101 ms) : 0, 60101
Debugger [candidate] (59.841 ms) : 0, 59841
Remote Config [baseline] (544.772 µs) : 0, 545
Remote Config [candidate] (546.605 µs) : 0, 547
Telemetry [baseline] (7.555 ms) : 0, 7555
Telemetry [candidate] (7.566 ms) : 0, 7566
Flare Poller [baseline] (3.43 ms) : 0, 3430
Flare Poller [candidate] (3.373 ms) : 0, 3373
IAST [baseline] (28.958 ms) : 0, 28958
IAST [candidate] (28.214 ms) : 0, 28214
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 2 performance regressions! Performance is the same for 18 metrics, 16 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section baseline
no_agent (18.156 ms) : 17969, 18344
. : milestone, 18156,
appsec (18.9 ms) : 18712, 19089
. : milestone, 18900,
code_origins (17.858 ms) : 17682, 18035
. : milestone, 17858,
iast (17.956 ms) : 17778, 18134
. : milestone, 17956,
profiling (18.71 ms) : 18525, 18894
. : milestone, 18710,
tracing (17.969 ms) : 17791, 18148
. : milestone, 17969,
section candidate
no_agent (18.33 ms) : 18139, 18520
. : milestone, 18330,
appsec (18.757 ms) : 18571, 18944
. : milestone, 18757,
code_origins (17.826 ms) : 17652, 18000
. : milestone, 17826,
iast (17.881 ms) : 17702, 18060
. : milestone, 17881,
profiling (18.517 ms) : 18330, 18703
. : milestone, 18517,
tracing (17.841 ms) : 17664, 18018
. : milestone, 17841,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section baseline
no_agent (1.206 ms) : 1193, 1218
. : milestone, 1206,
iast (3.335 ms) : 3287, 3384
. : milestone, 3335,
iast_FULL (5.895 ms) : 5837, 5954
. : milestone, 5895,
iast_GLOBAL (3.513 ms) : 3461, 3564
. : milestone, 3513,
profiling (2.062 ms) : 2045, 2080
. : milestone, 2062,
tracing (1.882 ms) : 1866, 1898
. : milestone, 1882,
section candidate
no_agent (1.196 ms) : 1185, 1208
. : milestone, 1196,
iast (3.327 ms) : 3275, 3379
. : milestone, 3327,
iast_FULL (5.809 ms) : 5751, 5867
. : milestone, 5809,
iast_GLOBAL (3.84 ms) : 3783, 3898
. : milestone, 3840,
profiling (2.22 ms) : 2200, 2240
. : milestone, 2220,
tracing (1.952 ms) : 1934, 1969
. : milestone, 1952,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (3.691 ms) : 3472, 3910
. : milestone, 3691,
iast (2.202 ms) : 2138, 2266
. : milestone, 2202,
iast_GLOBAL (2.249 ms) : 2184, 2314
. : milestone, 2249,
profiling (2.089 ms) : 2035, 2144
. : milestone, 2089,
tracing (2.047 ms) : 1996, 2098
. : milestone, 2047,
section candidate
no_agent (1.471 ms) : 1460, 1483
. : milestone, 1471,
appsec (2.512 ms) : 2457, 2566
. : milestone, 2512,
iast (2.201 ms) : 2137, 2265
. : milestone, 2201,
iast_GLOBAL (2.251 ms) : 2186, 2316
. : milestone, 2251,
profiling (2.066 ms) : 2014, 2118
. : milestone, 2066,
tracing (2.035 ms) : 1985, 2086
. : milestone, 2035,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~dd1b15c0c7, baseline=1.57.0-SNAPSHOT~c8bb44440b
dateFormat X
axisFormat %s
section baseline
no_agent (14.964 s) : 14964000, 14964000
. : milestone, 14964000,
appsec (14.809 s) : 14809000, 14809000
. : milestone, 14809000,
iast (18.148 s) : 18148000, 18148000
. : milestone, 18148000,
iast_GLOBAL (17.943 s) : 17943000, 17943000
. : milestone, 17943000,
profiling (14.65 s) : 14650000, 14650000
. : milestone, 14650000,
tracing (14.813 s) : 14813000, 14813000
. : milestone, 14813000,
section candidate
no_agent (15.019 s) : 15019000, 15019000
. : milestone, 15019000,
appsec (14.83 s) : 14830000, 14830000
. : milestone, 14830000,
iast (18.32 s) : 18320000, 18320000
. : milestone, 18320000,
iast_GLOBAL (17.886 s) : 17886000, 17886000
. : milestone, 17886000,
profiling (14.671 s) : 14671000, 14671000
. : milestone, 14671000,
tracing (14.868 s) : 14868000, 14868000
. : milestone, 14868000,
|
This comment has been minimized.
This comment has been minimized.
# Conflicts: # gradle/libs.versions.toml
PerfectSlayer
left a comment
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.
💭 thought: The title is a bit misleading, it seems to be more like a migration than a dependency bump.
As the PR grew a lot, should we split in some way to get product / team approval seperately?
The bump triggered these code updates 😭 (due to updated spotbugs engine). |
| private final boolean suppressFailures; | ||
| private int executions; | ||
| private boolean successfulExecutionSeen; | ||
| private final AtomicInteger executions = new AtomicInteger(0); |
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.
why those changes should be related to bumping a pluging?
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.
tl;dr New spotbugs engine
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.
Yes but I won't expect needing a code change because we're upgrading a plugin version. They don't have the same semantic
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.
That's the thing spotbugs now identifies this as an issue in this code and as such fails in CI, like in other places this is what I thought was a fix.
But as discussed Monday. We can instead ignore the n'es spotted issues.
Whether an suppress Annotation or code change, this file will be modified.
What Does This Do
This PR bumps spotbugs gradle plugin to the most recent version 6.4.5 to this day (from 5.0.14).
And require a JDK 11 minimum to run.
Important
This version of spotbugs has new rules, better detection engine that requires to change so of the warnings ignores, or fix some code.
Motivation
Latest tools provide more value.
Additional Notes
Need
Related to