Firefox Core Engineering: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(updated, 2017-03-27)
(update of main page to reflect Q3)
Line 1: Line 1:
== Function ==
== Function ==
Quoting the [[Platform/UI Team]]:
The purpose of this team is to address needs that fall between Toolkit and Platform, with an emphasis (currently) on improving stability, quality, and performance – supported by empirical data. As such, we overlap a bit with everyone from Gecko, Desktop, Data, and more.
<blockquote>"The Platform teams and Firefox teams are co-dependent, but oftentimes priorities between the two groups do not line up perfectly. If Platform has a goal to ship some new feature that requires UI, and the Firefox team has no cycles to help develop the UX or UI, then it that feature will often languish."</blockquote>
This can happen in the opposite direction too, with Toolkit needing support for a feature that doesn't line up with current Platform priorities.


The purpose of this team is to address needs that fall between Toolkit and Platform, with an emphasis (currently) on improving stability, quality, and performance &ndash; supported by empirical data. As such, we overlap a bit with everyone from Platform through Toolkit, Data, and more.
This team grew out of, in part, the Performance Engineering team, and owns some of that team's infrastructure &ndash; some performance-related dashboards on telemetry.mozilla.org, crash analysis, hang visualization, etc. It also includes the installer & updater applications.
 
This team grew out of, in part, the Performance Engineering team, and owns that team's previous infrastructure &ndash; performance-related dashboards on telemetry.mozilla.org, the symbolication server, and more. It also includes the installer & updater applications.


== Personnel ==
== Personnel ==
* Carl Corcoran (:ccorcoran)
* Neil Deakin (:enn)
* Neil Deakin (:enn)
* Adam Gashlin (:agashlin)
* Adam Gashlin (:agashlin)
Line 23: Line 18:
== Goals ==
== Goals ==
# <b>Enabling stability</b><br/>In general, things we do should be tied to enabling stability &ndash; so, making it measurable and/or addressing issues.
# <b>Enabling stability</b><br/>In general, things we do should be tied to enabling stability &ndash; so, making it measurable and/or addressing issues.
# <b>Supporting performance improvements</b><br/>Improving performance is certainly everyone's job&mdash;not just our team&mdash;but we hold the keys for some distinct pieces of the analysis that allow people to understand what needs to be improved. This is primarily, but not limited to, telemetry and data analysis.
# <b>Supporting performance improvements</b><br/>Improving performance is certainly everyone's job&mdash;not just our team&mdash;but we hold the keys for some distinct historical pieces of the analysis that allow people to understand what needs to be improved. This is primarily, but not limited to, telemetry and data analysis.
# <b>Improving the user/contributor experience</b><br/>This one is the weirdest: it covers things that we can do to further the web, and improve the experience for our users &ndash; both end-users and code contributors (some examples: Flash blocking, XUL performance analysis). This category is the most ripe for population.
# <b>Improving the user/contributor experience</b><br/>This one is the weirdest: it covers things that we can do to further the web, and improve the experience for our users &ndash; both end users and code contributors (some examples: Flash blocking, XUL performance analysis). This category is the most open-ended for future expansion.


== Communication ==
== Communication ==
You can typically find us in:
You can typically find us in:
=== IRC ===
=== IRC ===
* '''#fce'''
* '''#fce''' (primary)
* #developers
* #developers
* #e10s
* #e10s
* #fx-team
* #fx-team
* #perf
* #perf
* #releng
* #telemetry
* #telemetry
* #uptime
* #uptime
Line 40: Line 36:
* dev-platform
* dev-platform
* fhr-dev
* fhr-dev
* firefox-dev


== Process and Queuing ==
== Process and Queuing ==
There is currently no regimented process for regular triage of candidate work. Needs usually filter down through Benjamin Smedberg's team or tangentially related to performance analysis and experimentation.
There is currently no regimented process for regular triage of candidate work. Needs usually filter down through performance analysis and experimentation.


