Skip to content

Conversation

@anmarchenko
Copy link
Member

What does this PR do?

Migrates Datadog Test Optimization product (aka tracing tests) from outdated ddtrace gem to new datadog-ci gem

Review checklist

Please check relevant items below:

  • N/A - This PR includes all newly recorded cassettes for any modified tests.

  • This PR does not rely on API client schema changes.

    • The CI should be fully passing.

@anmarchenko anmarchenko force-pushed the anmarchenko/update_datadog_ci_setup branch from ae3967a to d520d92 Compare January 3, 2025 11:23
@datadog-datadog-prod-us1
Copy link

datadog-datadog-prod-us1 bot commented Jan 3, 2025

Datadog Report

Branch report: anmarchenko/update_datadog_ci_setup
Commit report: 1e34e97
Test service: datadog-api-client-ruby

✅ 0 Failed, 2821 Passed, 59 Skipped, 2m 40.59s Total Time
❄️ 11 New Flaky

New Flaky Tests (11)

  • Get all application keys returns "OK" response - Key Management at features/v2/key_management.feature

  • Create a Basic HTTP custom destination returns "OK" response - Logs Custom Destinations at features/v2/logs_custom_destinations.feature

  • Create a Custom Header HTTP custom destination returns "OK" response - Logs Custom Destinations at features/v2/logs_custom_destinations.feature

  • Create an Elasticsearch custom destination returns "OK" response - Logs Custom Destinations at features/v2/logs_custom_destinations.feature - Last Failure

    Expand for error
     
     expected: 200
          got: 422
     
     (compared using ==)
     
    
  • Create a Splunk custom destination returns "OK" response - Logs Custom Destinations at features/v2/logs_custom_destinations.feature

@skarimo skarimo merged commit cd87ab5 into master Jan 22, 2025
13 of 14 checks passed
@skarimo skarimo deleted the anmarchenko/update_datadog_ci_setup branch January 22, 2025 15:08
github-actions bot pushed a commit that referenced this pull request Jan 22, 2025
…#2172)

Co-authored-by: skarimo <40482491+skarimo@users.noreply.github.com> cd87ab5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants