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

Reply via email to