airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Blake Livingston (JIRA)" <>
Subject [jira] [Commented] (AIRFLOW-2246) Add error messaging to CLI variable import
Date Fri, 23 Mar 2018 14:01:00 GMT


Blake Livingston commented on AIRFLOW-2246:

I will see if I can put together a PR for this within the week.


> Add error messaging to CLI variable import
> ------------------------------------------
>                 Key: AIRFLOW-2246
>                 URL:
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: cli
>    Affects Versions: Airflow 1.8
>            Reporter: Blake Livingston
>            Priority: Minor
> When importing variables from the CLI, the load will be incomplete if any of the variable
values fail to parse. In the case I noticed, I had used a Number rather than a string.
> E.g.
> { "var1": "OK",
>   "var2": 4,
>   "var3", "skipped"}
> airflow variables -i vars.json
> will unhelpfully return:
> "1 of 3 variables successfully updated."
> I believe this is due to passing on Exception here:
> []
> Would be cool to just write an error with the variable, value, and exception message.
> next level of safety+usability would be to have it validate all variables before applying
them. An atomic failure is safer and preferable to a partial load for most users, I would

This message was sent by Atlassian JIRA

View raw message