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

build logs: faster navigation to errors #2415

Open
TomasTomecek opened this issue Dec 1, 2022 · 3 comments
Open

build logs: faster navigation to errors #2415

TomasTomecek opened this issue Dec 1, 2022 · 3 comments
Labels
packit RFE Enhancement, feature requests

Comments

@TomasTomecek
Copy link
Contributor

We have recently got an RFE in #packit for RPM builds in Copr. The suggestion was to highlight errors so the user can easily spot what went wrong and correct it. This would greatly improve user experience, especially for folks who don't have a ton of experience with RPM builds.

This could also be a great opportunity to even apply some of the machine learning here (as @xsuchy hinted a few weeks ago).

Related: https://pagure.io/copr/copr/issue/401

Originally reported to us by @ssbarnea

@ssbarnea
Copy link

ssbarnea commented Dec 1, 2022

IMHO, we should not need to go to machine learning, plain simple highlighting of the error lines would make a huge improvement. This can be achieve with at least one of the next two steps:

  • ANSI enabled logs. Rendering can even happen using the browser as there are React components that can display ANSI in browser.
  • Using regexes to highlight the lines with errors, in the end with less than 10 regexes we could probably highlight he important lines from >99% of the logs. I even have a good set of already made ones that we used with tripleo openstack.

PS. Try to not over-engineer the coloring, find the easiest way to enable it.

Related: #1183

@praiskup
Copy link
Member

praiskup commented Dec 1, 2022

Using regexes to highlight the lines with errors, in the end with less than 10 regexes we could probably highlight he important lines from >99% of the logs.

This is an interesting idea. There's a similar Mock approach rpm-software-management/mock#279

If we tweaked the tool so it was able to extract the "likely" build failure reason, we could even present it in the web UI.

@praiskup praiskup added RFE Enhancement, feature requests packit labels Dec 3, 2022
@praiskup
Copy link
Member

praiskup commented Feb 9, 2023

See also: #835

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packit RFE Enhancement, feature requests
Projects
Status: Someday in future
Development

No branches or pull requests

3 participants