Skip to content

Conversation

@dheerajturaga
Copy link
Member

Update the default behavior for task log source display from shown
to hidden, providing a cleaner initial view of logs. Users can still
toggle source visibility using the settings menu

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label Nov 27, 2025
Copy link
Member

@guan404ming guan404ming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for the change, but I'm not sure whether we should have this change since task log source seems like kind of important info. Would like to listen to another opinions!

@dheerajturaga
Copy link
Member Author

dheerajturaga commented Dec 2, 2025

Looks good for the change, but I'm not sure whether we should have this change since task log source seems like kind of important info. Would like to listen to another opinions!

The source line creates a lot of noise in the logs. Changing the default hides the noise unless someone wants to explicitly see the that information. This behavior is aligned with Airflow 2.x and I think having the source always on was unintentional in Airflow 3

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@pierrejeambrun pierrejeambrun added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Dec 2, 2025
@pierrejeambrun pierrejeambrun added this to the Airflow 3.1.4 milestone Dec 2, 2025
@pierrejeambrun
Copy link
Member

pierrejeambrun commented Dec 2, 2025

Looks good for the change, but I'm not sure whether we should have this change since task log source seems like kind of important info. Would like to listen to another opinions!

I think it's fine. Indeed I find it harder to read from the start when source is there by default. People can activate it if needed. Also people used to that in the first place have the setting persisted in the localstorage, so this will not change anything. (unless they clear their cache).

We can try like this and see what people think. (easy to revert otherwise)

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu
@dheerajturaga dheerajturaga force-pushed the nit/log-default-hidesource branch from 4c55692 to 59d9040 Compare December 4, 2025 11:11
@pierrejeambrun pierrejeambrun merged commit d024a46 into apache:main Dec 4, 2025
61 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 4, 2025
* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

pierrejeambrun pushed a commit that referenced this pull request Dec 4, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
ephraimbuddy pushed a commit that referenced this pull request Dec 8, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 8, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 8, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 8, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 8, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 9, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 11, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Dec 12, 2025
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
ephraimbuddy pushed a commit that referenced this pull request Jan 6, 2026
…) (#59045)

* Change task log source display to hidden by default

  Update the default behavior for task log source display from shown
  to hidden, providing a cleaner initial view of logs. Users can still
  toggle source visibility using the settings menu

* Fix tests
(cherry picked from commit d024a46)

Co-authored-by: Dheeraj Turaga <dheerajturaga@gmail.com>
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. backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants