bhirsz commented on PR #24058: URL: https://github.com/apache/airflow/pull/24058#issuecomment-1143202906
Looks good - especially the utility scripts! We are checking for required env vars (https://github.com/apache/airflow/blob/1dccaad46b901189c1928cef8419f1ea1160d550/tests/system/conftest.py#L44) and it may be good idea to offer some kind of interface that would allow every provider to provide some function to retrieve/set env variable value before running the tests. That's outside of this PR - I'm just wondering what we can steal from some of the ideas from different PR to get some nice common solutions :) -- 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]
