Closed
Bug 1007663
Opened 11 years ago
Closed 11 years ago
Avoid showing empty experiments list
Categories
(Firefox Health Report Graveyard :: Client: Desktop, defect)
Firefox Health Report Graveyard
Client: Desktop
Tracking
(firefox30 unaffected, firefox31 fixed, firefox32 verified)
VERIFIED
FIXED
Firefox 32
Tracking | Status | |
---|---|---|
firefox30 | --- | unaffected |
firefox31 | --- | fixed |
firefox32 | --- | verified |
People
(Reporter: gfritzsche, Assigned: gfritzsche)
References
Details
(Whiteboard: p=5 s=it-32c-31a-30b.2 [qa!])
Attachments
(1 file)
4.39 KB,
patch
|
Unfocused
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
* have only one active experiment and no previous experiment
* go to the about:addons, experiments category
* click remove on the experiment
Result: empty addon list until reloading or switching categories
Expected: experiment shown as inactive or pending uninstall
Bug 992258, comment 5 pointed out that uninstall for addons is only triggered after e.g. leaving the category (until bug 612168 is fixed).
However, the experiment addon already get's disabled and we end up not showing it as it's not in the previous experiment list until it is actually uninstalled.
We can probably fix this by doing what we dropped earlier - marking PreviousExperimentAddon with a property previousExperiment instead of filtering by the addons disabled properties.
Assignee | ||
Updated•11 years ago
|
Summary: Avoid showing empty → Avoid showing empty experiments list
Assignee | ||
Updated•11 years ago
|
Flags: firefox-backlog+
Whiteboard: p=5
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → georg.fritzsche
Updated•11 years ago
|
Status: NEW → ASSIGNED
Whiteboard: p=5 → p=5 s=it-32c-31a-30b.2 [qa?]
Updated•11 years ago
|
Whiteboard: p=5 s=it-32c-31a-30b.2 [qa?] → p=5 s=it-32c-31a-30b.2 [qa+]
Assignee | ||
Comment 1•11 years ago
|
||
Disregard the background in comment 0, i think we should just show the undo element etc. here (and leave the rest for bug 612168).
https://tbpl.mozilla.org/?tree=Try&rev=12c1e6dde555
Attachment #8422590 -
Flags: review?(bmcbride)
Comment 2•11 years ago
|
||
Comment on attachment 8422590 [details] [diff] [review]
Show undo element after manual addon remove
Review of attachment 8422590 [details] [diff] [review]:
-----------------------------------------------------------------
Agreed.
Attachment #8422590 -
Flags: review?(bmcbride) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 32
Updated•11 years ago
|
QA Contact: andrei.vaida
Comment 5•11 years ago
|
||
Georg, I was unable to verify the fix for this issue because I couldn't actually install an experiment.
Here are the steps that I followed in my numerous attempts on Nightly 32 2014-05-16, using two different machines:
1. Created a clean profile and added the pref "experiments.force-sample-value" as string, with the value "0.25".
2. Toggled the state of the pref "experiments.enabled", then closed the browser.
3. From the profile's folder, I deleted "experiments.json" file then launched back the browser.
4. Toggled again the state of "experiments.enabled" pref and closed the browser.
5. Launched back the browser, but the pref "experiments.activeExperiment" was still "false".
5.1. A new pref was also added eventually, "app.update.lastUpdateTime.experiments-update-timer" with the value "0", after repeating the above steps several times using the same profile.
I might be missing something here, since with the exact same steps I was able to install an experiment using Nightly 32 from 2014-05-07. Please let me know what are your thoughts on this.
Flags: needinfo?(georg.fritzsche)
Comment 6•11 years ago
|
||
Kamil can you help Andrei with QA tips for experiments?
Flags: needinfo?(georg.fritzsche) → needinfo?(kamiljoz)
Assignee | ||
Updated•11 years ago
|
status-firefox30:
--- → unaffected
status-firefox31:
--- → affected
status-firefox32:
--- → affected
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 7•11 years ago
|
||
Comment on attachment 8422590 [details] [diff] [review]
Show undo element after manual addon remove
[Approval Request Comment]
Bug caused by (feature/regressing bug #): Telemetry experiments.
User impact if declined: Empty experiment addon list after clicking remove, undo element should be visible.
Testing completed (on m-c, etc.): Fine on m-c.
Risk to taking this patch (and alternatives if risky): Low, just visibility and test changes.
String or IDL/UUID changes made by this patch: None.
Attachment #8422590 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #8422590 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 8•11 years ago
|
||
Comment 9•11 years ago
|
||
Andrei,
I took a quick look and it looks like the experiment that's being listed under "experiments.manifest.uri" has expired. As you can see below, the "endTime" check failed which basically means the current experiment expired: (expired on May 15, 2014)
"1400777176345 Browser.Experiments.Experiments DEBUG ExperimentEntry #0::isApplicable() - id=tile-switcher@experiments.mozilla.org - test 'endTime' failed"
"1400777962433 Browser.Experiments.Experiments TRACE Experiments #0::evaluateExperiments() - added EXPERIMENT_ACTIVATION to TelemetryLog: ["REJECTED","tile-switcher@experiments.mozilla.org","endTime"]"
Some quick tips when debugging experiments:
- Check "Enable chrome and addon debugging" (Open Menu -> Developer -> Toggle Tools -> Settings)
- Set "experiments.logging.level;0" (this will allow you to view similar logs that I added above)
- Open the "Browser Console" which will display logs when enabling/disabling "experiments.enabled"
- Make sure you're on the correct channel (the current experiment was only available under Nightly and would fail under Aurora or any other channel)
- Ensure that you set "experiments.force-sample-value" to match the experiments value or you'll spend a long time enabling/disabling to get the experiment installed
Not sure if Benjamin will launch another experiment any time soon, but your best bet is to create a local staging server so you can control all the parameters. In this case you would simply change "endTime" under the template and generate a new staging server that's not expired.
In the meantime you can use the server I use for testing, I changed the "endTime" to expire on July 15, 2014
- Experiment Info: http://kamiljozwiak.com/telemetry.html
- Change: "experiments.manifest.uri;http://kamiljozwiak.com/firefox-manifest.json"
If you're going to use the above staging server for testing, make sure you change "experiments.manifest.cert.checkAttributes;false"
Flags: needinfo?(kamiljoz)
Comment 10•11 years ago
|
||
Benjamin, would you be able to increase the "endTime" on the development server for QA?
Development Telemetry Server: https://telemetry-experiment-dev.allizom.org
Flags: needinfo?(benjamin)
Comment 11•11 years ago
|
||
No, the current experiment is done (and that really is a staging site). You'll need to set up your own server.
Flags: needinfo?(benjamin)
Comment 12•11 years ago
|
||
Gotcha, thanks Benjamin!
Andrei, you can either set your own staging server using the wiki I created or you can use the one I posted under comment #9.
Comment 13•11 years ago
|
||
(In reply to Kamil Jozwiak [:kjozwiak] from comment #12)
> Gotcha, thanks Benjamin!
>
> Andrei, you can either set your own staging server using the wiki I created
> or you can use the one I posted under comment #9.
Kamil, thank you for following up on this, your instructions worked perfectly.
I was able to confirm the fix for this issue on Nightly 32.0a1 (2014-05-22), using Kamil's staging server with:
* Windows 7 64-bit [1],
* Ubuntu 12.04 LTS 32-bit [2],
* Mac OS X 10.9.2 [3].
[1] Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:32.0) Gecko/20100101 Firefox/32.0
[2] Mozilla/5.0 (X11; Linux i686; rv:32.0) Gecko/20100101 Firefox/32.0
[3] Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:32.0) Gecko/20100101 Firefox/32.0
Status: RESOLVED → VERIFIED
Whiteboard: p=5 s=it-32c-31a-30b.2 [qa+] → p=5 s=it-32c-31a-30b.2 [qa!]
Updated•6 years ago
|
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•