Bileto

Differences between revisions 25 and 26
Revision 25 as of 2015-01-30 17:46:28
Size: 11512
Editor: S0106602ad0804439
Comment: Merge & Clean is now automatic.
Revision 26 as of 2015-05-05 16:14:29
Size: 11756
Editor: sil2100
Comment:
Deletions are marked like this. Additions are marked like this.
Line 21: Line 21:
   * QA sign off needed?: usually this should be left blank, but the rule is: in normal operation, utopic silos do not need QA sign-off while ubuntu-rtm/14.09 landings by default require it. See [[#QA]] for more information.    * QA sign off needed?: usually this should be left blank, but the rule is: any stable-phone targets like ubuntu/vivid+overlay or ubuntu-rtm/14.09 landings by default require it. See [[#QA]] for more information.
Line 27: Line 27:
   * If you have added some 'Additional source packages to land', you need to provide the trainguards or Ubuntu core-dev with the required source package so that they can upload those to your silo.    * If you have added some 'Additional source packages to land', you need to provide the trainguards or Ubuntu core-dev with the required source package so that they can upload those to your silo. Any core-developer should be able to dput packages to the silo PPAs as well.
Line 51: Line 51:

Generally we are now moving away from the ubuntu-rtm distribution, with the next planned stable branch being vivid + the overlay PPA. More info about this [[#overlay|here]].

Landing changes through the CI Train

Landing your change to Ubuntu

The train provides two options for landing packages. You can either land your changes into the archive through listing merge proposals that you want to release, or by uploading source packages directly to the ppa. The first approach is recommended in all cases where the source package's upstream branch is managed by the CI Train.

Whenever you want to request a release of your component to the archive:

  • If this is your first time running a landing, ask the trainguards in #ubuntu-ci-eng to give you the required permissions for operating the CI Train.

  • Open the CI Train spreadsheet.

  • On the main 'Pending' sheet add a new row to the list, filling out the following required fields.
    • Description of landing: write here an overview of what your landing is about - is it fixing bugs? Adding some features? Try being informative without being over-verbose.
    • Lander: your irc nickname, and that of other people that will also drive the landing (space separated)
    • Comments (optional): used for any additional comments and offline discussion with the Landing Team.
    • Test plans to run: each project handled by CI Train needs to have a test plan that documents how you, the lander, intend to test the given component(s). It's best to present this in the form of a wiki page or other document that can be referred to outside the spreadsheet. See #QA for more details.

    • Merge proposals to land: here you list all the links to the merge proposals you want to land in this landing. Please note, this must be a space separated list of MP links, not branch links. More on this later.

    • Additional source packages to land: here you list the 'additional' source packages that you would like to upload to the silo PPA besides what is already handled by MPs. Can be used for projects that Ubuntu is not upstream for or which have no bzr trunk.
    • Target distribution: select which distribution and series you want to land into.
    • QA sign off needed?: usually this should be left blank, but the rule is: any stable-phone targets like ubuntu/vivid+overlay or ubuntu-rtm/14.09 landings by default require it. See #QA for more information.

    • Ready for a silo?: change this to 'Yes' once you think this landing is ready to get a silo assigned. Before this is done, the Landing Team will not allocate a silo.
  • Once the landing row in the spreadsheet is filled in and the 'Ready for a silo?' field is set to Yes, wait for the trainguards to assign you a silo.
    • If a silo assignment is needed more quickly, you can ping the trainguards directly on #ubuntu-ci-eng
  • After you have a silo allocated, it will appear on the CI Train Dashboard.

  • Use the 'Build' button above your landing card in the dashboard to start building packages from the specified merge requests.
    • If you have added some 'Additional source packages to land', you need to provide the trainguards or Ubuntu core-dev with the required source package so that they can upload those to your silo. Any core-developer should be able to dput packages to the silo PPAs as well.
  • After the packages finish building, it is now the lander's responsibility to test the packages from the PPA on a real device. More information about this is in the #QA section. Remember: you are responsible not only to test that the fix works, but also to test that your change didn't break anything else. Use the Test Plan.

  • If your packages pass your testing criteria, mark the silo as tested:
    • Get all your MP:s top-approved. If you don't use the feature in your team's processes, just mark it as top-approved yourself.

    • Find your landing row on the CI Train Spreadsheet again.

    • Go to the 'Testing pass?' column.
    • Set it to the following: Yes (<image_number> <device_name> <person_that_tested>), e.g. 'Yes (#88 krillin sil2100).

  • In case your silo needs QA sign-off, wait for the QA Team to finish testing on the silo. This will be indicated in the spreadsheet and the dashboard.
  • The landing team will now publish the silo to the archive if everything is OK packaging-wise. Note: Merge Proposals need to be top-approved latest at this point, otherwise landing team cannot publish the silo.

This is the standard process. When you prepare a list of MPs to land and provide them to CI Train, the infrastructure will merge all of them together and build a source package using bzr bd. This package is then uploaded to the silo PPA that has been assigned to your landing. CI Train also automatically generates the debian/changelog for you using the MPs' commit-messages. Every landing creates a new version in the changelog with one entry per merge. The version number is also auto-generated every time. After all changes land into the archive, during the 'Merge & Clean' state these merges are all finally merged into trunk.

So, the general steps in the landing process from the lander's perspective are:

  1. Fill in a landing request
  2. Wait for silo assignment
  3. Build packages in silo PPA
  4. Test packages from the silo PPA on real hardware
  5. Mark silo as tested
  6. (in some cases) Wait for silo to be QA signed-off
  7. Wait for silo to be published

Landing your change to Ubuntu RTM

Generally we are now moving away from the ubuntu-rtm distribution, with the next planned stable branch being vivid + the overlay PPA. More info about this here.

The process of landing things to Ubuntu RTM is vaguely similar to the process for Ubuntu. The CI Train supports multiple distributions and series to which it can release packages. Just select 'ubuntu-rtm/14.09' as the Target distribution.

There are, however, some additional rules and formalities that need to be followed in case a change is to be landed into Ubuntu RTM.

  1. The change needs to land into the current development series first (currently: vivid)
  2. The Ubuntu RTM silo is required to pass the QA Sign-off process

With these rules in mind, there are two main formal ways of landing changes into Ubuntu RTM:

  1. Separate branches for ubuntu and ubuntu-rtm. In this case a change that lands in the ubuntu branch needs to be cherry-picked into the ubuntu-rtm branch and re-released.
  2. One branch targeting ubuntu and ubuntu-rtm. A change is released to ubuntu and this change is then synced up to ubuntu-rtm as a source sync (source copy through the CI Train).

More details on those approaches can be found in citrain/RTMLandingApproaches.

Modifying your existing landing

Sometimes after a landing is prepared, a silo assigned and the packages built, during testing the lander can notice that some changes are missing and need to be added for the feature to be complete. This usually requires additional merges or sources included in the silo. CI Train handles this case, but requires certain actions.

Whenever a merge or source is added to the landing in the spreadsheet after assignment, the merge and source is not automatically noticed by CI Train. A silo reconfigure is needed first. A silo can be reconfigured by either the trainguards or the owning lander, where the latter can only reconfigure in certain cases.

A lander can reconfigure a silo on his own when:

  • Only merge requests field has been changed
  • No additional projects/components are added to the silo besides the ones already in use
  • Or, in the case of removing a merge from the list

Example: if a silo has 3 merges for unity8 and the lander decides, after assignment, that he needs to add another unity8 merge to the list - this can be done by the lander. But if the lander decides he/she needs to add a qtmir merge to the mix, then this would involve adding a new component (qtmir) - which is not possible without the Landing Team intervention. In this case the merge can be added to the list, but only the trainguards can reconfigure it for the changes to take effect.

To reconfigure, please edit the list of merges on the CI Train spreadsheet and follow the Reconfigure Silo link present in your landing line.

In all other cases, please ping the trainguards with the details and wait for a reconfigure from his side. This is required in case the new component is already locked in a different silo or, for instance, thought to be too risky in terms of the Landing Team.

QA Sign-off Needed

Under certain circumstances your landing may need to go through QA sign off. This is usually during situations when the quality level of the image has regressed severely or must be maintained at a high-level. Right now these are:

  • During TRAINCON-0
  • For all landings to the ubuntu-rtm branch

There is an exception to this rule if your landing contains an isolated bug fix. Right now there is not a strict definition of this term, but as guidance an isolated bug fix landing is:

  • Only one or a couple of bug fixes
  • A small change and not impacting a large number of sub-components of that package
  • A change in a package that does not have a lot of reverse dependencies, since this makes it much harder to assess the impact of the landing.

If QA sign-off is required then you will be expected to provide more detailed information about the testing you have done in the Test plans to run column of the landing spreadsheet so that QA have a good basis on which to build their additional testing. Here are some tips:

  • Never just test that the bug(s) alone are fixed - remember the potential for regressions
  • Look at what is actually landing and make sure testing is done for all the updated packages. For example if the primary intent of the silo is to fix music player controls, but the fix requires a media-hub update then you should probably run the whole media-hub test plan.
  • Take into consideration the dependencies and reverse dependencies involved - ask yourself, who could I break? Ideally your package's test plan should have some guidance on this (for example see the Dependents/Clients section media-hub's test plan

  • If you only provide a link to the test plan, then bear in mind that QA will assume you've run all the tests in that plan and they have passed. If tests then fail for a reason you haven't mentioned then it will cause confusion and delay the landing, so make sure to record all cases where you have not run a test or a test has failed (and why).

Lastly, if you don't use the citrain tool for installing the silo then you need to mention what steps you used to install the silo, since citrain is the assumed way.

Isolated bugfixes for ubuntu-rtm - confirmation

Whenever a landing is targeting ubuntu-rtm and is marked by the lander as not requiring QA sign-off because of an isolated bugfix, the Landing Team tries to verify if this indeed falls into this category. The Landing Team either by themselves or with the help of the QA team look at the landing description, looking for any rationale for this exception. Merge requests or diffs are also examined in case that's needed.

The best practice is to directly ping the QA team with a question if your change can go through without a full QA sign-off. The QA Team is here to ensure that things landing into ubuntu-rtm are as safe as possible.

Bileto (last edited 2021-02-10 09:39:58 by sil2100)