workflows: update marker from require_non_self_hosted to require_docker_in_docker in Pposargs

[mohamedawnallah]

Description

I forgot to update the marker. It is follow-up on #35585

---

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

• Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
• Update CHANGES.md with noteworthy changes.
• If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

[mohamedawnallah]

cc @damccorm

[damccorm]

Thanks!

[damccorm]

Hm, it looks like this will currently not work because of

beam/sdks/python/scripts/run_pytest.sh

Line 34 in 8930c8d

echo "$0 cannot be called with -m as it interferes with 'no_xdist' logic, see BEAM-12985."

[damccorm]

https://github.com/apache/beam/actions/runs/16300739721/job/46034393155

[damccorm]

This is the context - https://issues.apache.org/jira/browse/BEAM-12985

[damccorm]

Probably a simple fix would be to replace

beam/sdks/python/scripts/run_pytest.sh

Line 34 in 8930c8d

echo "$0 cannot be called with -m as it interferes with 'no_xdist' logic, see BEAM-12985."

and instead of failing, collect the extra -m arguments and add the appropriate not no_xdist and no_xdist flags in

beam/sdks/python/scripts/run_pytest.sh

Lines 44 to 48 in 8930c8d

	pytest -v -rs -o junit_suite_name=${envname} \
	--junitxml=pytest_${envname}.xml -m 'not no_xdist' -n 6 --import-mode=importlib ${pytest_args} --pyargs ${posargs}
	status1=$?
	pytest -v -rs -o junit_suite_name=${envname}_no_xdist \
	--junitxml=pytest_${envname}_no_xdist.xml -m 'no_xdist' --import-mode=importlib ${pytest_args} --pyargs ${posargs}

[mohamedawnallah]

Just seen this notification comments...

[mohamedawnallah]

Probably a simple fix would be to replace

beam/sdks/python/scripts/run_pytest.sh

Line 34 in 8930c8d

echo "$0 cannot be called with -m as it interferes with 'no_xdist' logic, see BEAM-12985."

and instead of failing, collect the extra -m arguments and add the appropriate not no_xdist and no_xdist flags in

beam/sdks/python/scripts/run_pytest.sh

Lines 44 to 48 in 8930c8d

	pytest -v -rs -o junit_suite_name=${envname} \
	--junitxml=pytest_${envname}.xml -m 'not no_xdist' -n 6 --import-mode=importlib ${pytest_args} --pyargs ${posargs}
	status1=$?
	pytest -v -rs -o junit_suite_name=${envname}_no_xdist \
	--junitxml=pytest_${envname}_no_xdist.xml -m 'no_xdist' --import-mode=importlib ${pytest_args} --pyargs ${posargs}

This approach looks good 👍

[mohamedawnallah]

I've also another approach of just using -k instead -m for keyword matching but I think your approach is more reliable in the future

(.venv) dev@dev:~/beam_dev/beam/sdks/python/apache_beam/ml/rag/enrichment$ pytest -v -k "require_docker_in_docker"  --test-pipeline-options="--runner=DirectRunner"
====================================================== test session starts =======================================================
platform linux -- Python 3.9.22, pytest-7.4.4, pluggy-1.6.0 -- /home/dev/beam_dev/beam/sdks/python/.venv/bin/python
cachedir: .pytest_cache
hypothesis profile 'default'
rootdir: /home/dev/beam_dev/beam/sdks/python
configfile: pytest.ini
plugins: xdist-3.8.0, requests-mock-1.12.1, timeout-2.4.0, anyio-4.9.0, hypothesis-6.135.31
timeout: 600.0s
timeout method: signal
timeout func_only: False
collected 36 items / 27 deselected / 9 selected

milvus_search_it_test.py::TestMilvusSearchEnrichment::test_empty_input_chunks PASSED                                       [ 11%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_filtered_search_with_bm25_full_text_and_batching PASSED         [ 22%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_filtered_search_with_cosine_similarity_and_batching PASSED      [ 33%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_hybrid_search PASSED                                            [ 44%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_invalid_query_on_non_existent_collection PASSED                 [ 55%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_invalid_query_on_non_existent_field PASSED                      [ 66%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_keyword_search_with_inner_product_sparse_embedding PASSED       [ 77%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_vector_search_with_euclidean_distance PASSED                    [ 88%]
milvus_search_it_test.py::TestMilvusSearchEnrichment::test_vector_search_with_inner_product_similarity PASSED              [100%]

=============================================== 9 passed, 27 deselected in 30.26s ================================================