Closed
Bug 757090
Opened 12 years ago
Closed 12 years ago
Output platform and application details to console
Categories
(Mozilla QA Graveyard :: Mozmill Automation, defect)
Mozilla QA Graveyard
Mozmill Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: davehunt, Assigned: davehunt)
Details
Attachments
(1 file, 1 obsolete file)
2.23 KB,
patch
|
whimboo
:
review+
|
Details | Diff | Splinter Review |
In order to report the platform details (name, version, CPU architecture) in the Mozmill CI build notification emails, we need to be able to regex it from the console log. Something like: Platform name: Windows NT Platform version: 6.1.7601 CPU Architecture: x86_64
Assignee | ||
Comment 1•12 years ago
|
||
This patch outputs for example: *** Platform: Mac OS X 10.7.4 64bit
Assignee | ||
Updated•12 years ago
|
Attachment #625728 -
Flags: review? → review?(hskupin)
Comment 2•12 years ago
|
||
Comment on attachment 625728 [details] [diff] [review] Add platform details to console. v1.0 >+ # Print platform details >+ print '*** Platform: %s %s %sbit' % ( >+ str(mozinfo.os).capitalize(), >+ mozinfo.version, >+ mozinfo.bits) I would move this to run_tests so we could also include the version of Firefox like print '*** Start: Testing %app% %app_version% on %os% %os_version% %bits%.' Otherwise it looks fine.
Attachment #625728 -
Flags: review?(hskupin) → feedback+
Assignee | ||
Comment 3•12 years ago
|
||
If we have this in run tests then it will appear multiple times for testruns. When extracting via regular expression we would then get every match. As discussed, I will leave the platform details where they are, and add Application details to the console for each binary under test.
Assignee | ||
Comment 4•12 years ago
|
||
Can you let me know if this is what you had in mind?
Attachment #625728 -
Attachment is obsolete: true
Attachment #637479 -
Flags: review?(hskupin)
Comment 5•12 years ago
|
||
Comment on attachment 637479 [details] [diff] [review] Add platform and application details to console. v1.1 Yes, that looks great. Can't await to see Mozmill CI parsing it!
Attachment #637479 -
Flags: review?(hskupin) → review+
Updated•12 years ago
|
Summary: Output platform details to console → Output platform and application details to console
Comment 6•12 years ago
|
||
Pushed: http://hg.mozilla.org/qa/mozmill-automation/rev/4d69652a6bfe
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•