[airflow] branch master updated: Output of installing remaining packages is shown also on success (#12723)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/master by this push:
     new e4cb0ef  Output of installing remaining packages is shown also on success (#12723)
e4cb0ef is described below

commit e4cb0ef192eecfa68929bf213e7f28d2672b9e94
Author: Jarek Potiuk <ja...@polidea.com>
AuthorDate: Tue Dec 1 09:51:05 2020 +0100

    Output of installing remaining packages is shown also on success (#12723)
    
    Previously the output of instaling remaining packges when testing
    provider imports was only shown on error. However it is useful
    to know what's going on even if it clutters the log.
    
    Note that this installation is only needed until we include
    apache-beam in the installed packages on CI.
    
    Related to #12703
    
    This PR shows the output always .
---
 scripts/in_container/run_prepare_provider_readme.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/in_container/run_prepare_provider_readme.sh b/scripts/in_container/run_prepare_provider_readme.sh
index cc7d823..ab0c22f 100755
--- a/scripts/in_container/run_prepare_provider_readme.sh
+++ b/scripts/in_container/run_prepare_provider_readme.sh
@@ -27,10 +27,11 @@ export PYTHONPATH="${AIRFLOW_SOURCES}"
 
 verify_suffix_versions_for_package_preparation
 
+# TODO: remove it when devel_all == devel_ci
 echo
 echo "Installing remaining packages from 'all' extras"
 echo
-pip install -e ".[devel_all]" >"${OUT_FILE_PRINTED_ON_ERROR}" 2>&1
+pip install -e ".[devel_all]"
 
 cd "${AIRFLOW_SOURCES}/provider_packages" || exit 1