Skip to content

docs(test_parallelization): reflect pytest --command support - #39548

Merged
gnufede merged 3 commits into
masterfrom
gnufede/ddtest-docs-update
Sep 1, 2026
Merged

docs(test_parallelization): reflect pytest --command support#39548
gnufede merged 3 commits into
masterfrom
gnufede/ddtest-docs-update

Conversation

@gnufede

@gnufede gnufede commented Aug 28, 2026

Copy link
Copy Markdown
Member

What does this PR do? What is the motivation?

Updates the Test Parallelization docs to reflect that since ddtest 1.7.0, Python/pytest honors --command / DD_TEST_OPTIMIZATION_RUNNER_COMMAND to override the default python -m pytest invocation. Previously the docs stated pytest ignored --command and that PYTEST_ADDOPTS was the only way to customize the pytest invocation.

Motivation: ddtest PR DataDog/ddtest#123 added commandOverride support to PyTest.RunTests and DiscoverTests.

Changes:

  • configuration.md: DD_TEST_OPTIMIZATION_RUNNER_COMMAND entry, "Custom test commands" pytest paragraph, and "Pytest test discovery" line now describe the override.
  • best_practices.md: "Configure pytest" paragraph now describes the override.
  • PYTEST_ADDOPTS guidance is retained, as it remains valid for passing extra pytest flags without changing the base command.

Merge readiness

  • Ready for merge

AI assistance

Used AI to locate the affected doc sections and draft the wording. Reviewed and adjusted manually to follow the Datadog documentation style guide (short sentences, active voice, imperative mood).

Additional notes

Depends on ddtest 1.7.0, which is not yet released. The "since 1.7.0" notes are written to match the upcoming release.

@github-actions

Copy link
Copy Markdown
Contributor

ddtest PR DataDog/ddtest#123 made PyTest.RunTests and DiscoverTests
honor commandOverride (--command / DD_TEST_OPTIMIZATION_RUNNER_COMMAND)
instead of hardcoding 'python -m pytest'. Update the configuration and
best_practices pages to describe the override and retain PYTEST_ADDOPTS
guidance for passing extra flags.
@gnufede
gnufede force-pushed the gnufede/ddtest-docs-update branch from e019c11 to bd1bde6 Compare August 28, 2026 09:44
@gnufede
gnufede marked this pull request as ready for review September 1, 2026 07:26
@gnufede
gnufede requested review from a team as code owners September 1, 2026 07:26

@datadog-official datadog-official Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Datadog Autotest: PASS

More details

The pages state that pytest command overrides require ddtest 1.7.0 or later. They keep PYTEST_ADDOPTS guidance for extra flags.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Datadog Autotest · Commit bd1bde6 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest

@hestonhoffman hestonhoffman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left you some edits to replace "since." The word can be a little ambiguous - it can also mean "because" in come contexts, especially at the start of a sentence.

Comment thread hugo/content/en/tests/test_parallelization/best_practices.md Outdated
Comment thread hugo/content/en/tests/test_parallelization/configuration.md Outdated
Comment thread hugo/content/en/tests/test_parallelization/configuration.md Outdated
Co-authored-by: Heston Hoffman <hestonhoffman@gmail.com>
@gnufede
gnufede merged commit e7378ca into master Sep 1, 2026
25 of 26 checks passed
@gnufede
gnufede deleted the gnufede/ddtest-docs-update branch September 1, 2026 18:21
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.

3 participants