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

Use the stopsign as icon for timeouts and exceeded log-limits #724

Merged

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented Mar 21, 2020

When I browse failed builds in a jobset-eval on Hydra, I regularly
mistake actual build-failures with temporary issues like timeouts (that
probably disappear at the next eval).

To prevent this kind of issue, I figured that using the stopsign-svg for
builds with timeouts or exceeded log-limits is a reasonable choice for
the following reasons:

  • A user can now distinguish between actual build-errors (like
    compilation-failures or oversized outputs) and (usually) temporary issues
    (like a bloated log or a timeout).

  • The stopsign is also used for aborted jobs that are shown in a
    different tab and can't be confused with timeouts for that reason.

When I browse failed builds in a jobset-eval on Hydra, I regularly
mistake actual build-failures with temporary issues like timeouts (that
probably disappear at the next eval).

To prevent this kind of issue, I figured that using the stopsign-svg for
builds with timeouts or exceeded log-limits is a reasonable choice for
the following reasons:

* A user can now distinguish between actual build-errors (like
  compilation-failures or oversized outputs) and (usually) temporary issues
  (like a bloated log or a timeout).

* The stopsign is also used for aborted jobs that are shown in a
  different tab and can't be confused with timeouts for that reason.
@edolstra edolstra merged commit c85ee73 into NixOS:master Mar 23, 2020
@Ma27 Ma27 deleted the different-icons-for-timeout-and-loglimit branch March 23, 2020 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants