Skip to content

Auto-activate rust easy wins for feature-flagging-and-experimentation-sdk#7041

Open
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
easy-win/feature-flagging-and-experimentation-sdk/rust
Open

Auto-activate rust easy wins for feature-flagging-and-experimentation-sdk#7041
dd-octo-sts[bot] wants to merge 1 commit into
mainfrom
easy-win/feature-flagging-and-experimentation-sdk/rust

Conversation

@dd-octo-sts
Copy link
Copy Markdown
Contributor

@dd-octo-sts dd-octo-sts Bot commented May 29, 2026

Automated activation of easy-win tests for rust owned by feature-flagging-and-experimentation-sdk
View nightly workflow run

  • If you approve this PR please also merge it.
  • If the tests are failing it might be due to a change made since the last nightly system-tests run. You can close the PR, an updated one will be available tomorrow.
  • If you close the PR please also delete the branch

…tation-sdk

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@dd-octo-sts dd-octo-sts Bot requested a review from a team as a code owner May 29, 2026 07:10
@dd-octo-sts dd-octo-sts Bot enabled auto-merge (squash) May 29, 2026 07:10
@github-actions
Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/rust.yml                                                      @DataDog/apm-rust

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a4a7a08534

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread manifests/rust.yml
tests/parametric/test_dynamic_configuration.py::TestDynamicConfigV2: missing_feature # Created by easy win activation script
tests/parametric/test_ffe/test_dynamic_evaluation.py::Test_Feature_Flag_Dynamic_Evaluation: missing_feature
tests/parametric/test_ffe/test_span_enrichment.py: missing_feature
tests/parametric/test_ffe/test_span_enrichment.py: '>=0.3.4-dev+c6880b8' # TODO: a lower version might be supported
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep pure utility span-enrichment tests skipped

With this file-level activation, Rust versions >=0.3.4-dev+c6880b8 now run every descendant not explicitly overridden; the added overrides skip all tracer-backed span-enrichment classes, leaving only Test_Span_Enrichment_Delta_Varint, which is a pure Python utility test with no test_library/Rust tracer interaction. That makes the Rust manifest report FFE span-enrichment coverage as passing without exercising the product behavior, so either keep the file skipped or add an explicit override for the delta-varint class until a tracer-backed class is enabled.

Useful? React with 👍 / 👎.

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.

0 participants