Confirmed users
360
edits
B2G/V1StabilizationPhase: Difference between revisions
→Landing Requirements
(Created page with "=Development Policy for Stabilization Period= ==All bug activity will move to Bugzilla== * New Gaia bugs should be filed in Bugzilla, under the Boot2Gecko product and the Gaia ...") |
|||
| (4 intermediate revisions by 2 users not shown) | |||
| Line 1: | Line 1: | ||
=Development Policy for Stabilization Period= | =Development Policy for Stabilization Period= | ||
==All | ==All Issue Tracking is Done in Bugzilla== | ||
* New Gaia bugs should be filed in Bugzilla, under the Boot2Gecko product and the Gaia component | * New Gaia bugs should be filed in Bugzilla, under the Boot2Gecko product and the Gaia component | ||
* | * Enhancement requests (v2 work) should still be filed in Github | ||
* Existing Gaia blockers have been migrated to Bugzilla, with owners and labels intact. | |||
* Existing Gaia blockers | |||
== | ==Landing Requirements== | ||
* All commits require: | |||
** Blocking-basecamp+ or driver approval (see below) | |||
** Unit or integration tests attached | |||
** Confirmation that smoketests are not regressed | |||
* All other commits will be backed out. | * All other commits will be backed out. | ||
* Driver approval can be requested on the bug or on IRC from: | * Driver approval can be requested on the bug or on IRC from: | ||
** Platform: Andrew Overholt, Chris Jones, Johnny Stenback, Jonas Sicking, Andreas Gal | ** Platform: Andrew Overholt, Chris Jones, Johnny Stenback, Jonas Sicking, Andreas Gal | ||
** Gaia: Vivien Nicolas, Chris Jones | ** Gaia: Vivien Nicolas, Chris Jones, Francisco Jordano | ||
** | ** Any of the Release Management team (Alex Keybl, Lukas Blakk, Bhavana Bajaj) | ||
** To request driver approval on a patch for Gaia changes, set the approval-gaia-master flag to "?" | ** Miguel Schneider, Daniel Coloma, Chris Lee for product decisions | ||
** To request driver approval on a patch for Gaia changes, set the approval-gaia-master flag to "?" and put one of the previously mentioned people in the field so that a specific person is asked. There is no approval triage like there is for blocking-basecamp. Setting the flag will create a template asking you for information, please fill it in. | |||
** As we come closer to shipping, approval-gaia-master will be used for all landings (even if blocking+) | ** As we come closer to shipping, approval-gaia-master will be used for all landings (even if blocking+) | ||
==All commits require | ==Commit Changes== | ||
* All commits now require: | |||
** Unit or integration tests in the pull request or patch | |||
** A commit message format of "Bug {bugNumber} - {summary} (r={reviewer})", or at least have all of those components included. | |||
* Devs are expected to have run their tests locally before submitting any code changes for review, and including the test files in the pull request. | * Devs are expected to have run their tests locally before submitting any code changes for review, and including the test files in the pull request. | ||
* If your commit message doesn't | * If your commit message doesn't have all the required components, it will be backed out. The Gaia dev leads and the release drivers will be triaging the commit logs daily to ensure these requirements are followed. | ||
* Continuous integration + automated smoketests is being implemented at this moment and will be enabled as soon as possible. Once that happens, each and every commit will run all the tests and spit out a report letting you know if your change caused any test failures anywhere in the system. | * Continuous integration + automated smoketests is being implemented at this moment and will be enabled as soon as possible. Once that happens, each and every commit will run all the tests and spit out a report letting you know if your change caused any test failures anywhere in the system. | ||