put xorglog.py into Precise

Bug #1035499 reported by Jeff Lane 
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
xdiagnose
Fix Released
Undecided
Unassigned
xdiagnose (Ubuntu)
Fix Released
Undecided
Unassigned
Precise
Won't Fix
High
Bryce Harrington

Bug Description

[Impact]
Xorg.0.log has a ton of useful information for QA purposes, yet the file is not well structured for programmatic examination, as is necessary when running automated QA tests. During the raring cycle a number of efforts seek to establish automated testing of X.org, so having this code more widely available will help facilitate the scripts.

[Test Case]
$ python
>>> from xdiagnose.xorglog import XorgLog
>>> xorglog = XorgLog('/var/log/Xorg.0.log')
>>> print xorglog.video_driver
RADEON
>>> print xorglog.xserver_version
2:1.11.4-0ubuntu10.6
>>> print xorglog.errors
['Failed to load module "fglrx" (module does not exist, 0)', 'Failed to load module "fglrx" (module does not exist, 0)']

Expected Results: As above
Actual Results: The import will fail with "module not found"

[Regression Potential]
None, this only adds code: A python module and corresponding unit test (and the test's support library). No other code is altered or affected by these additions.

[Original Report]
I've been working on a script for Hardware Certification and OEM QA usage (and hopefully more) that reports on some graphics driver info.

For now, it reports the running graphics driver and version, and does a check for hybrid graphics and lists DDX drivers listed in Xorg.0.log.

For the moment, I have had to copy the XorgLog class from xorglog.py into my script directly because our code needs to be supported on Precise, but xorglog.py only exists currently in Trunk.

So I'm asking for an SRU to get xorglog.py into the xdiagnose for Precise to support this testing, and potentially future tests that could rely on code in xdiagnose for the Hardware Certification and OEM QA Teams.

Getting this into Precise will help a LOT as I'll be able to remove my hacked copy from my test scripts and use a simple Import. This will make my code infinitely more maintainable and I'll not have to maintain a copy of the XorgLog class that is out of sync with the copy that Bryce is developing/maintaining. This will also allow me to write one script that works on everything from Precise onward.

Precise is requested because these tests will be part of the Cert suites as well as OEM QA test suites, and most of that work will be based on Precise for some time to come.

Bryce Harrington (bryce)
Changed in xdiagnose (Ubuntu Precise):
status: New → In Progress
importance: Undecided → High
Changed in xdiagnose (Ubuntu):
status: New → Fix Released
Changed in xdiagnose:
status: New → Fix Released
Changed in xdiagnose (Ubuntu Precise):
assignee: nobody → Bryce Harrington (bryce)
Bryce Harrington (bryce)
description: updated
Bryce Harrington (bryce)
description: updated
Revision history for this message
Steve Langasek (vorlon) wrote :

Looking at this, I don't think it makes sense for this to be in an SRU. The justification seems to be that it will make things more convenient for testing with tools that are not part of Ubuntu. Nothing in the updated xdiagnose package makes use of this new python module. So this isn't fixing a user-affecting bug at all, and I don't believe this should be SRUed as an update that will be applied to all users' systems.

Changed in xdiagnose (Ubuntu Precise):
status: In Progress → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.