Make extra link work in UI #25500
Make extra link work in UI #25500
Conversation
uranusjr
requested review from
XD-DENG,
ashb,
bbovenzi,
kaxil and
ryanahamilton
as code owners
boring-cyborg
bot
added
area:core-operators
area:providers
area:UI
airflow/models/abstractoperator.py
Outdated
There was a problem hiding this comment.
Unrelated, but Mypy is complaining. context here is implemented as a MutableMapping (although we supply a type stub to say it’s not to prevent users from getting a wrong idea), so this is fine.
uranusjr
force-pushed
the
extra-links-js-req
branch
3 times, most recently
from
bbf31ef to
72d55bd
Compare
This has two parts: First, the existing extra links mechanism relies on the operator being unmapped, so we do that for most operators to keep working. Second, the UI needs to be slightly updated to ask for those links at the right time -- namely, it must only make requests against unmapped tasks, not the mapped ones.
uranusjr
force-pushed
the
extra-links-js-req
branch
from
72d55bd to
4a6632e
Compare
Hmmm. Ideally It would be nice if each individual mapped task could have it's own mapped links too. For example if your mapped task submits jobs to EMR/Databricks etc then being able to have mapped links to the remote logs/job is desirable. |
Yes, this is what this PR intends to do. What we’re currently doing is completely the other way around: we display the links against the entire mapped task (and that doesn’t work well for most things), but not for individual (mapped) task instances. This PR removes the former, and adds the links for the latter. |
5 participants
Close #25360. Two parts in this: