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

GLib-GObject-CRITICAL in Firefox Travis logs makes them very verbose and harder to understand #9657

Closed
foolip opened this issue Feb 25, 2018 · 5 comments

Comments

@foolip
Copy link
Member

foolip commented Feb 25, 2018

https://api.travis-ci.org/v3/job/345549233/log.txt is a raw log (from #9641) and it has lots of this stuff:

PID 6210 | (firefox:6210): GLib-GObject-CRITICAL **: g_object_ref: assertion 'object->ref_count > 0' failed
PID 6210 | 
PID 6210 | (firefox:6210): GLib-GObject-CRITICAL **: g_object_unref: assertion 'object->ref_count > 0' failed

It drowns out the rest of the content.

@jgraham, do you know where this is coming from? If it can't be fixed in Firefox, do you think we can just filter them out in the Travis runs to make these logs smaller? One has to download the raw logs normally for Firefox because they're too big to get the nice UI.

@foolip
Copy link
Member Author

foolip commented Mar 1, 2018

I think the Firefox failure in #9680 was probably caused by this:

Log reached capacity (3145728 bytes); output disabled.


No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#Build-times-out-because-no-output-was-received

The build has been terminated

Hypothesis being that the output is disabled at layer above the "output has been received" logic, so that producing too large a log will lead to this result.

@jgraham, any ideas?

@jgraham
Copy link
Contributor

jgraham commented Mar 1, 2018

I don't know how to fix it in Firefox, certainly.

We could add some filtering in the logger to remove these lines from the formatted output. The hacky way to do this would be in the on_output method in tools/wptrunner/wptruner/browsers/firefox.py

@jgraham
Copy link
Contributor

jgraham commented Mar 7, 2018

So let's try the simplest possible solution here. I think a regexp in tools/wptrunner/wptruner/browsers/firefox.py::on_output discarding input that matches the GLib-GObject-CRITICAL lines above would be sufficient. Ideally we would have some infrastructure to allow multiple regex to run, since I have a few other use cases that might need this in the future.

@kritisingh1
Copy link
Member

kritisingh1 commented Mar 8, 2018

Hi. Can I take this up? Thank you!

@foolip
Copy link
Member Author

foolip commented Mar 8, 2018

@kritisingh1, awesome, please let us know if you run into any trouble!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants