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

Add support for parsing standard Xorg.log output #46

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

imirkin
Copy link

@imirkin imirkin commented Nov 23, 2019

There's a wealth of xorg.log files posted for various debug purposes.
Make it easy to import data from these.

This output is produced by hw/xfree86/ddc/print_edid.c so all DDX's
should produce the same output.

@imirkin
Copy link
Author

imirkin commented Nov 23, 2019

FWIW the failing test is "ERROR: test_valid (frontend.selenium_tests.test_upload.UploadSeleniumTestCase)" which fails with a timeout. I don't think that's a result of anything done in this commit.

@@ -52,6 +52,11 @@ <h4>Thanks!</h4>
<input type="radio" id="id_text_type_xrandr" name="text_type" value="xrandr"{% ifequal form.text_type.value 'xrandr' %} checked{% endifequal %}> XRandR
</label>
<p>Linux users, run <code>xrandr --props</code> and paste its output.</p>
<label class="radio">
<input type="radio" id="id_text_type_xoglog" name="text_type" value="xorglog"{% ifequal form.text_type.value 'xorglog' %} checked{% endifequal %}> Xorg.log
Copy link

Choose a reason for hiding this comment

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

"id_text_type_xoglog" maybe meant ...xorglog?

Copy link
Author

Choose a reason for hiding this comment

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

Maybe :) Fixed, but FWIW I don't think this has any practical implications on implementation.

@imirkin
Copy link
Author

imirkin commented Nov 24, 2019

Hm, actually I can reproduce that selenium failure locally. I'll figure out what's going on. The binary upload gets a 500 error.

@imirkin
Copy link
Author

imirkin commented Nov 24, 2019

Ugh, the error is caused by frontend/views.py:123 doing edid_object.save() which results in a 'database table is locked: frontend_edid" error. Clearly not caused by my change (reproduced same thing on master), so must be a change in django or sqlite driver.

Looks like something in selenium has changed, but this code was dubious
to begin with. upload-id-upload is not a form but rather a (submit)
button. I suspect that the "submit" action was causing a click *and* a
submit at the same time, so two requests were getting dispatched,
causing a deadlock in sqlite.

The fix would either be to submit the form, or simply click on the
button. The latter is chosen to avoid any unnecessary changes.
There's a wealth of xorg.log files posted for various debug purposes.
Make it easy to import data from these.

This output is produced by hw/xfree86/ddc/print_edid.c so all DDX's
should produce the same output.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 99.217% when pulling fd9e77a on imirkin:master into 0675d67 on timvideos:master.

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.

3 participants