Abacn commented on code in PR #35585: URL: https://github.com/apache/beam/pull/35585#discussion_r2205671469
########## sdks/python/pytest.ini: ########## @@ -70,6 +70,7 @@ markers = uses_mock_api: tests that uses the mock API cluster. uses_feast: tests that uses feast in some way gemini_postcommit: gemini postcommits that need additional deps. + require_non_self_hosted: tests which can't be run on Beam's self hosted GitHub Actions runners. Usually this is because of issues with docker-in-docker or other environmental challenges. Context: https://github.com/apache/beam/pull/35585 Review Comment: we try to decouple GitHub Actions with the other part of Beam code, so developers can run tests on their machines. Can we rename "require_non_self_hosted" to something like require_docker_in_docker or other informative marker names that does not assume the tests run in GitHub Action ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org