Skip to content

ci: experiment with conservative smart system-test selection - #6452

Draft
cgcote wants to merge 1 commit into
masterfrom
codex/smart-system-tests-shadow
Draft

cgcote wants to merge 1 commit into
masterfrom
codex/smart-system-tests-shadow

Conversation

@cgcote

@cgcote cgcote commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds a shadow-only system-test selector that recommends scenario groups from changed tracer source paths. The existing tracer_release system-test execution remains unchanged.

The selector narrows only changes isolated to a recognized product area. Shared, mixed, empty, or unclassified changes fail open to tracer_release. Each recommendation is published in the GitHub job summary with a reason per changed file.

Motivation:

Collect evidence for smarter system-test selection without reducing PR coverage or asking developers to trust an enforcement mechanism.

Change log entry

None. This is internal CI tooling.

Additional Notes:

This first shadow experiment validates deterministic selection, reporting, and conservative fallback behavior. It does not skip tests. Runtime coverage can replace or augment the initial source-area evidence after the shadow data establishes the workflow.

How to test the change?

  • Ruby syntax checks pass for the selector and its spec.
  • The workflow parses as YAML and git diff checks pass.
  • Standalone smoke checks cover isolated AppSec, native profiling, and mixed shared changes.
  • The repository RSpec task could not run locally because the checkout is missing development gems benchmark-ips and zstd-ruby; CI will run spec/github/system_tests_selector_spec.rb.

@cgcote cgcote added the AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos label Oct 8, 2026
private

def escape(value)
value.to_s.gsub("|", "\\|")
@datadog-prod-us1-3

datadog-prod-us1-3 Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Pipelines  Tests

❌ Errors

Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 1 Pipeline job failed

Check Pull Request CI Status | all-jobs-are-green

View more details · View in GitHub Actions

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 90.64% (-0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 261be34 | Docs | View more details | Give us feedback!

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI Generated Largely based on code generated by an AI or LLM. This label is the same across all dd-trace-* repos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants