Skip to content

java: mark appsec flaky test as bug to monitor the flakiness#6423

Merged
florentinl merged 1 commit into
mainfrom
florentin.labelle/mark-java-test-resource-renaming-as-bug
Mar 4, 2026
Merged

java: mark appsec flaky test as bug to monitor the flakiness#6423
florentinl merged 1 commit into
mainfrom
florentin.labelle/mark-java-test-resource-renaming-as-bug

Conversation

@florentinl
Copy link
Copy Markdown
Contributor

@florentinl florentinl commented Mar 3, 2026

Motivation

This test was marked as flaky in #6358.
However flaky tests are skipped so it makes it impossible to monitor the recovery.

I have made a change to the test and fixed an aggregation problem in #6374 that may fix the issue, I'd like to check that the test is not flaky for a week before enabling it back.

Without any knowledge in Java and knowing that its works without problem in Python and now in NodeJS too this is a bit hard for me to debug. Help welcomed.

Changes

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added, removed or renamed?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 3, 2026

CODEOWNERS have been resolved as:

manifests/java.yml                                                      @DataDog/asm-java @DataDog/apm-java

@florentinl florentinl marked this pull request as ready for review March 4, 2026 08:25
@florentinl florentinl requested review from a team as code owners March 4, 2026 08:25
@florentinl florentinl merged commit aee0921 into main Mar 4, 2026
134 of 136 checks passed
@florentinl florentinl deleted the florentin.labelle/mark-java-test-resource-renaming-as-bug branch March 4, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants