Closed Bug 1247327 Opened 4 years ago Closed 4 years ago

Add WebGL acceptance rates to telemetry

Categories

(Core :: Graphics, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla47
Tracking Status
firefox45 --- wontfix
firefox46 --- fixed
firefox47 --- fixed

People

(Reporter: jrmuizel, Assigned: jrmuizel)

References

(Blocks 1 open bug)

Details

(Whiteboard: gfx-noted)

Attachments

(1 file, 1 obsolete file)

No description provided.
Attachment #8718001 - Flags: review?(vladan.bugzilla)
Whiteboard: gfx-noted
Vladan reviewed this in person.

- What are the questions that these probes are designed to address?
How many people are not getting WebGL
- Is this the minimum amount of data necessary to answer those questions?
Yes
- Who is responsible for monitoring the data?
The GFX team/Me
- How often will they do it?
Maybe once a quarter
- Do the necessary monitoring dashboards already exist or who is going to create them?
Not yet, but it should be easy to add to dvander's telemetry stuff
- What's the user value?
Hopefully more people will WebGL
https://hg.mozilla.org/mozilla-central/rev/1fb660fde013
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Feels like there is overlap between CANVAS_WEBGL_USED and CANVAS_WEBGL_SUCCESS.  I'm not sure we need CANVAS_WEBGL_USED anymore, since the existence of either value in CANVAS_WEBGL_SUCCESS would suggest we were trying to use it.
Also, does it make sense to combine the WebGL1 and WebGL2 and just have an integer histogram that captures all of these?  WebGL1-Failed, WebGL1-Succeeded, WebGL2-Failed, WebGL2-Succeeded - a single histogram with four values instead of three histograms?
Flags: needinfo?(jmuizelaar)
Comment on attachment 8718001 [details] [diff] [review]
Track whether webgl context creation succeeds

Review of attachment 8718001 [details] [diff] [review]:
-----------------------------------------------------------------

I'd like to get this on aurora/beta so that we can gather data sooner without waiting.
Attachment #8718001 - Flags: approval-mozilla-aurora?
Comment on attachment 8718001 [details] [diff] [review]
Track whether webgl context creation succeeds

Approval Request Comment
[Feature/regressing bug #]: None
[User impact if declined]: None
[Describe test coverage new/current, TreeHerder]: None
[Risks and why]:  Just adds telemetry
Flags: needinfo?(jmuizelaar)
Attachment #8718001 - Flags: approval-mozilla-beta?
I'll make another patch that removes CANVAS_WEBGL_USED
Flags: needinfo?(jmuizelaar)
Assignee: nobody → jmuizelaar
Comment on attachment 8718001 [details] [diff] [review]
Track whether webgl context creation succeeds

Too late for 45.
Attachment #8718001 - Flags: approval-mozilla-beta?
Attachment #8718001 - Flags: approval-mozilla-beta-
Attachment #8718001 - Flags: approval-mozilla-aurora?
Attachment #8718001 - Flags: approval-mozilla-aurora+
Comment on attachment 8718001 [details] [diff] [review]
Track whether webgl context creation succeeds

Review of attachment 8718001 [details] [diff] [review]:
-----------------------------------------------------------------

I looked at the telemetry results for this patch and it turns out the patch is garbage.
Attachment #8718001 - Flags: approval-mozilla-aurora+ → approval-mozilla-aurora-
Attachment #8718001 - Attachment is obsolete: true
Attachment #8718001 - Flags: review?(vladan.bugzilla)
Flags: needinfo?(jmuizelaar)
Attachment #8724956 - Flags: review?(milan)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #8724956 - Flags: review?(milan) → review+
https://hg.mozilla.org/mozilla-central/rev/863c6740f637
Status: REOPENED → RESOLVED
Closed: 4 years ago4 years ago
Resolution: --- → FIXED
Looks great! Jeff, once we've got a bit more data on this, could you post them my way? Or if the data is public, can I visit a page to track it manually?
Flags: needinfo?(jmuizelaar)
Comment on attachment 8724956 [details] [diff] [review]
Try for more success with counting this

Approval Request Comment
[Feature/regressing bug #]: None
[User impact if declined]: We'll have less visibility into WebGL success rates.
[Describe test coverage new/current, TreeHerder]: The data seems to be showing up correctly on telemetry.mozilla.com
[Risks and why]: Low risk as it's just a telemetry addition.
Flags: needinfo?(jmuizelaar)
Attachment #8724956 - Flags: approval-mozilla-aurora?
Comment on attachment 8724956 [details] [diff] [review]
Try for more success with counting this

This is already in Aurora47, I believe the request was to uplift to 46 (Beta). Flipping the uplift flag to reflect that.
Attachment #8724956 - Flags: approval-mozilla-aurora? → approval-mozilla-beta?
Comment on attachment 8724956 [details] [diff] [review]
Try for more success with counting this

Since Jeff says this now gives the correct data (in 47/48) let's uplift this for beta 4.
Attachment #8724956 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #17)
> Comment on attachment 8724956 [details] [diff] [review]
> Try for more success with counting this
> 
> Since Jeff says this now gives the correct data (in 47/48) let's uplift this
> for beta 4.

backed out for bustage like https://treeherder.mozilla.org/logviewer.html#?job_id=923464&repo=mozilla-beta
Flags: needinfo?(jmuizelaar)
Jeff do you want to try this again ? I am aiming to start the beta 4 build early this afternoon. So we could still get it in if you can successfully land it. Or, wontfix for beta.
We can still land this in beta 5 which will gtb on Thursday.
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #21)
> We can still land this in beta 5 which will gtb on Thursday.

this is in risk missing beta 5 :(
Milan do you want to take a look here?
Flags: needinfo?(milan)
Crap. I forgot to land... I'll do it now.
Flags: needinfo?(milan)
Flags: needinfo?(jmuizelaar)
Blocks: 1257692
You need to log in before you can comment on or make changes to this bug.