Skip to content

Support multiple ressources for stats#6375

Open
VianneyRuhlmann wants to merge 2 commits intomainfrom
vianney/support-multiple-resources-for-stats
Open

Support multiple ressources for stats#6375
VianneyRuhlmann wants to merge 2 commits intomainfrom
vianney/support-multiple-resources-for-stats

Conversation

@VianneyRuhlmann
Copy link
Contributor

@VianneyRuhlmann VianneyRuhlmann commented Feb 24, 2026

Motivation

CSS tests are failing on some weblogs (django, tornado) because the resource for requests to the /stats-unique endpoint isn't GET /stats-unique.

Changes

  • Add ressource list support to get_stats
  • Add a list of resource name for all weblogs
  • remove bug/unimplemented marker

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

CODEOWNERS have been resolved as:

manifests/python.yml                                                    @DataDog/apm-python @DataDog/asm-python
tests/stats/test_stats.py                                               @DataDog/system-tests-core
utils/interfaces/_agent.py                                              @DataDog/system-tests-core

@datadog-official
Copy link

datadog-official bot commented Feb 24, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: a3735e1 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@VianneyRuhlmann VianneyRuhlmann changed the title support multiple ressources for stats Support multiple ressources for stats Feb 25, 2026
@VianneyRuhlmann VianneyRuhlmann marked this pull request as ready for review February 25, 2026 11:50
@VianneyRuhlmann VianneyRuhlmann requested review from a team as code owners February 25, 2026 11:50
@VianneyRuhlmann VianneyRuhlmann requested review from KowalskiThomas, gnufede and ichinaski and removed request for a team February 25, 2026 11:50
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