Trac ticket workflow: Difference between revisions
Jump to navigation
Jump to search
Line 6: | Line 6: | ||
== Approval for Release == |
|||
We enter change-control near the end of each release (e.g. [[8.2.0#Schedule|8.2.0 release schedule]]). When we enter change control, the ticket workflow expands to include the 'approve for release', 'add to release', and 'test in release' actions. |
|||
In order to pass the 'approve for release' action, you are expected to provide: |
|||
; packages |
|||
: a complete list of the packages (in source: (E)NVR form, e.g. koji: rainbow-0.7.20.fc9) which must be included to effect your change. |
|||
; changelog |
|||
: a nice human-readable description of your change including citations of tickets that will need to be updated as it is processed. |
|||
; test cases |
|||
: checkable test cases in either the <tt>|TestCase|</tt> Sugar style, the [[Systematic testing]] style, or the [[Tinderbox]] style. |
|||
; test results |
|||
: the results of executing your test cases on a development build (or after having installed your packages on release-stream build) |
|||
; (optional) risk assessment |
|||
: an explanation, if convenient, of what risks are incurred by accepting your change. |
|||
Revision as of 19:58, 29 December 2008
This page is monitored by the OLPC team.
For Developers
This page discusses the workflow which Trac tickets move through. It presents the workflow states, a toy user story, and a brief discussion of the requirements which must be satisfied to move tickets through a delicate portion of the workflow. Questions about this workflow can be recorded on the discussion page. Finally, be aware that this workflow differs from the previous one and will probably change again during the 9.1.0 release cycle.
User Story
Characters
- Anonymous Alice -- generic (possibly minimally skilled) Trac user.
- Terry Ticket-Herder -- ticket herder who watches incoming tickets and improves their quality.
- Carol Coder -- coder who diagnoses, designs, codes, and reviews code.
- Mark Maintainer -- module maintainer (packager) aggregates code and pushes it into build streams (i.e. into Fedora or Joyride).
- Release Rob -- release team member who approves requests to modify a stable build stream.
- Bill Builder -- build team member
- Quality Assurance -- QA team member
Story
- Anonymous Alice observed curious behavior while exploring the system. She records it in Trac but does not set the next_action field or sets it to unknown.
- Terry Ticket-Herder decides Alice's reported behavior is significant but doesn't know how to reproduce it so he changes the next_action to reproduce to indicate his need for instructions on how to reproduce the issue.
- Terry might instead have closed Alice's ticket as invalid, worksforme, etc or have sent it on to a later state if Alice wrote a particularly good ticket.
- If necessary, Terry would also fill in the Milestone, Component, Keywords, and CC fields to conform to our Trac conventions.
- Once Alice's behavior can be reproduced, it can be diagnose[d] by Carol Coder (probably with help from Alice).
- Carol will eventually propose a design that she thinks will resolve the issue.
- Carol will code to her design and will seek review of her work. She will also write a testcase.
- When all is in order, Carol or Mark Maintainer will package the changes and add them to a development build (in the add to build state).
- When built in Joyride, Carol or Mark will test in build.
- If the test fails, then the tester may remand the issue to an earlier state and owner; otherwise, the tester should advance the ticket to the next state (see below).
- Test results should be recorded according to the Trac conventions (e.g as a keyword like joyride-2230:+ for a 'pass' or 8.2-757:- for a 'fail').
- During change control, changes must pass through three additional steps:
- Changes cannot be committed to the stable build stream without meeting basic requirements. Release Rob will regularly review tickets in the approval for release state.
- Approved changes will be sent to the Bill Builder in the add to release state.
- Once Bill updates the stable build stream, he will place the ticket in the test in release state for further testing by Alice, Carol, or another friend.
- After being tested as fixed in a release-stream build, issues should be pushed through some of the qa signoff, finalize, or no action states. See below.
- Occasionally, an issue may require test facilities beyond those available to ordinary developers and testers or may be so important that it requires additional systematic testing. Such issues can be pushed, at anyone's discretion, through the qa signoff state where Quality Assurance will examine them and pass them on.
- Rob will document tickets which reach the finalize state in the Release Notes.
- Tickets will eventually require no action.
- Rob is responsible for closing user-facing issues which result in changes to the build or work-arounds.
- Other issues can be closed by anyone when they cease to be relevant to the release process.