Skip to content

Re-configure GitLab deployment jobs#11757

Draft
sarahchen6 wants to merge 4 commits into
masterfrom
sarahchen6/maven-deployment-dependencies-adjustment
Draft

Re-configure GitLab deployment jobs#11757
sarahchen6 wants to merge 4 commits into
masterfrom
sarahchen6/maven-deployment-dependencies-adjustment

Conversation

@sarahchen6

@sarahchen6 sarahchen6 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What Does This Do

Reconfigure Maven and GitHub deployment jobs to run after GitLab system-tests succeed.

Also introduce the new shared-pipeline stages from https://github.com/DataDog/libdatadog-build/pull/236.

Motivation

In release 1.63.1 where GitLab system-tests failed, Maven and GitHub artifacts were deployed while OCI artifacts were not because only OCI artifacts depended on system-tests success. Ensure that publishing steps occur in line with each other after system-tests succeed.

Additional Notes

Check the effects of the changes in this PR by going to the GitLab pipeline, clicking "Job dependencies" and toggling "Show dependencies" (https://gitlab.ddbuild.io/DataDog/apm-reliability/dd-trace-java/-/pipelines/121679382).

Contributor Checklist

  • Format the title according to the contribution guidelines
  • Assign the type: and (comp: or inst:) labels in addition to any other useful labels
  • Avoid using close, fix, or any linking keywords when referencing an issue
    Use solves instead, and assign the PR milestone to the issue
  • Update the CODEOWNERS file on source file addition, migration, or deletion
  • Update public documentation with any new configuration flags or behaviors
  • Add your completed PR to the merge queue by commenting /merge. You can also:
    • Customize the commit message associated with the merge with /merge --commit-message "..."
    • Remove your PR from the merge queue with /merge -c
    • Skip all merge queue checks with /merge -f --reason "reason"; please use this judiciously, as some checks do not run at the PR-level (note: the PR still needs to be mergeable, this will only skip the pre-merge build)
    • Get more information in this doc

Jira ticket: [PROJ-IDENT]

@sarahchen6 sarahchen6 added type: enhancement Enhancements and improvements tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels Jun 26, 2026
@sarahchen6 sarahchen6 marked this pull request as ready for review June 26, 2026 21:36
@sarahchen6 sarahchen6 requested review from a team as code owners June 26, 2026 21:36
@sarahchen6 sarahchen6 requested review from bric3 and randomanderson and removed request for a team June 26, 2026 21:36
@dd-octo-sts

dd-octo-sts Bot commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.93 s 13.93 s [-0.7%; +0.8%] (no difference)
startup:insecure-bank:tracing:Agent 12.90 s 13.04 s [-1.9%; -0.2%] (maybe better)
startup:petclinic:appsec:Agent 16.99 s 16.87 s [-0.5%; +2.0%] (no difference)
startup:petclinic:iast:Agent 16.98 s 17.06 s [-1.4%; +0.5%] (no difference)
startup:petclinic:profiling:Agent 16.79 s 17.07 s [-2.8%; -0.5%] (maybe better)
startup:petclinic:sca:Agent 17.06 s 16.72 s [+1.2%; +2.8%] (significantly worse)
startup:petclinic:tracing:Agent 16.15 s 16.16 s [-0.9%; +0.8%] (no difference)

Commit: e20d0fbd · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@sarahchen6 sarahchen6 marked this pull request as draft June 29, 2026 17:17
@sarahchen6 sarahchen6 marked this pull request as ready for review June 29, 2026 20:05
@sarahchen6 sarahchen6 marked this pull request as draft June 30, 2026 17:38
@sarahchen6 sarahchen6 force-pushed the sarahchen6/maven-deployment-dependencies-adjustment branch from ca8e02e to 40a4c3d Compare June 30, 2026 20:35
@sarahchen6 sarahchen6 force-pushed the sarahchen6/maven-deployment-dependencies-adjustment branch from 40a4c3d to e20d0fb Compare June 30, 2026 20:37
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 type: enhancement Enhancements and improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant