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

Fix Try Selector in Mapped Tasks also on Index 0 #43590

Conversation

jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Nov 1, 2024

While doing a regression on release tests on 2.10.3rc2 I noticed that the fixed logic still has a problem on mapped in dex 0 - seem I did not click on the first mapped element when testing :-(

This PR fixes the logic such that the try history is correctly fetched also for index 0 if dynamic mapped tasks.

Hope we get rid of this code in Airflow 3, this manual client function is really error prone.

related: #43565 #42357

@jscheffl jscheffl added this to the Airlfow 2.10.4 milestone Nov 1, 2024
@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Nov 1, 2024
@jscheffl jscheffl added type:bug-fix Changelog: Bug Fixes legacy ui Whether legacy UI change should be allowed in PR labels Nov 1, 2024
@jscheffl jscheffl closed this Nov 1, 2024
@jscheffl jscheffl reopened this Nov 1, 2024
Co-authored-by: Brent Bovenzi <[email protected]>
@jscheffl jscheffl merged commit ccbf1da into apache:main Nov 1, 2024
51 checks passed
jscheffl added a commit that referenced this pull request Nov 1, 2024
…43591)

* Fix Try Selector in Mapped Tasks also on Index 0

(cherry picked from commit 1c3d555)

* Review Feedback, direct commit

Co-authored-by: Brent Bovenzi <[email protected]>

---------

Co-authored-by: Brent Bovenzi <[email protected]>
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* Fix Try Selector in Mapped Tasks also on Index 0

* Review Feedback, direct commit

Co-authored-by: Brent Bovenzi <[email protected]>

---------

Co-authored-by: Brent Bovenzi <[email protected]>
utkarsharma2 pushed a commit that referenced this pull request Dec 4, 2024
…43591)

* Fix Try Selector in Mapped Tasks also on Index 0

(cherry picked from commit 1c3d555)

* Review Feedback, direct commit

Co-authored-by: Brent Bovenzi <[email protected]>

---------

Co-authored-by: Brent Bovenzi <[email protected]>
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. area:webserver Webserver related Issues legacy ui Whether legacy UI change should be allowed in PR type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants