541732 - OS version set in QAC not recognized as "legal."

Status: VERIFIED FIXED

(Other Applications Graveyard :: QA Companion, defect)

Description

[aslam karachiwala]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.1pre) Gecko/20100123 Ubuntu/9.10 (karmic) Namoroka/3.6.1pre
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.1pre) Gecko/20100123 Ubuntu/9.10 (karmic) Namoroka/3.6.1pre

The Platform, OS, etc. settings chosen in Preferences/QA Companion are not seen as "legal." This prevents me from submitting a bug.

Reproducible: Always

Steps to Reproduce:
1. Set the QAC prefs for Linux & Firefox 3.6
2. Launch QAC
3. Submit new bug
Actual Results:  
Error: "A legal OS/Version was not set."

Bug is not submitted.

Expected Results:  
Bug is submitted.

Comment 1

[aslam karachiwala]

Attachment: Screenshot showing the QAC settings & error.

Comment 2

[aslam karachiwala]

QAC v. 1.2.1

Comment 3

[Aaron Train [:aaronmt]]

Strange, this is a caught error on Bugzilla's side, i.e., see (http://mxr.mozilla.org/webtools/source/bugzilla/contrib/bugzilla-submit/bugzilla-submit#238). 

Apparently it's not letting you file a bug - this is bad. Aslam, can you check this preference (qa.extension.config.os)  in Firefox and report what it is currently set at? 

I dont suspect there should be any issues with you running x86_64, but we wont rule it out.

Comment 4

[Aaron Train [:aaronmt]]

I just reproduced with 1.2.1 on Linux Ubuntu 9.04 i686 and Windows XP on the 3.6 branch. I have a suspicion that some backend code changed with Bugzilla 3.44 recently.

We create bug objects with the following specifiers and post them through Bugzilla RPC: summary, description, product, component, version, status, op_sys, platform

Dave: Did the recent upgrade change anything RPC related, in particular perhaps op_sys?
Heather: Any ideas?

Comment 5

[aslam karachiwala]

(In reply to comment #3)
> Aslam, can you check
> this preference (qa.extension.config.os)  in Firefox and report what it is
> currently set at? 
> 

qa.extension.config.os = Linux

I'll attach a screenshot showing all qa.extension settings.

Comment 6

[aslam karachiwala]

Attachment: Screenshot showing all qa.extension.* settings

Comment 7

[Dave Miller [:justdave]]

Where do you get your list of op_sys values from?  They can be retrieved from Bugzilla, and bmo's list of legal values does not match the upstream list, so you shouldn't be hard-coding them.

And yes, the 3.4 upgrade had a major overhaul of the XMLRPC API, but that was back in November.

Comment 8

[Dave Miller [:justdave]]

Looks at the screenshot, I think it's your platform value that's actually in error.  There is no "Linux" value in the Platform field on bmo.

Comment 9

[Heather Arthur [:harth]]

Attachment: fix for reporting opsys to Bugzilla

Thanks for reporting aslam! The preferences coorespond to the Litmus values, it was submitting the platform correctly but not submitting anything for the operating system. I was afraid to test it out on the real Bugzilla, but I guess it might have been a good idea )-: This patch will fix this issue.

Comment 10

[Reed Loden [:reed]]

Comment on attachment 423216 [details] [diff] [review]
fix for reporting opsys to Bugzilla

>@@ -261,6 +261,9 @@
>     else if(/Linux-maemo/.exec(platformPref)) {
>       opsys = 'Linux(embedded)';
>     }

From where are you getting "Linux(embedded)"? It's actually "Linux (embedded)" (with the space).

Comment 11

[Aaron Train [:aaronmt]]

Comment on attachment 423216 [details] [diff] [review]
fix for reporting opsys to Bugzilla

Tested this out, and I submitted a bug report successfully. Thanks Harth!

Comment 12

[Heather Arthur [:harth]]

Sending        chrome/content/tabs/bugzilla.js
Transmitting file data .
Committed revision 61095.

Fixed Linux embedding space as well. When we start using the REST API we can test against the staging that has the same platforms/products as real Bugzilla, will be sweet.

Comment 13

[Heather Arthur [:harth]]

uploaded new version with fix to AMO

Comment 14

[Aaron Train [:aaronmt]]

verified fixed with 1.2.2