You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by cr...@apache.org on 2017/10/06 17:46:56 UTC

incubator-airflow git commit: [AIRFLOW-1690] Add detail to gcs error messages

Repository: incubator-airflow
Updated Branches:
  refs/heads/master 3300d6167 -> 2078daca3


[AIRFLOW-1690] Add detail to gcs error messages

Provide the system error detail in the log
message.

Closes #2670 from wrp/master


Project: http://git-wip-us.apache.org/repos/asf/incubator-airflow/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-airflow/commit/2078daca
Tree: http://git-wip-us.apache.org/repos/asf/incubator-airflow/tree/2078daca
Diff: http://git-wip-us.apache.org/repos/asf/incubator-airflow/diff/2078daca

Branch: refs/heads/master
Commit: 2078daca3948d1969919a27e9b677de50125e5e3
Parents: 3300d61
Author: William Pursell <wi...@upthere.com>
Authored: Fri Oct 6 10:46:42 2017 -0700
Committer: Chris Riccomini <cr...@apache.org>
Committed: Fri Oct 6 10:46:42 2017 -0700

----------------------------------------------------------------------
 airflow/utils/log/gcs_task_handler.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-airflow/blob/2078daca/airflow/utils/log/gcs_task_handler.py
----------------------------------------------------------------------
diff --git a/airflow/utils/log/gcs_task_handler.py b/airflow/utils/log/gcs_task_handler.py
index bb40e11..c11e7ad 100644
--- a/airflow/utils/log/gcs_task_handler.py
+++ b/airflow/utils/log/gcs_task_handler.py
@@ -167,8 +167,8 @@ class GCSTaskHandler(FileTaskHandler, LoggingMixin):
                 # closed).
                 tmpfile.flush()
                 self.hook.upload(bkt, blob, tmpfile.name)
-        except:
-            self.log.error('Could not write logs to %s', remote_log_location)
+        except Exception as e:
+            self.log.error('Could not write logs to %s: %s', remote_log_location, e)
 
     def parse_gcs_url(self, gsurl):
         """