You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by jl...@apache.org on 2017/02/08 13:33:25 UTC
incubator-airflow git commit: [AIRFLOW-830][AIRFLOW-829][AIRFLOW-88]
Reduce Travis log verbosity
Repository: incubator-airflow
Updated Branches:
refs/heads/master d428a9028 -> 61fb6a5e5
[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/61fb6a5e
Tree: http://git-wip-us.apache.org/repos/asf/incubator-airflow/tree/61fb6a5e
Diff: http://git-wip-us.apache.org/repos/asf/incubator-airflow/diff/61fb6a5e
Branch: refs/heads/master
Commit: 61fb6a5e52c3e53dba7214d0de24c5ac5953c857
Parents: d428a90
Author: Jeremiah Lowin <jl...@apache.org>
Authored: Wed Feb 8 08:32:25 2017 -0500
Committer: Jeremiah Lowin <jl...@apache.org>
Committed: Wed Feb 8 08:32:25 2017 -0500
----------------------------------------------------------------------
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/61fb6a5e/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/61fb6a5e/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