[AIRFLOW-830][AIRFLOW-829][AIRFLOW-88] Reduce Travis log verbosity [AIRFLOW-829][AIRFLOW-88] Reduce verbosity of Travis tests
Remove the -s flag for Travis unit tests to suppress output from successful tests. [AIRFLOW-830] Reduce plugins manager verbosity The plugin manager prints all status to INFO, which is unnecessary and overly verbose. Closes #2049 from jlowin/reduce-logs Project: http://git-wip-us.apache.org/repos/asf/incubator-airflow/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-airflow/commit/3b1e81ac Tree: http://git-wip-us.apache.org/repos/asf/incubator-airflow/tree/3b1e81ac Diff: http://git-wip-us.apache.org/repos/asf/incubator-airflow/diff/3b1e81ac Branch: refs/heads/v1-8-test Commit: 3b1e81ac9e8e97b6d2a4c3217df81db9ddbd0900 Parents: e1d0adb Author: Jeremiah Lowin <[email protected]> Authored: Wed Feb 8 08:32:25 2017 -0500 Committer: Bolke de Bruin <[email protected]> Committed: Sun Mar 12 08:03:25 2017 -0700 ---------------------------------------------------------------------- airflow/plugins_manager.py | 4 ++-- run_unit_tests.sh | 36 ++++++++++++++++++++++++------------ 2 files changed, 26 insertions(+), 14 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-airflow/blob/3b1e81ac/airflow/plugins_manager.py ---------------------------------------------------------------------- diff --git a/airflow/plugins_manager.py b/airflow/plugins_manager.py index e0af20c..83aae23 100644 --- a/airflow/plugins_manager.py +++ b/airflow/plugins_manager.py @@ -72,7 +72,7 @@ for root, dirs, files in os.walk(plugins_folder, followlinks=True): if file_ext != '.py': continue - logging.info('Importing plugin module ' + filepath) + logging.debug('Importing plugin module ' + filepath) # normalize root path as namespace namespace = '_'.join([re.sub(norm_pattern, '__', root), mod_name]) @@ -92,7 +92,7 @@ for root, dirs, files in os.walk(plugins_folder, followlinks=True): def make_module(name, objects): - logging.info('Creating module ' + name) + logging.debug('Creating module ' + name) name = name.lower() module = imp.new_module(name) module._name = name.split('.')[-1] http://git-wip-us.apache.org/repos/asf/incubator-airflow/blob/3b1e81ac/run_unit_tests.sh ---------------------------------------------------------------------- diff --git a/run_unit_tests.sh b/run_unit_tests.sh index c922a55..b3ee074 100755 --- a/run_unit_tests.sh +++ b/run_unit_tests.sh @@ -28,17 +28,6 @@ export AIRFLOW_USE_NEW_IMPORTS=1 # any argument received is overriding the default nose execution arguments: nose_args=$@ -if [ -z "$nose_args" ]; then - nose_args="--with-coverage \ ---cover-erase \ ---cover-html \ ---cover-package=airflow \ ---cover-html-dir=airflow/www/static/coverage \ ---with-ignore-docstrings \ --s \ --v \ ---logging-level=DEBUG " -fi #--with-doctest @@ -50,7 +39,18 @@ yes | airflow resetdb airflow initdb if [ "${TRAVIS}" ]; then - # For impersonation tests running on SQLite on Travis, make the database world readable so other + if [ -z "$nose_args" ]; then + nose_args="--with-coverage \ + --cover-erase \ + --cover-html \ + --cover-package=airflow \ + --cover-html-dir=airflow/www/static/coverage \ + --with-ignore-docstrings \ + -v \ + --logging-level=DEBUG " + fi + + # For impersonation tests running on SQLite on Travis, make the database world readable so other # users can update it AIRFLOW_DB="/home/travis/airflow/airflow.db" if [ -f "${AIRFLOW_DB}" ]; then @@ -60,6 +60,18 @@ if [ "${TRAVIS}" ]; then # For impersonation tests on Travis, make airflow accessible to other users via the global PATH # (which contains /usr/local/bin) sudo ln -s "${VIRTUAL_ENV}/bin/airflow" /usr/local/bin/ +else + if [ -z "$nose_args" ]; then + nose_args="--with-coverage \ + --cover-erase \ + --cover-html \ + --cover-package=airflow \ + --cover-html-dir=airflow/www/static/coverage \ + --with-ignore-docstrings \ + -s \ + -v \ + --logging-level=DEBUG " + fi fi echo "Starting the unit tests with the following nose arguments: "$nose_args
