Skip to content

Quickwin: remove unneded strictTraceWrites overrides#11765

Open
amarziali wants to merge 1 commit into
masterfrom
andrea.marziali/strict-quick
Open

Quickwin: remove unneded strictTraceWrites overrides#11765
amarziali wants to merge 1 commit into
masterfrom
andrea.marziali/strict-quick

Conversation

@amarziali

Copy link
Copy Markdown
Contributor

What Does This Do

Removes the strictTraceWrites overrides for those tests since they are passing without.

Motivation

Additional Notes

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]

@amarziali amarziali added type: enhancement Enhancements and improvements comp: testing Testing tag: no release notes Changes to exclude from release notes labels Jun 29, 2026
@dd-octo-sts

dd-octo-sts Bot commented Jun 29, 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 14.03 s 13.97 s [-0.5%; +1.2%] (no difference)
startup:insecure-bank:tracing:Agent 12.98 s 12.94 s [-0.3%; +0.9%] (no difference)
startup:petclinic:appsec:Agent 16.89 s 16.82 s [-0.5%; +1.3%] (no difference)
startup:petclinic:iast:Agent 16.40 s 16.91 s [-7.2%; +1.3%] (no difference)
startup:petclinic:profiling:Agent 16.77 s 16.83 s [-1.3%; +0.6%] (no difference)
startup:petclinic:sca:Agent 16.23 s 16.82 s [-9.0%; +2.0%] (unstable)
startup:petclinic:tracing:Agent 16.16 s 16.12 s [-0.7%; +1.3%] (no difference)

Commit: e429fa78 · 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.

@amarziali amarziali marked this pull request as ready for review June 29, 2026 14:52
@amarziali amarziali requested a review from a team as a code owner June 29, 2026 14:52
@amarziali amarziali requested review from ValentinZakharov and removed request for a team June 29, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: testing Testing 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