If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Output platform and application details to console

RESOLVED FIXED

Status

Mozilla QA Graveyard
Mozmill Automation
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: davehunt, Assigned: davehunt)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
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

5 years ago
Created attachment 625728 [details] [diff] [review]
Add platform details to console. v1.0

This patch outputs for example:

*** Platform: Mac OS X 10.7.4 64bit
Assignee: nobody → dave.hunt
Status: NEW → ASSIGNED
Attachment #625728 - Flags: review?
(Assignee)

Updated

5 years ago
Attachment #625728 - Flags: review? → review?(hskupin)
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

5 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

5 years ago
Created attachment 637479 [details] [diff] [review]
Add platform and application details to console. v1.1

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 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+
Summary: Output platform details to console → Output platform and application details to console
Pushed:
http://hg.mozilla.org/qa/mozmill-automation/rev/4d69652a6bfe
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.