You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "bbovenzi (via GitHub)" <gi...@apache.org> on 2024/12/04 21:31:12 UTC

[I] AIP-38 | Add Task Instance Logs [airflow]

bbovenzi opened a new issue, #44663:
URL: https://github.com/apache/airflow/issues/44663

   (no comment)


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "set92 (via GitHub)" <gi...@apache.org>.

set92 commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2558097249

   @bbovenzi Great, thanks! Although I feel the other option would have been better in functionality. Although more complex in terms of UI, and therefore harder to maintain.
   
   In case it was not clear what I meant, I did a quick draw. We would have the dropdown with all the task tries. But at a side, also the latest tries (they can be the last 3, 5 or customizable by config up to a number), which will allow to quickly select them, without having to open the dropdown.
   <img width="464" alt="image" src="https://github.com/user-attachments/assets/3e852ed0-9c28-4a6b-8077-1eb40d50d71e" />
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.

bbovenzi closed issue #44663: AIP-38 | Add Task Instance Logs
URL: https://github.com/apache/airflow/issues/44663


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "tirkarthi (via GitHub)" <gi...@apache.org>.

tirkarthi commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2519455632

   It will be good to have a full screen view of the logs at least without the filters like graph modal which was one of the requested features as legacy logs UI was relatively full screen.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.

bbovenzi commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2575920445

   I think we can close and make other issues for colors, groups, and filters


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.

pierrejeambrun commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2572916879

   I believe we are only missing the colors on this one before closing ?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.

bbovenzi commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2520512667

   Perhaps an expand icon button which will open the same logs but in a full page modal?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "set92 (via GitHub)" <gi...@apache.org>.

set92 commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2555982264

I was searching for the exact Issue, but I couldn't find it. I was testing today AF 2.10.4, and I found that when you have more than 10 fails in a task, the buttons are replaced by a dropdown list?

If you get to that number of tries is most probably because you are debugging, so you don't want more steps to go to your last log error. So, from this point of view I see 2 improvements
- Make the order in reverse. Probably will be weird, but if you are with an error, you want to select the last error log as soon as possible, and when you have to select the dropdown list, and then go to select the last element is not very nice or comfortable. Other option would be to put a limit or make the list smaller to only shows 5 elements, and have a scrollbar.
- The other option would be to have both, the dropdown, and the last or the last 3 log error. Which will appear on the right side of the dropdown. This way it is easier to see error logs, that's most people are interested in. But, if you want to check previous ones, you still have the dropdown.

--
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] AIP-38 | Add Task Instance Logs [airflow]

Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.

bbovenzi commented on issue #44663:
URL: https://github.com/apache/airflow/issues/44663#issuecomment-2557233787

   @set92 good call. Updated for 3.0: https://github.com/apache/airflow/pull/45117


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org