Skip to content

Ignore all gitlab jobs in Check CI pipeline #8972

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

Merged
merged 2 commits into from
Jun 16, 2025
Merged

Conversation

bric3
Copy link
Contributor

@bric3 bric3 commented Jun 12, 2025

What Does This Do

gitlab/default-pipeline regroups all gitlab jobs and has been added as a required check for master, project/* and release/ in rulesets. (See LANGPLAT-589)

Also these job names don't exist anymore since #8945 (i.e. there are split by 4), as such they are not used

  • dd-gitlab/test_smoke: \[11, 2/2\]
  • dd-gitlab/test_smoke: \[8, 2/2\]

Motivation

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@bric3 bric3 requested a review from a team as a code owner June 12, 2025 10:48
@bric3 bric3 requested review from sarahchen6 and removed request for a team June 12, 2025 10:48
@bric3 bric3 added the tag: no release notes Changes to exclude from release notes label Jun 12, 2025
Copy link
Contributor

@PerfectSlayer PerfectSlayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to rework the whole integration of this workflow.
It's not working as is and @cbeauchesne recommended to use it differently from the workflow repository documentation.
Would you like to have a look at it?

@pr-commenter
Copy link

pr-commenter bot commented Jun 12, 2025

Benchmarks

Startup

Parameters

Baseline Candidate
baseline_or_candidate baseline candidate
git_branch master bdu/clean-green-ci-check
git_commit_date 1750092705 1750092938
git_commit_sha d56d38c b2e247f
release_version 1.50.0-SNAPSHOT~d56d38c890 1.50.0-SNAPSHOT~b2e247fee1
See matching parameters
Baseline Candidate
application insecure-bank insecure-bank
ci_job_date 1750095071 1750095071
ci_job_id 984120573 984120573
ci_pipeline_id 67879167 67879167
cpu_model Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz
kernel_version Linux runner-63wmvl1-project-304-concurrent-0-qy9awyt6 6.8.0-1029-aws #31~22.04.1-Ubuntu SMP Thu Apr 24 21:16:18 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux Linux runner-63wmvl1-project-304-concurrent-0-qy9awyt6 6.8.0-1029-aws #31~22.04.1-Ubuntu SMP Thu Apr 24 21:16:18 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
module Agent Agent
parent None None
variant iast iast

Summary

Found 0 performance improvements and 0 performance regressions! Performance is the same for 58 metrics, 13 unstable metrics.

Startup time reports for petclinic
gantt
    title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~b2e247fee1, baseline=1.50.0-SNAPSHOT~d56d38c890

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.034 s) : 0, 1033715
Total [baseline] (10.683 s) : 0, 10683281
Agent [candidate] (1.03 s) : 0, 1030465
Total [candidate] (10.636 s) : 0, 10635637
section appsec
Agent [baseline] (1.18 s) : 0, 1179574
Total [baseline] (10.7 s) : 0, 10700018
Agent [candidate] (1.18 s) : 0, 1179727
Total [candidate] (10.675 s) : 0, 10675106
section iast
Agent [baseline] (1.155 s) : 0, 1154797
Total [baseline] (10.925 s) : 0, 10925117
Agent [candidate] (1.157 s) : 0, 1156648
Total [candidate] (10.908 s) : 0, 10908060
section profiling
Agent [baseline] (1.273 s) : 0, 1273249
Total [baseline] (10.901 s) : 0, 10901093
Agent [candidate] (1.272 s) : 0, 1271872
Total [candidate] (10.9 s) : 0, 10899826
Loading
  • baseline results
Module Variant Duration Δ tracing
Agent tracing 1.034 s -
Agent appsec 1.18 s 145.858 ms (14.1%)
Agent iast 1.155 s 121.082 ms (11.7%)
Agent profiling 1.273 s 239.534 ms (23.2%)
Total tracing 10.683 s -
Total appsec 10.7 s 16.737 ms (0.2%)
Total iast 10.925 s 241.836 ms (2.3%)
Total profiling 10.901 s 217.811 ms (2.0%)
  • candidate results
Module Variant Duration Δ tracing
Agent tracing 1.03 s -
Agent appsec 1.18 s 149.262 ms (14.5%)
Agent iast 1.157 s 126.183 ms (12.2%)
Agent profiling 1.272 s 241.407 ms (23.4%)
Total tracing 10.636 s -
Total appsec 10.675 s 39.469 ms (0.4%)
Total iast 10.908 s 272.422 ms (2.6%)
Total profiling 10.9 s 264.188 ms (2.5%)
gantt
    title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~b2e247fee1, baseline=1.50.0-SNAPSHOT~d56d38c890

    dateFormat X
    axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.024 ms) : 0, 688024
BytebuddyAgent [candidate] (684.644 ms) : 0, 684644
GlobalTracer [baseline] (241.9 ms) : 0, 241900
GlobalTracer [candidate] (242.84 ms) : 0, 242840
AppSec [baseline] (59.898 ms) : 0, 59898
AppSec [candidate] (60.24 ms) : 0, 60240
Debugger [baseline] (6.84 ms) : 0, 6840
Debugger [candidate] (6.156 ms) : 0, 6156
Remote Config [baseline] (746.065 µs) : 0, 746
Remote Config [candidate] (742.417 µs) : 0, 742
Telemetry [baseline] (12.741 ms) : 0, 12741
Telemetry [candidate] (12.281 ms) : 0, 12281
section appsec
BytebuddyAgent [baseline] (706.152 ms) : 0, 706152
BytebuddyAgent [candidate] (706.653 ms) : 0, 706653
GlobalTracer [baseline] (235.477 ms) : 0, 235477
GlobalTracer [candidate] (235.357 ms) : 0, 235357
IAST [baseline] (21.836 ms) : 0, 21836
IAST [candidate] (21.746 ms) : 0, 21746
AppSec [baseline] (178.852 ms) : 0, 178852
AppSec [candidate] (178.709 ms) : 0, 178709
Debugger [baseline] (5.893 ms) : 0, 5893
Debugger [candidate] (5.871 ms) : 0, 5871
Remote Config [baseline] (599.894 µs) : 0, 600
Remote Config [candidate] (607.116 µs) : 0, 607
Telemetry [baseline] (7.307 ms) : 0, 7307
Telemetry [candidate] (7.314 ms) : 0, 7314
section iast
BytebuddyAgent [baseline] (804.278 ms) : 0, 804278
BytebuddyAgent [candidate] (806.967 ms) : 0, 806967
GlobalTracer [baseline] (230.955 ms) : 0, 230955
GlobalTracer [candidate] (230.33 ms) : 0, 230330
IAST [baseline] (27.672 ms) : 0, 27672
IAST [candidate] (30.033 ms) : 0, 30033
AppSec [baseline] (53.995 ms) : 0, 53995
AppSec [candidate] (51.366 ms) : 0, 51366
Debugger [baseline] (5.938 ms) : 0, 5938
Debugger [candidate] (5.919 ms) : 0, 5919
Remote Config [baseline] (584.038 µs) : 0, 584
Remote Config [candidate] (582.697 µs) : 0, 583
Telemetry [baseline] (7.86 ms) : 0, 7860
Telemetry [candidate] (7.889 ms) : 0, 7889
section profiling
BytebuddyAgent [baseline] (676.272 ms) : 0, 676272
BytebuddyAgent [candidate] (675.014 ms) : 0, 675014
GlobalTracer [baseline] (361.097 ms) : 0, 361097
GlobalTracer [candidate] (359.997 ms) : 0, 359997
AppSec [baseline] (64.492 ms) : 0, 64492
AppSec [candidate] (64.513 ms) : 0, 64513
Debugger [baseline] (6.17 ms) : 0, 6170
Debugger [candidate] (6.17 ms) : 0, 6170
Remote Config [baseline] (671.922 µs) : 0, 672
Remote Config [candidate] (648.266 µs) : 0, 648
Telemetry [baseline] (8.116 ms) : 0, 8116
Telemetry [candidate] (8.168 ms) : 0, 8168
ProfilingAgent [baseline] (105.401 ms) : 0, 105401
ProfilingAgent [candidate] (106.499 ms) : 0, 106499
Profiling [baseline] (105.426 ms) : 0, 105426
Profiling [candidate] (106.524 ms) : 0, 106524
Loading
Startup time reports for insecure-bank
gantt
    title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~b2e247fee1, baseline=1.50.0-SNAPSHOT~d56d38c890

    dateFormat X
    axisFormat %s
section tracing
Agent [baseline] (1.027 s) : 0, 1026889
Total [baseline] (8.503 s) : 0, 8503028
Agent [candidate] (1.028 s) : 0, 1027528
Total [candidate] (8.554 s) : 0, 8554238
section iast
Agent [baseline] (1.164 s) : 0, 1163860
Total [baseline] (9.192 s) : 0, 9192101
Agent [candidate] (1.155 s) : 0, 1155407
Total [candidate] (9.188 s) : 0, 9188465
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.159 s) : 0, 1159375
Total [baseline] (9.193 s) : 0, 9193495
Agent [candidate] (1.163 s) : 0, 1163467
Total [candidate] (9.157 s) : 0, 9156863
section iast_TELEMETRY_OFF
Agent [baseline] (1.152 s) : 0, 1152046
Total [baseline] (9.188 s) : 0, 9187881
Agent [candidate] (1.152 s) : 0, 1152056
Total [candidate] (9.184 s) : 0, 9183903
Loading
  • baseline results
Module Variant Duration Δ tracing
Agent tracing 1.027 s -
Agent iast 1.164 s 136.971 ms (13.3%)
Agent iast_HARDCODED_SECRET_DISABLED 1.159 s 132.486 ms (12.9%)
Agent iast_TELEMETRY_OFF 1.152 s 125.157 ms (12.2%)
Total tracing 8.503 s -
Total iast 9.192 s 689.073 ms (8.1%)
Total iast_HARDCODED_SECRET_DISABLED 9.193 s 690.467 ms (8.1%)
Total iast_TELEMETRY_OFF 9.188 s 684.853 ms (8.1%)
  • candidate results
Module Variant Duration Δ tracing
Agent tracing 1.028 s -
Agent iast 1.155 s 127.879 ms (12.4%)
Agent iast_HARDCODED_SECRET_DISABLED 1.163 s 135.94 ms (13.2%)
Agent iast_TELEMETRY_OFF 1.152 s 124.529 ms (12.1%)
Total tracing 8.554 s -
Total iast 9.188 s 634.227 ms (7.4%)
Total iast_HARDCODED_SECRET_DISABLED 9.157 s 602.625 ms (7.0%)
Total iast_TELEMETRY_OFF 9.184 s 629.665 ms (7.4%)
gantt
    title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~b2e247fee1, baseline=1.50.0-SNAPSHOT~d56d38c890

    dateFormat X
    axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.494 ms) : 0, 683494
BytebuddyAgent [candidate] (684.723 ms) : 0, 684723
GlobalTracer [baseline] (240.712 ms) : 0, 240712
GlobalTracer [candidate] (241.186 ms) : 0, 241186
AppSec [baseline] (60.485 ms) : 0, 60485
AppSec [candidate] (59.154 ms) : 0, 59154
Debugger [baseline] (6.057 ms) : 0, 6057
Debugger [candidate] (7.648 ms) : 0, 7648
Remote Config [baseline] (735.375 µs) : 0, 735
Remote Config [candidate] (739.832 µs) : 0, 740
Telemetry [baseline] (12.007 ms) : 0, 12007
Telemetry [candidate] (10.546 ms) : 0, 10546
section iast
BytebuddyAgent [baseline] (810.847 ms) : 0, 810847
BytebuddyAgent [candidate] (804.517 ms) : 0, 804517
GlobalTracer [baseline] (232.733 ms) : 0, 232733
GlobalTracer [candidate] (231.335 ms) : 0, 231335
IAST [baseline] (27.108 ms) : 0, 27108
IAST [candidate] (28.634 ms) : 0, 28634
AppSec [baseline] (55.077 ms) : 0, 55077
AppSec [candidate] (52.962 ms) : 0, 52962
Debugger [baseline] (5.987 ms) : 0, 5987
Debugger [candidate] (5.968 ms) : 0, 5968
Remote Config [baseline] (591.509 µs) : 0, 592
Remote Config [candidate] (592.318 µs) : 0, 592
Telemetry [baseline] (7.97 ms) : 0, 7970
Telemetry [candidate] (7.884 ms) : 0, 7884
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (806.471 ms) : 0, 806471
BytebuddyAgent [candidate] (810.06 ms) : 0, 810060
GlobalTracer [baseline] (232.53 ms) : 0, 232530
GlobalTracer [candidate] (232.71 ms) : 0, 232710
IAST [baseline] (26.409 ms) : 0, 26409
IAST [candidate] (28.007 ms) : 0, 28007
AppSec [baseline] (55.878 ms) : 0, 55878
AppSec [candidate] (54.38 ms) : 0, 54380
Debugger [baseline] (6.026 ms) : 0, 6026
Debugger [candidate] (6.056 ms) : 0, 6056
Remote Config [baseline] (597.807 µs) : 0, 598
Remote Config [candidate] (591.226 µs) : 0, 591
Telemetry [baseline] (7.972 ms) : 0, 7972
Telemetry [candidate] (7.982 ms) : 0, 7982
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (801.949 ms) : 0, 801949
BytebuddyAgent [candidate] (801.57 ms) : 0, 801570
GlobalTracer [baseline] (231.446 ms) : 0, 231446
GlobalTracer [candidate] (231.678 ms) : 0, 231678
IAST [baseline] (28.482 ms) : 0, 28482
IAST [candidate] (28.586 ms) : 0, 28586
AppSec [baseline] (52.243 ms) : 0, 52243
AppSec [candidate] (52.26 ms) : 0, 52260
Debugger [baseline] (6.024 ms) : 0, 6024
Debugger [candidate] (6.033 ms) : 0, 6033
Remote Config [baseline] (590.491 µs) : 0, 590
Remote Config [candidate] (590.777 µs) : 0, 591
Telemetry [baseline] (7.824 ms) : 0, 7824
Telemetry [candidate] (7.868 ms) : 0, 7868
Loading

Load

Dacapo

@bric3 bric3 changed the title chore(ci): Clean Check CI pipeline chore(ci): Ignore all gitlab jobs in Check CI pipeline Jun 13, 2025
@bric3 bric3 force-pushed the bdu/clean-green-ci-check branch 2 times, most recently from 156ed57 to 15c9dc5 Compare June 13, 2025 19:53
@PerfectSlayer PerfectSlayer changed the title chore(ci): Ignore all gitlab jobs in Check CI pipeline Ignore all gitlab jobs in Check CI pipeline Jun 14, 2025
@PerfectSlayer PerfectSlayer added the comp: tooling Build & Tooling label Jun 14, 2025
Copy link
Contributor

@PerfectSlayer PerfectSlayer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick feedback about your PRs: can you make sure to follow the PR title conventions from the first bullet of contribution checklist? Thanks!

I added dd-gitlab/default-pipeline as PR check where it’s needed so let’s merge it now.

@bric3 bric3 force-pushed the bdu/clean-green-ci-check branch from 15c9dc5 to 26c3bcb Compare June 16, 2025 08:23
bric3 added 2 commits June 16, 2025 18:55
Checks on gitlab are enabled via dd-gitlab/default-pipeline status on the branch protection rules.
@bric3 bric3 force-pushed the bdu/clean-green-ci-check branch from 26c3bcb to b2e247f Compare June 16, 2025 16:55
@bric3 bric3 merged commit 3e93f43 into master Jun 16, 2025
484 checks passed
@bric3 bric3 deleted the bdu/clean-green-ci-check branch June 16, 2025 19:20
@github-actions github-actions bot added this to the 1.50.0 milestone Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: tooling Build & Tooling tag: no release notes Changes to exclude from release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants