Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BugFix: Fix taskInstance API call fails if a task is removed from running DAG #14381

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

ephraimbuddy
Copy link
Contributor

Closes: #14331


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Feb 23, 2021
@github-actions
Copy link

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Feb 23, 2021
@ashb ashb changed the title BugFix: Fix taskInstance call fails if a task is removed from running DAG BugFix: Fix taskInstance API call fails if a task is removed from running DAG Feb 23, 2021
@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@ephraimbuddy ephraimbuddy force-pushed the add-removed-state-for-tasks branch 3 times, most recently from c19af47 to 3858761 Compare February 26, 2021 06:59
@ephraimbuddy
Copy link
Contributor Author

@ashb MySQL 8 job is failing and I can't find the reason

@ashb
Copy link
Member

ashb commented Feb 26, 2021

Yeah, not sure what happened there. Likely a timeout or some other glitch.

@ephraimbuddy ephraimbuddy force-pushed the add-removed-state-for-tasks branch 3 times, most recently from 22ba9b9 to 60e79ec Compare February 28, 2021 22:27
@ephraimbuddy
Copy link
Contributor Author

Yeah, not sure what happened there. Likely a timeout or some other glitch.

Have pushed severally and MySQL 8 still failing. I can't pin what it is. 🙂

@kaxil kaxil added this to the Airflow 2.0.2 milestone Mar 8, 2021
@kaxil kaxil merged commit 7418679 into apache:master Mar 8, 2021
@kaxil kaxil deleted the add-removed-state-for-tasks branch March 8, 2021 21:25
ashb pushed a commit that referenced this pull request Mar 19, 2021
kaxil pushed a commit to astronomer/airflow that referenced this pull request Apr 1, 2021
…ning DAG (apache#14381)

Closes: apache#14331

(cherry picked from commit 7418679)
(cherry picked from commit 0cb2a96)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API full tests needed We need to run full set of tests for this PR to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Airflow stable API taskInstance call fails if a task is removed from running DAG
3 participants