datadog-ci 1.27.0 → 1.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 47e2dae93f942112cea718e6772f7c586214d97a4d3fb9d8ecf9643826c4e9a0
-  data.tar.gz: 80558ee95a33818f80f937981b9c0533c8e638444ac181a31683ea22d53c7ad7
+  metadata.gz: e9b1bc98a924cc6601633633b9feb70bf48f79872266845b55505f1c5cd12837
+  data.tar.gz: e3aa6b68b3d65959178370fb7c4f2ee26094efb8de1da9e7f22d0b35e0a07729
 SHA512:
-  metadata.gz: d442d8921f4ba7fd8ae372544f518edeaac97a6a4b89aa52d2cc8e6ff8035e3131d61104fc4bce7030634970fc2cd52a13d929944ad08497e7abfcf174634ccb
-  data.tar.gz: 4f58631b0792099fdff630a65c10f162e3c933ac2e5c32b53addbaebc4533daf4c87ae7112b69abbd4cf113257a4ac1cc58dc59fb2f8b63184ba3f9abb1aec8d
+  metadata.gz: c0562103f425826f918fd9c2c53ceacdb27e6e19280fd2e9ce47c67e1a7da631ab0b4c6dcf2fe28868aa166f5c5822b160a539f6f65ec462d5f7417c4d491a58
+  data.tar.gz: 4dbf986a49b767d5bc579800bd89716ef24f3263017c40c3410026fc879a5774626bb84f572386eb7ed93298031ed680ccbf1e7ff280061ba2d5d3674ab5c8fe

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [1.28.0] - 2026-02-16
+
+### Added
+
+* Early exit test retries when further retries no longer provide value ([#473][])
+
+### Fixed
+
+* Fix source location tags for specs created by rswag-specs ([#468][])
+
 ## [1.27.0] - 2026-01-22
 
 
@@ -598,7 +608,8 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 
 - Ruby versions < 2.7 no longer supported ([#8][])
 
-[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.27.0...main
+[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.28.0...main
+[1.28.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.27.0...v1.28.0
 [1.27.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.26.0...v1.27.0
 [1.26.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.25.0...v1.26.0
 [1.25.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.24.0...v1.25.0
@@ -849,4 +860,6 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 [#458]: https://github.com/DataDog/datadog-ci-rb/issues/458
 [#460]: https://github.com/DataDog/datadog-ci-rb/issues/460
 [#463]: https://github.com/DataDog/datadog-ci-rb/issues/463
-[#464]: https://github.com/DataDog/datadog-ci-rb/issues/464
+[#464]: https://github.com/DataDog/datadog-ci-rb/issues/464
+[#468]: https://github.com/DataDog/datadog-ci-rb/issues/468
+[#473]: https://github.com/DataDog/datadog-ci-rb/issues/473

data/ext/datadog_ci_native/datadog_cov.c

@@ -381,8 +381,8 @@ static VALUE dd_cov_stop(VALUE self) {
 void Init_datadog_cov(void) {
   VALUE mDatadog = rb_define_module("Datadog");
   VALUE mCI = rb_define_module_under(mDatadog, "CI");
-  VALUE mTestOptimisation = rb_define_module_under(mCI, "TestOptimisation");
-  VALUE mCoverage = rb_define_module_under(mTestOptimisation, "Coverage");
+  VALUE mTestImpactAnalysis = rb_define_module_under(mCI, "TestImpactAnalysis");
+  VALUE mCoverage = rb_define_module_under(mTestImpactAnalysis, "Coverage");
   VALUE cDatadogCov = rb_define_class_under(mCoverage, "DDCov", rb_cObject);
 
   rb_define_alloc_func(cDatadogCov, dd_cov_allocate);

data/lib/datadog/ci/cli/command/exec.rb

@@ -1,5 +1,5 @@
 require_relative "base"
-require_relative "../../test_optimisation/skippable_percentage/estimator"
+require_relative "../../test_impact_analysis/skippable_percentage/estimator"
 
 module Datadog
   module CI

data/lib/datadog/ci/cli/command/skippable_tests_percentage.rb

@@ -1,5 +1,5 @@
 require_relative "base"
-require_relative "../../test_optimisation/skippable_percentage/calculator"
+require_relative "../../test_impact_analysis/skippable_percentage/calculator"
 
 module Datadog
   module CI
@@ -9,7 +9,7 @@ module Datadog
           private
 
           def build_action
-            ::Datadog::CI::TestOptimisation::SkippablePercentage::Calculator.new(
+            ::Datadog::CI::TestImpactAnalysis::SkippablePercentage::Calculator.new(
               rspec_cli_options: (options[:"rspec-opts"] || "").split,
               verbose: !options[:verbose].nil?,
               spec_path: options[:"spec-path"] || "spec"

data/lib/datadog/ci/cli/command/skippable_tests_percentage_estimate.rb

@@ -1,5 +1,5 @@
 require_relative "base"
-require_relative "../../test_optimisation/skippable_percentage/estimator"
+require_relative "../../test_impact_analysis/skippable_percentage/estimator"
 
 module Datadog
   module CI
@@ -9,7 +9,7 @@ module Datadog
           private
 
           def build_action
-            ::Datadog::CI::TestOptimisation::SkippablePercentage::Estimator.new(
+            ::Datadog::CI::TestImpactAnalysis::SkippablePercentage::Estimator.new(
               verbose: !options[:verbose].nil?,
               spec_path: options[:"spec-path"] || "spec"
             )

data/lib/datadog/ci/configuration/components.rb

@@ -17,21 +17,21 @@ require_relative "../remote/library_settings_client"
 require_relative "../test_management/component"
 require_relative "../test_management/null_component"
 require_relative "../test_management/tests_properties"
-require_relative "../test_optimisation/null_component"
-require_relative "../test_optimisation/component"
-require_relative "../test_optimisation/coverage/transport"
+require_relative "../test_impact_analysis/null_component"
+require_relative "../test_impact_analysis/component"
+require_relative "../test_impact_analysis/coverage/transport"
 require_relative "../test_retries/component"
 require_relative "../test_retries/null_component"
 require_relative "../test_discovery/component"
 require_relative "../test_discovery/null_component"
-require_relative "../test_visibility/component"
-require_relative "../test_visibility/flush"
-require_relative "../test_visibility/known_tests"
-require_relative "../test_visibility/null_component"
-require_relative "../test_visibility/serializers/factories/test_level"
-require_relative "../test_visibility/serializers/factories/test_suite_level"
-require_relative "../test_visibility/null_transport"
-require_relative "../test_visibility/transport"
+require_relative "../test_tracing/component"
+require_relative "../test_tracing/flush"
+require_relative "../test_tracing/known_tests"
+require_relative "../test_tracing/null_component"
+require_relative "../test_tracing/serializers/factories/test_level"
+require_relative "../test_tracing/serializers/factories/test_suite_level"
+require_relative "../test_tracing/null_transport"
+require_relative "../test_tracing/transport"
 require_relative "../transport/adapters/telemetry_webmock_safe_adapter"
 require_relative "../transport/api/builder"
 require_relative "../utils/parsing"
@@ -44,12 +44,12 @@ module Datadog
     module Configuration
       # Adds CI behavior to Datadog trace components
       module Components
-        attr_reader :test_visibility, :test_optimisation, :git_tree_upload_worker, :ci_remote, :test_retries,
+        attr_reader :test_tracing, :test_impact_analysis, :git_tree_upload_worker, :ci_remote, :test_retries,
           :test_management, :agentless_logs_submission, :impacted_tests_detection, :test_discovery, :code_coverage
 
         def initialize(settings)
-          @test_optimisation = TestOptimisation::NullComponent.new
-          @test_visibility = TestVisibility::NullComponent.new
+          @test_impact_analysis = TestImpactAnalysis::NullComponent.new
+          @test_tracing = TestTracing::NullComponent.new
           @git_tree_upload_worker = DummyWorker.new
           @ci_remote = Remote::NullComponent.new
           @test_retries = TestRetries::NullComponent.new
@@ -69,8 +69,8 @@ module Datadog
         def shutdown!(replacement = nil)
           super
 
-          @test_visibility&.shutdown!
-          @test_optimisation&.shutdown!
+          @test_tracing&.shutdown!
+          @test_impact_analysis&.shutdown!
           @agentless_logs_submission&.shutdown!
           @test_discovery&.shutdown!
           @code_coverage&.shutdown!
@@ -122,7 +122,7 @@ module Datadog
           # Activate underlying tracing test mode with async worker
           settings.tracing.test_mode.enabled = true
           settings.tracing.test_mode.async = true
-          settings.tracing.test_mode.trace_flush = settings.ci.trace_flush || CI::TestVisibility::Flush::Partial.new
+          settings.tracing.test_mode.trace_flush = settings.ci.trace_flush || CI::TestTracing::Flush::Partial.new
 
           trace_writer_options = settings.ci.writer_options
           trace_writer_options[:shutdown_timeout] = 60
@@ -151,9 +151,9 @@ module Datadog
             tests_properties_client: TestManagement::TestsProperties.new(api: test_visibility_api)
           )
 
-          # @type ivar @test_optimisation: Datadog::CI::TestOptimisation::Component
-          @test_optimisation = build_test_optimisation(settings, test_visibility_api)
-          @test_visibility = TestVisibility::Component.new(
+          # @type ivar @test_impact_analysis: Datadog::CI::TestImpactAnalysis::Component
+          @test_impact_analysis = build_test_impact_analysis(settings, test_visibility_api)
+          @test_tracing = TestTracing::Component.new(
             test_suite_level_visibility_enabled: !settings.ci.force_test_level_visibility,
             logical_test_session_name: settings.ci.test_session_name,
             known_tests_client: build_known_tests_client(settings, test_visibility_api),
@@ -167,7 +167,7 @@ module Datadog
           @code_coverage = build_code_coverage(settings, test_visibility_api)
         end
 
-        def build_test_optimisation(settings, test_visibility_api)
+        def build_test_impact_analysis(settings, test_visibility_api)
           if settings.ci.itr_code_coverage_use_single_threaded_mode &&
               settings.ci.itr_test_impact_analysis_use_allocation_tracing
             Datadog.logger.warn(
@@ -191,7 +191,7 @@ module Datadog
             settings.ci.itr_test_impact_analysis_use_allocation_tracing = false
           end
 
-          TestOptimisation::Component.new(
+          TestImpactAnalysis::Component.new(
             api: test_visibility_api,
             dd_env: settings.env,
             config_tags: custom_configuration(settings),
@@ -243,11 +243,11 @@ module Datadog
 
         def build_tracing_transport(settings, api)
           # NullTransport ignores traces
-          return TestVisibility::NullTransport.new if settings.ci.discard_traces
+          return TestTracing::NullTransport.new if settings.ci.discard_traces
           # nil means that default legacy APM transport will be used (only for very old Datadog Agent versions)
           return nil if api.nil?
 
-          TestVisibility::Transport.new(
+          TestTracing::Transport.new(
             api: api,
             serializers_factory: serializers_factory(settings),
             dd_env: settings.env
@@ -259,7 +259,7 @@ module Datadog
           return nil if api.nil? || settings.ci.discard_traces
 
           AsyncWriter.new(
-            transport: TestOptimisation::Coverage::Transport.new(api: api)
+            transport: TestImpactAnalysis::Coverage::Transport.new(api: api)
           )
         end
 
@@ -283,7 +283,7 @@ module Datadog
         end
 
         def build_known_tests_client(settings, api)
-          TestVisibility::KnownTests.new(
+          TestTracing::KnownTests.new(
             api: api,
             dd_env: settings.env,
             config_tags: custom_configuration(settings)
@@ -330,9 +330,9 @@ module Datadog
 
         def serializers_factory(settings)
           if settings.ci.force_test_level_visibility
-            TestVisibility::Serializers::Factories::TestLevel
+            TestTracing::Serializers::Factories::TestLevel
           else
-            TestVisibility::Serializers::Factories::TestSuiteLevel
+            TestTracing::Serializers::Factories::TestSuiteLevel
           end
         end
 

data/lib/datadog/ci/contrib/cucumber/formatter.rb

@@ -36,14 +36,14 @@ module Datadog
           end
 
           def on_test_run_started(event)
-            test_visibility_component.start_test_session(
+            test_tracing_component.start_test_session(
               tags: {
                 CI::Ext::Test::TAG_FRAMEWORK => Ext::FRAMEWORK,
                 CI::Ext::Test::TAG_FRAMEWORK_VERSION => datadog_integration.version.to_s
               },
               service: datadog_configuration[:service_name]
             )
-            test_visibility_component.start_test_module(Ext::FRAMEWORK)
+            test_tracing_component.start_test_module(Ext::FRAMEWORK)
           end
 
           def on_test_run_finished(event)
@@ -78,7 +78,7 @@ module Datadog
               )
             end
 
-            test_span = test_visibility_component.trace_test(
+            test_span = test_tracing_component.trace_test(
               event.test_case.name,
               test_suite_name,
               tags: tags,
@@ -90,18 +90,18 @@ module Datadog
           end
 
           def on_test_case_finished(event)
-            test_span = test_visibility_component.active_test
+            test_span = test_tracing_component.active_test
             return if test_span.nil?
 
             finish_span(test_span, event.result)
           end
 
           def on_test_step_started(event)
-            test_visibility_component.trace(event.test_step.to_s, type: Ext::STEP_SPAN_TYPE)
+            test_tracing_component.trace(event.test_step.to_s, type: Ext::STEP_SPAN_TYPE)
           end
 
           def on_test_step_finished(event)
-            current_step_span = test_visibility_component.active_span
+            current_step_span = test_tracing_component.active_span
             return if current_step_span.nil?
 
             finish_span(current_step_span, event.result)
@@ -135,8 +135,8 @@ module Datadog
           end
 
           def finish_session(result)
-            test_session = test_visibility_component.active_test_session
-            test_module = test_visibility_component.active_test_module
+            test_session = test_tracing_component.active_test_session
+            test_module = test_tracing_component.active_test_module
 
             return unless test_session && test_module
 
@@ -155,7 +155,7 @@ module Datadog
           def start_test_suite(test_suite_name, tags: {})
             finish_current_test_suite
 
-            @current_test_suite = test_visibility_component.start_test_suite(test_suite_name, tags: tags)
+            @current_test_suite = test_tracing_component.start_test_suite(test_suite_name, tags: tags)
           end
 
           def finish_current_test_suite
@@ -208,8 +208,8 @@ module Datadog
             Datadog.configuration.ci[:cucumber]
           end
 
-          def test_visibility_component
-            Datadog.send(:components).test_visibility
+          def test_tracing_component
+            Datadog.send(:components).test_tracing
           end
 
           def test_suite_source_file_tags(test_case)

data/lib/datadog/ci/contrib/knapsack/runner.rb

@@ -20,7 +20,7 @@ module Datadog
               return super unless datadog_configuration[:enabled]
 
               # @type var test_session: Datadog::CI::TestSession?
-              test_session = test_visibility_component.start_test_session(
+              test_session = test_tracing_component.start_test_session(
                 tags: {
                   CI::Ext::Test::TAG_FRAMEWORK => CI::Contrib::RSpec::Ext::FRAMEWORK,
                   CI::Ext::Test::TAG_FRAMEWORK_VERSION => datadog_integration.version.to_s
@@ -29,7 +29,7 @@ module Datadog
               )
 
               # @type var test_module: Datadog::CI::TestModule?
-              test_module = test_visibility_component.start_test_module(CI::Contrib::RSpec::Ext::FRAMEWORK)
+              test_module = test_tracing_component.start_test_module(CI::Contrib::RSpec::Ext::FRAMEWORK)
 
               result = super
               return result unless test_module && test_session
@@ -57,8 +57,8 @@ module Datadog
               Datadog.configuration.ci[:rspec]
             end
 
-            def test_visibility_component
-              Datadog.send(:components).test_visibility
+            def test_tracing_component
+              Datadog.send(:components).test_tracing
             end
           end
         end

data/lib/datadog/ci/contrib/minitest/helpers.rb

@@ -23,8 +23,8 @@ module Datadog
               {}
             end
 
-            test_visibility_component = Datadog.send(:components).test_visibility
-            test_suite = test_visibility_component.start_test_suite(
+            test_tracing_component = Datadog.send(:components).test_tracing
+            test_suite = test_tracing_component.start_test_suite(
               test_suite_name,
               tags: test_suite_tags
             )

data/lib/datadog/ci/contrib/minitest/runnable_minitest_6.rb

@@ -34,8 +34,8 @@ module Datadog
               Datadog.configuration.ci[:minitest]
             end
 
-            def _dd_test_visibility_component
-              Datadog.send(:components).test_visibility
+            def _dd_test_tracing_component
+              Datadog.send(:components).test_tracing
             end
 
             def _dd_test_retries_component

data/lib/datadog/ci/contrib/minitest/runner.rb

@@ -21,7 +21,7 @@ module Datadog
 
               tests_count = ::Minitest::Runnable.runnables.sum { |runnable| runnable.runnable_methods.size }
 
-              test_visibility_component.start_test_session(
+              test_tracing_component.start_test_session(
                 tags: {
                   CI::Ext::Test::TAG_FRAMEWORK => Ext::FRAMEWORK,
                   CI::Ext::Test::TAG_FRAMEWORK_VERSION => datadog_integration.version.to_s
@@ -32,7 +32,7 @@ module Datadog
                 # we need to always start/stop test suites in the parent process in this case
                 local_test_suites_mode: false
               )
-              test_visibility_component.start_test_module(Ext::FRAMEWORK)
+              test_tracing_component.start_test_module(Ext::FRAMEWORK)
             end
 
             def old_run_one_method(klass, method_name)
@@ -54,7 +54,7 @@ module Datadog
               # get the current test suite and mark this method as done, so we can check if all tests were executed
               # for this test suite
               test_suite_name = Helpers.test_suite_name(klass, method_name)
-              test_suite = test_visibility_component.active_test_suite(test_suite_name)
+              test_suite = test_tracing_component.active_test_suite(test_suite_name)
               test_suite&.expected_test_done!(method_name)
 
               result
@@ -80,8 +80,8 @@ module Datadog
               Datadog.configuration.ci[:minitest]
             end
 
-            def test_visibility_component
-              Datadog.send(:components).test_visibility
+            def test_tracing_component
+              Datadog.send(:components).test_tracing
             end
 
             def test_retries_component

data/lib/datadog/ci/contrib/minitest/test.rb

@@ -44,7 +44,7 @@ module Datadog
               tags[CI::Ext::Test::TAG_SOURCE_START] = first_line_number.to_s if first_line_number
               tags[CI::Ext::Test::TAG_SOURCE_END] = last_line_number.to_s if last_line_number
 
-              test_span = _dd_test_visibility_component.trace_test(
+              test_span = _dd_test_tracing_component.trace_test(
                 name,
                 test_suite_name,
                 tags: tags,
@@ -59,7 +59,7 @@ module Datadog
             end
 
             def after_teardown
-              test_span = _dd_test_visibility_component.active_test
+              test_span = _dd_test_tracing_component.active_test
               return super unless test_span
 
               finish_with_result(test_span, result_code)
@@ -95,8 +95,8 @@ module Datadog
               Datadog.configuration.ci[:minitest]
             end
 
-            def _dd_test_visibility_component
-              Datadog.send(:components).test_visibility
+            def _dd_test_tracing_component
+              Datadog.send(:components).test_tracing
             end
           end
 

data/lib/datadog/ci/contrib/parallel_tests/cli.rb

@@ -18,7 +18,7 @@ module Datadog
               return super if @runner != ::ParallelTests::RSpec::Runner
 
               begin
-                test_session = test_visibility_component.start_test_session(
+                test_session = test_tracing_component.start_test_session(
                   tags: {
                     CI::Ext::Test::TAG_FRAMEWORK => CI::Contrib::RSpec::Ext::FRAMEWORK,
                     CI::Ext::Test::TAG_FRAMEWORK_VERSION => datadog_extract_rspec_version
@@ -27,10 +27,10 @@ module Datadog
                   estimated_total_tests_count: 10_000, # temporary value, updated by child processes
                   distributed: true
                 )
-                test_module = test_visibility_component.start_test_module("rspec")
+                test_module = test_tracing_component.start_test_module("rspec")
 
                 options[:env] ||= {}
-                options[:env][CI::Ext::Settings::ENV_TEST_VISIBILITY_DRB_SERVER_URI] = test_visibility_component.context_service_uri
+                options[:env][CI::Ext::Settings::ENV_TEST_VISIBILITY_DRB_SERVER_URI] = test_tracing_component.context_service_uri
 
                 super
               ensure
@@ -42,8 +42,8 @@ module Datadog
             def any_test_failed?(test_results)
               res = super
 
-              test_session = test_visibility_component.active_test_session
-              test_module = test_visibility_component.active_test_module
+              test_session = test_tracing_component.active_test_session
+              test_module = test_tracing_component.active_test_module
               if res
                 test_module&.failed!
                 test_session&.failed!
@@ -73,8 +73,8 @@ module Datadog
               Datadog.configuration.ci[:paralleltests]
             end
 
-            def test_visibility_component
-              Datadog.send(:components).test_visibility
+            def test_tracing_component
+              Datadog.send(:components).test_tracing
             end
           end
         end