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

Grid JavaScript error on new task added #24162

Closed
1 of 2 tasks
tothandor opened this issue Jun 3, 2022 · 2 comments
Closed
1 of 2 tasks

Grid JavaScript error on new task added #24162

tothandor opened this issue Jun 3, 2022 · 2 comments
Labels
area:UI Related to UI/UX. For Frontend Developers. duplicate Issue that is duplicated kind:bug This is a clearly a bug

Comments

@tothandor
Copy link

tothandor commented Jun 3, 2022

Apache Airflow version

2.3.1 (latest released)

What happened

I have added a new task to an existsting workflow (DAG), where the other tasks already had previous runs.
When I added the new task, the grid became inaccessible, because it wanted to get runIds for the new task, which did not exist.
An error occured in the grid..js, but since the file is minified/obfuscated, it's unclear for me where exactly.

What you think should happen instead

The grid should handle missing runId without causing an exception.

How to reproduce

  1. Create a simple dag, with a few dummy tasks, and let it run a few times. Check out the grid (it should be okay).
  2. After that, add a new dummy task, and check out the grid again. The grid's area will be empty, and the JS console will have the following error: TypeError: Cannot read properties of null (reading 'runId')

Probably any other scenarios where the tasks run history is uneven, could also cause this error.

Operating System

Linux

Versions of Apache Airflow Providers

No response

Deployment

Virtualenv installation

Deployment details

No response

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@tothandor tothandor added area:core kind:bug This is a clearly a bug labels Jun 3, 2022
@tirkarthi
Copy link
Contributor

Seems to be same as

#23908
#23933

cc @bbovenzi

@bbovenzi bbovenzi added area:UI Related to UI/UX. For Frontend Developers. duplicate Issue that is duplicated and removed area:core labels Jun 3, 2022
@bbovenzi
Copy link
Contributor

bbovenzi commented Jun 3, 2022

Yes, this seems to be a duplicate. I just tried the exact steps in main and it is working as expected.

Fixed in #23947 for 2.3.2

@bbovenzi bbovenzi closed this as completed Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. duplicate Issue that is duplicated kind:bug This is a clearly a bug
Projects
None yet
Development

No branches or pull requests

3 participants