potiuk commented on issue #20254:
URL: https://github.com/apache/airflow/issues/20254#issuecomment-1008180796


   Yeah. I think this is as simple as this -  just getting rid of any "bash" 
executed.
   
   Currently the tests to execute is really executing "pytest docker_tests" 
(more or less)
   
   * script_init is not needed (as in most of our python code we can reuse 
common code)
   * then what really want to do is to initialize the virtualenv and move the 
docker_tests.py script to "dev/breeze" so that it becomes a standalone command 
- similar to "freespace" (say "dockertest") 
   
   Eventually I think what should happen is that the docker tests "CI job" for 
it should be: 
   1) initialize the venv by `pip install ../dev/breeze" (actually this will 
likely be done by a previous freespace step already)
   2) switch to the "airflow" source directory
   3) running pytest (it can be easily run from python code) 
https://docs.pytest.org/en/latest/how-to/usage.html#calling-pytest-from-python-code
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to