[jira] [Created] (AIRFLOW-4524) "Ignore All Deps" and "Ignore Task Deps" doesn't ignore task dependencies

["James Meickle (JIRA)" <ji...@apache.org>]

James Meickle created AIRFLOW-4524:
--------------------------------------

             Summary: "Ignore All Deps" and "Ignore Task Deps" doesn't ignore task dependencies
                 Key: AIRFLOW-4524
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-4524
             Project: Apache Airflow
          Issue Type: Bug
          Components: ui
    Affects Versions: 1.10.3
            Reporter: James Meickle


After an upgrade from Airflow 1.10.1->1.10.3, we're seeing this behavior when trying to "Run" a task in the UI with "Ignore All Deps" and "Ignore Task Deps":
{code:java}
"Could not queue task instance for execution, dependencies not met: Trigger Rule: Task's trigger rule 'all_success' requires all upstream tasks to have succeeded, but found 1 non-success(es). upstream_tasks_state={'successes': 1, 'skipped': 0, 'upstream_failed': 0, 'failed': 0, 'done': 1, 'total': 2}, upstream_task_ids={'p72adhoc_restricted_list', 'restricted_list'}"
{code}
I suspected this was due to the checkbox not getting applied so I investigated this in Chrome. Here's a 1.10.3 POST:
{code:java}
dag_id: MY_DAG_ID
task_id: MY_TASK_ID
execution_date: 2019-05-15T00:00:00+00:00
origin: MY_AIRFLOW_SERVER
ignore_all_deps: 
ignore_ti_state: 
ignore_task_deps: 
ignore_all_deps: true
ignore_task_deps: true
downstream: true
recursive: true
{code}
And here's a 1.10.1 POST:
{code:java}
task_id: MY_TASK_ID
dag_id: MY_DAG_ID
ignore_all_deps: true
ignore_task_deps: true
ignore_ti_state: false
execution_date: 2019-05-16T11:00:00+00:00
origin: MY_AIRFLOW_SERVER
{code}
It looks like the JS here is doing something wonky and sending blank values for unchecked boxes, as well as checkboxes that don't apply to the current action, which is presumably breaking parsing when the server receives a json object with multiple conflicting keys.



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