[https://bugzilla.mozilla.org/buglist.cgi?cmdtype=dorem&remaction=run&namedcmd=all_fce-active&sharer_id=500559 All actively tracked work is marked with the whiteboard "<nowiki>[fce-active]</nowiki>"] (for now). Or look at the [[#Active Bug List]] on this page.
[https://bugzilla.mozilla.org/buglist.cgi?cmdtype=dorem&remaction=run&namedcmd=all_fce-active&sharer_id=500559 All actively tracked work is marked with the whiteboard "<nowiki>[fce-active]</nowiki>"] (for now). Or look at the [[#Active Bug List]] on this page.
Line 50: Line 47:
== Owned Infrastructure (needs updating per {{Bugzilla|1298080}}) ==
== Owned Infrastructure (needs updating per {{Bugzilla|1298080}}) ==
=== telemetry.mozilla.org Dashboards ===
=== telemetry.mozilla.org Dashboards ===
All of these dashboards are in the process of transferring ownership. Please contact ddurst if you need data that is currently not functional.
* '''Update Orphaning:''' functional
* '''Update Orphaning:''' functional
* '''Stability Dashboard:''' functional


=== symbolapi.mozilla.org ===
=== symbolapi.mozilla.org ===
This is the [[Snappy_Symbolication_Server|symbolication server]] (aka "[https://github.com/mozilla/Snappy-Symbolication-Server Snappy Symbolication Server]") used by platform developers and performance dashboards. It is '''not''' used for the analogous process on Socorro. '''This is currently being re-written by the owner of symbols, peterbe.'''
This is the [[Snappy_Symbolication_Server|symbolication server]] (aka "[https://github.com/mozilla/Snappy-Symbolication-Server Snappy Symbolication Server]") used by platform developers and performance dashboards. It is '''not''' used for the analogous process on Socorro. '''This is currently slated to be replaced by the owner of symbols, peterbe. See [https://github.com/mozilla-services/tecken Tecken].'''


== Historical knowledge areas ==
== Historical knowledge areas ==
* back-end of the user interface/XUL, et al (enn)
* back-end of the user interface/XUL, et al (enn)
* e10s system add-on (felipe)
* e10s system add-ons & system add-ons for feature rollout (felipe)
* e10s data analysis (chutten)
* e10s data analysis (chutten)
* install and update (rstrong, mhowell, agashlin)
* install and update (rstrong, mhowell)
* telemetry, histograms, pings, and data reporting (chutten)
* telemetry, histograms, pings, and data reporting (chutten)
* stack-walking and breakpad (gsvelto, ccorcoran)
* stack-walking, breakpad, and crash pings (gsvelto, ccorcoran)
* flash plugin-related (bytesized, felipe, dthayer)
* flash plugin-related (bytesized, felipe, dthayer)


Line 69: Line 64:
=== Current efforts ===
=== Current efforts ===
==== Get More Data Faster ====
==== Get More Data Faster ====
We need to reduce known blind spots and barriers to getting data. For this, our goals are to:
We need to reduce known blind spots and barriers to getting data AND commit to non-ADI based metrics. For this, our goals are to:
* <strike>enable client-side stackwalking and send basic stack traces with crash pings (beginning in Nightly/Aurora, see {{Bugzilla|1280484}})</strike>
* <strike>enable client-side stackwalking and send basic stack traces with crash pings (beginning in Nightly/Aurora, see {{Bugzilla|1280484}})</strike>
* <strike>enable content process crash reports ({{Bugzilla|1293656}})</strike>
* <strike>enable content process crash reports ({{Bugzilla|1293656}})</strike>
Line 75: Line 70:
* process stack data in crash pings into a queryable result ({{Bugzilla|1310695}})
* process stack data in crash pings into a queryable result ({{Bugzilla|1310695}})
* <strike>create pingSender to handle crash pings instead of Gecko ({{Bugzilla|1310703}})</strike>
* <strike>create pingSender to handle crash pings instead of Gecko ({{Bugzilla|1310703}})</strike>
* enable client-side stackwalking and send basic stack traces with crash pings on Beta/release
* enable client-side stackwalking and send basic stack traces with crash pings on all channels


See the [[Firefox_Core_Engineering/Get_More_Data_Faster|roadmap here]].
See the [[Firefox_Core_Engineering/Get_More_Data_Faster|roadmap here]].


==== Stability Dashboard for Relman ====
==== Set Flash to CTA by default ====
Relman has been using arewestableyet.com and related graphs to understand stability by build and channel; this is fine, but it relies on ADI and crash-stats rather than telemetry, and this is known to be unreliable. For this, our goals are to:
This includes:
* <strike>create a dashboard like arewestableyet.com, but based on telemetry ({{Bugzilla|1297146}})</strike> '''Stability dashboard''': https://telemetry.mozilla.org/crashes/
* establish confidence levels based on kilousagehours by comparing telemetry-based stability data with ADI-based stability data (ON HOLD)
 
==== Experiment with setting Flash to CTA by default ====
All major browsers are stopping support for Flash; Firefox soon will only support Flash. Because there can be unpleasant user impact in blocking all Flash, we want to understand and attempt to smooth the transition to a post-Flash world. This includes:
* prefer fallback content to Flash ({{Bugzilla|1277346}})
* prefer fallback content to Flash ({{Bugzilla|1277346}})
* establish allowedlists and deniedlists ({{Bugzilla|1307604}}, {{Bugzilla|1307605}})
* <strike>establish allowedlists and deniedlists ({{Bugzilla|1307604}}, {{Bugzilla|1307605}})</strike>
* use heuristics to control when Flash is set to Click To Activate ({{Bugzilla|1307606}})
* <strike>use heuristics to control when Flash is set to Click To Activate ({{Bugzilla|1307606}})</strike>
We hope to do a geography-specific SHIELD study in FF53 release to understand user response and impact ({{Bugzilla|1277346}}).
* <strike>geography-specific SHIELD study in FF53 release to understand user response and impact ({{Bugzilla|1335232}})</strike>


See the [https://docs.google.com/document/d/1sYp0DNioPA5iF3iw9LHGf1uN5B5AgdCu7jJxAh0MiqA/edit details here].
See the [https://docs.google.com/document/d/1sYp0DNioPA5iF3iw9LHGf1uN5B5AgdCu7jJxAh0MiqA/edit details here].
==== Dashboard (repairs and additions) ====
Dashboards. So many dashboards.
* <strike>change Update Orphaning dashboard to use MainSummary instead of longitudinal dataset</strike>
* <strike>Removing potentially unused dashboards from t.m.o ({{Bugzilla|1324526}})</strike>
* Updating dashboards ({{Bugzilla|1324528}})


<!--
<!--
Line 103: Line 87:
XUL is supposed to go away, but it would seem that we don't know what the performance implications could/will be. This work builds on Neil Deakin's 2015 experiment to shed some light on where we need to focus our optimization/change efforts.
XUL is supposed to go away, but it would seem that we don't know what the performance implications could/will be. This work builds on Neil Deakin's 2015 experiment to shed some light on where we need to focus our optimization/change efforts.
-->
-->
==== Updater and Orphan remediation ====
 
Remediation efforts have been tested for both system add-on capable and non (44.x and 43.0.1, respectively). Analysis thus far confirms the reach but not the effectiveness or rate of conversion that we'd hoped for. This means:
=== App Updater and Installers ===
 
==== Update Orphan remediation ====  
Remediation efforts have been tested for both system add-on capable and non (44.x and 43.0.1, respectively). Efforts are identified by ongoing analysis, including the [https://telemetry.mozilla.org/update-orphaning/ update orphaning dashboard]. This has yielded such things as:
* continue the download instead of starting over after other networking errors occur ({{Bugzilla|1309124}})
* create an Update Agent, responsible for running independently, daily, and downloading an update if found ({{Bugzilla|1343669}})
* <strike>continue the download instead of starting over after NS_ERROR_DOCUMENT_NOT_CACHED occurs ({{Bugzilla|1272585}})</strike> (FF49)
* <strike>continue the download instead of starting over after NS_ERROR_DOCUMENT_NOT_CACHED occurs ({{Bugzilla|1272585}})</strike> (FF49)
* continue the download instead of starting over after other networking errors occur ({{Bugzilla|1309124}})
* <strike>download the update MAR file unthrottled (already landed) ({{Bugzilla|1309125}}, {{Bugzilla|1309668}})</strike>
* <strike>download the update MAR file unthrottled (already landed) ({{Bugzilla|1309125}}, {{Bugzilla|1309668}})</strike>
* <strike>serve a partial MAR file to Firefox 43.0.1 clients ({{Bugzilla|1309130}})</strike>
* <strike>serve a partial MAR file to Firefox 43.0.1 clients ({{Bugzilla|1309130}})</strike>
* <strike>push either a system or hotfix add-on that changed the download throttle preference to 0 for FF 50+</strike>
* <strike>push either a system or hotfix add-on that changed the download throttle preference to 0 for FF 50+</strike>
* <strike>run another method (non sysaddon, non SHIELD?, etc) to urge 43.0.1 users to upgrade</strike>
* <strike>run another method (non sysaddon, non SHIELD?, etc) to urge 43.0.1 users to upgrade</strike>
* Updater UI is outdated, too big, and needs to be updated ({{Bugzilla|893505}}) (FF55)
* <strike>Updater UI is outdated, too big, and needs to be updated ({{Bugzilla|893505}}) (FF55)</strike>
** most recent mockups: https://mozilla.invisionapp.com/share/Y776FIBWS#/screens
* <strike>change compression to LZMA for updates ({{Bugzilla|641212}}) (FF56)</strike>
* change compression to LZMA for updates ({{Bugzilla|641212}}) (FF55?)
* <strike>move to SHA-384 for MAR signing ({{Bugzilla|1324498}}) (FF56)</strike>
* move to SHA-384 for MAR signing ({{Bugzilla|1324498}}) (FF55?)
* create an Update Agent, responsible for running independently, daily, and downloading an update if found ({{Bugzilla|1343669}})


==== Install UI ====
==== Install UI ====
* Streamlined installer testing in QX onboarding funnelcake ({{Bugzilla|1328445}})  
* <strike>Streamlined installer testing in QX onboarding funnelcake ({{Bugzilla|1328445}})</strike>


==== Windows 64 ====
==== Windows 64 ====
Line 126: Line 112:


=== Current projects ===
=== Current projects ===
==== 2017 Q1 goals ====
==== 2017 Q3 goals ====
* <strike>update https://telemetry.mozilla.org/update-orphaning/ to v2</strike>
* UPDATE/INSTALL:
* start querying stacks received from crash pings
** improve update security and performance via LZMA for MAR file compression and SHA384 certificates for MAR file signing
* <strike>relaunch of symbolapi.m.o (ON HOLD)</strike>
** Increase visibility for update telemetry
* implement crash ping signatures
** Support onboarding
* create "Plugin Safety" (Flash) SHIELD study
* CRASH MACHINERY:
* simplify Updater UI
** Optimize crash data that we send
* LZMA and SHA384 for MAR files
** Start querying stacks received from crash pings
* <strike>land pingSender and implement for crashes and main pings</strike>
** Implement crash ping signatures
* PERFORMANCE:
** Assist with measuring (and identifying) jank and hang via BHR
** Assist with Quantum Flow and Photon Performance efforts for FF57 and FF58
 




== Potential future projects ==
== Potential future projects ==
This list should be considered a work in progress. Decisions will be reflected for a particular quarter.
This list should be considered a work in progress. Decisions will be reflected for a particular quarter.
* Assisting with measuring (and addressing) jank and hang
 
* BHR
* TBD
* assisting with Quantum Flow
* assisting with Photon


== (partial) Active Bug List ==
== (partial) Active Bug List ==

Revision as of 21:42, 11 September 2017

Function

The purpose of this team is to address needs that fall between Toolkit and Platform, with an emphasis (currently) on improving stability, quality, and performance – supported by empirical data. As such, we overlap a bit with everyone from Gecko, Desktop, Data, and more.

This team grew out of, in part, the Performance Engineering team, and owns some of that team's infrastructure – some performance-related dashboards on telemetry.mozilla.org, crash analysis, hang visualization, etc. It also includes the installer & updater applications.

Personnel

  • Neil Deakin (:enn)
  • Adam Gashlin (:agashlin)
  • Felipe Gomes (:felipe)
  • Matt Howell (:mhowell)
  • Chris HC (:chutten) -- honorary
  • Kirk Steuber (:bytesized)
  • Robert Strong (:rstrong)
  • Gabriele Svelto (:gsvelto)
  • Doug Thayer (:dthayer)
  • David Durst (:ddurst)

Goals

  1. Enabling stability
    In general, things we do should be tied to enabling stability – so, making it measurable and/or addressing issues.
  2. Supporting performance improvements
    Improving performance is certainly everyone's job—not just our team—but we hold the keys for some distinct historical pieces of the analysis that allow people to understand what needs to be improved. This is primarily, but not limited to, telemetry and data analysis.
  3. Improving the user/contributor experience
    This one is the weirdest: it covers things that we can do to further the web, and improve the experience for our users – both end users and code contributors (some examples: Flash blocking, XUL performance analysis). This category is the most open-ended for future expansion.

Communication

You can typically find us in:

IRC

  • #fce (primary)
  • #developers
  • #e10s
  • #fx-team
  • #perf
  • #releng
  • #telemetry
  • #uptime

Mailing lists

  • dev-platform
  • fhr-dev
  • firefox-dev

Process and Queuing

There is currently no regimented process for regular triage of candidate work. Needs usually filter down through performance analysis and experimentation.

All actively tracked work is marked with the whiteboard "[fce-active]" (for now). Or look at the #Active Bug List on this page.

Major initiatives are listed on this page.

Owned Infrastructure (needs updating per 1298080)

telemetry.mozilla.org Dashboards

  • Update Orphaning: functional

symbolapi.mozilla.org

This is the symbolication server (aka "Snappy Symbolication Server") used by platform developers and performance dashboards. It is not used for the analogous process on Socorro. This is currently slated to be replaced by the owner of symbols, peterbe. See Tecken.

Historical knowledge areas

  • back-end of the user interface/XUL, et al (enn)
  • e10s system add-ons & system add-ons for feature rollout (felipe)
  • e10s data analysis (chutten)
  • install and update (rstrong, mhowell)
  • telemetry, histograms, pings, and data reporting (chutten)
  • stack-walking, breakpad, and crash pings (gsvelto, ccorcoran)
  • flash plugin-related (bytesized, felipe, dthayer)

Pipeline

Current efforts

Get More Data Faster

We need to reduce known blind spots and barriers to getting data AND commit to non-ADI based metrics. For this, our goals are to:

  • enable client-side stackwalking and send basic stack traces with crash pings (beginning in Nightly/Aurora, see 1280484)
  • enable content process crash reports (1293656)
  • differentiate between process types in crash pings (1310664)
  • process stack data in crash pings into a queryable result (1310695)
  • create pingSender to handle crash pings instead of Gecko (1310703)
  • enable client-side stackwalking and send basic stack traces with crash pings on all channels

See the roadmap here.

Set Flash to CTA by default

This includes:

  • prefer fallback content to Flash (1277346)
  • establish allowedlists and deniedlists (1307604, 1307605)
  • use heuristics to control when Flash is set to Click To Activate (1307606)
  • geography-specific SHIELD study in FF53 release to understand user response and impact (1335232)

See the details here.


App Updater and Installers

Update Orphan remediation

Remediation efforts have been tested for both system add-on capable and non (44.x and 43.0.1, respectively). Efforts are identified by ongoing analysis, including the update orphaning dashboard. This has yielded such things as:

  • continue the download instead of starting over after other networking errors occur (1309124)
  • create an Update Agent, responsible for running independently, daily, and downloading an update if found (1343669)
  • continue the download instead of starting over after NS_ERROR_DOCUMENT_NOT_CACHED occurs (1272585) (FF49)
  • download the update MAR file unthrottled (already landed) (1309125, 1309668)
  • serve a partial MAR file to Firefox 43.0.1 clients (1309130)
  • push either a system or hotfix add-on that changed the download throttle preference to 0 for FF 50+
  • run another method (non sysaddon, non SHIELD?, etc) to urge 43.0.1 users to upgrade
  • Updater UI is outdated, too big, and needs to be updated (893505) (FF55)
  • change compression to LZMA for updates (641212) (FF56)
  • move to SHA-384 for MAR signing (1324498) (FF56)

Install UI

  • Streamlined installer testing in QX onboarding funnelcake (1328445)

Windows 64

We want to start moving users to 64-bit when appropriate:

  • stub installer should automatically select 32-bit or 64-bit (797208)


Current projects

2017 Q3 goals

  • UPDATE/INSTALL:
    • improve update security and performance via LZMA for MAR file compression and SHA384 certificates for MAR file signing
    • Increase visibility for update telemetry
    • Support onboarding
  • CRASH MACHINERY:
    • Optimize crash data that we send
    • Start querying stacks received from crash pings
    • Implement crash ping signatures
  • PERFORMANCE:
    • Assist with measuring (and identifying) jank and hang via BHR
    • Assist with Quantum Flow and Photon Performance efforts for FF57 and FF58


Potential future projects

This list should be considered a work in progress. Decisions will be reflected for a particular quarter.

  • TBD

(partial) Active Bug List

No results.

0 Total; 0 Open (0%); 0 Resolved (0%); 0 Verified (0%);