[GitHub] [airflow] potiuk commented on issue #20254: CI: Rewrite testing examples of images in python

[GitBox <gi...@apache.org>]

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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org