[jira] [Commented] (AIRFLOW-2548) Output Plugin Import Errors to WebUI

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/AIRFLOW-2548?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16732036#comment-16732036 ] 

ASF GitHub Bot commented on AIRFLOW-2548:
-----------------------------------------

Fokko commented on pull request #3930: [AIRFLOW-2548] Output plugin import errors to web UI
URL: https://github.com/apache/incubator-airflow/pull/3930
 
 
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Output Plugin Import Errors to WebUI
> ------------------------------------
>
>                 Key: AIRFLOW-2548
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-2548
>             Project: Apache Airflow
>          Issue Type: Bug
>            Reporter: Andy Cooper
>            Priority: Major
>             Fix For: 2.0.0
>
>
> All,
>  
> We currently output all DAG import errors to the webUI. I propose we do the same with plugin errors as well. This will provide a better user experience by bubbling up all errors to the webUI instead of hiding them in stdOut.
>  
> Proposal...
>  * Extend models.ImportError to have a "type" field to distinguish from error types.
>  * Prevent class SchedulerJob methods from clearing out and pulling from models.ImportError if type = 'plugin'
>  * Create new ImportError records in plugins_manager.py for each plugin that fails to import
>  * Prompt user in views.py with plugin ImportErrors - specifying that they need to fix and restart webserver to resolve.
>  
> Does this seem reasonable to everyone? I'd be interested in taking on this work if needed



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)