Skip to content

enable java cap test#6467

Draft
raphaelgavache wants to merge 2 commits intomainfrom
raphael/enable_java_cap_test
Draft

enable java cap test#6467
raphaelgavache wants to merge 2 commits intomainfrom
raphael/enable_java_cap_test

Conversation

@raphaelgavache
Copy link
Member

@raphaelgavache raphaelgavache commented Mar 11, 2026

Enable system test for DataDog/dd-trace-java#10715

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
Contributor

github-actions bot commented Mar 11, 2026

CODEOWNERS have been resolved as:

.github/workflows/run-end-to-end.yml                                    @DataDog/system-tests-core
manifests/java.yml                                                      @DataDog/asm-java @DataDog/apm-java

@datadog-official
Copy link

datadog-official bot commented Mar 11, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 21 Tests failed

tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease.test_sampling_rate_capped_increase[akka-http] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: Expected at least one intermediate _dd.agent_psr value between 0.1 and 1.0, but only saw: [0.1, 1.0]. The tracer should cap sampling rate increases to 2x per interval, not jump directly.
assert 0 > 0
 +  where 0 = len([])

self = <tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease object at 0x7f8e4c72f8c0>

    def test_sampling_rate_capped_increase(self):
        """Verify that the tracer ramps up sampling rate gradually instead of jumping directly."""
        # Only look at spans from AFTER the low-rate phase to avoid the default 1.0 at startup
        agent_psr_values = set()
...
tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease.test_sampling_rate_capped_increase[chi] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: Expected to see the low rate (0.1) in _dd.agent_psr values: [0.8333, 1.0]
assert False
 +  where False = any(<generator object Test_SamplingRateCappedIncrease.test_sampling_rate_capped_increase.<locals>.<genexpr> at 0x7f76c45a04a0>)

self = <tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease object at 0x7f76e06629c0>

    def test_sampling_rate_capped_increase(self):
        """Verify that the tracer ramps up sampling rate gradually instead of jumping directly."""
        # Only look at spans from AFTER the low-rate phase to avoid the default 1.0 at startup
        agent_psr_values = set()
...
tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease.test_sampling_rate_capped_increase[echo] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: Expected to see the low rate (0.1) in _dd.agent_psr values: [1.0]
assert False
 +  where False = any(<generator object Test_SamplingRateCappedIncrease.test_sampling_rate_capped_increase.<locals>.<genexpr> at 0x7fda94765b60>)

self = <tests.test_sampling_rate_capping.Test_SamplingRateCappedIncrease object at 0x7fdabc56f140>

    def test_sampling_rate_capped_increase(self):
        """Verify that the tracer ramps up sampling rate gradually instead of jumping directly."""
        # Only look at spans from AFTER the low-rate phase to avoid the default 1.0 at startup
        agent_psr_values = set()
...
View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: c204cca | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

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.

1 participant