system-tests icon indicating copy to clipboard operation
system-tests copied to clipboard

[PROF-12799] Fix python crashtracker smoke test

Open gyuheon0h opened this issue 1 month ago • 2 comments
trafficstars

Motivation

Related to this https://datadoghq.atlassian.net/browse/INPLAT-603

Changes

  1. Update JSON schema for crash report; The payload field of the telemetry log is an Array, as seen in the examples attached to this JIRA ticket. We should validate correctly, but I kept the previous schema for backwards compatibility.
  2. Added /crashme endpoints for node and python test app. This is because, when debugging the test that fails in CI locally in workspaces, I noticed that the test behavior was different depending on whether I ran the failing test as part of the default scenario, or the specific docker_ssi_crashtracking scenario. The test seemed to fail when run all together, but when run specifically within its scenario, it would pass. I updated the test setup code to simply use Weblog for whatever scenario env it is currently in.

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

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

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • [ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
  • [ ] No system-tests internal is modified. Otherwise, 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 (or removed)?

gyuheon0h avatar Oct 22 '25 18:10 gyuheon0h