Jorricks opened a new pull request #17179:
URL: https://github.com/apache/airflow/pull/17179


   **Description of the problem:**
   Currently there is a BUG inside the Task Instance details (/task) view.
   It loads the TaskInstance by calling `TI(task, execution_date)` and then 
uses `refresh_from_db()` to refresh many fields that are no filled in yet.
   However, the assumption is made in that case that it refreshes all values, 
which it does not.
   `external_executor_id` and `queued_by_job_id` are not updated at all and 
`executor_config` is only instantiated by the original `TI(task, 
execution_date)` call but also not updated in `refresh_from_db()`.
   This also shows in the UI where these values are always showing None, while 
the TaskInstance view shows you these values are not None.
   
   **The changes in the PR:**
   1. Changes to the `update_from_db()` method to include the missing three 
values.
   2. A new test that checks we are really updating ALL values in 
`update_from_db()`
   3. Removal of an incorrect comment as we do need the `execution_date` for 
that view.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to