10455
Comment:
|
10564
|
Deletions are marked like this. | Additions are marked like this. |
Line 18: | Line 18: |
|| [http://kb-prod-udv-001.kb.dk:8060/cru/NS-54 NS-54] || Abandoned || - || - || - || - ||-- ||- ||- ||- || |
Reviews in Iteration 37BR Use IterationReviewsOverviewTemplate template to start new review pages.
Contains: [#CodeReViewTplCrTable Code Review Table], [#CodeReViewTplOrTable Document Review Table], [#CodeReViewTplCrColumns Code Review Column Description], [#CodeReViewTplOrColumns Document Review Column Description] Anchor(CodeReViewTplCrTable)
Code ReviewsBR Description of columns can be found in section [#CodeReViewTplCrColumns Explanation of the columns in Code Review table]. BR For details on how to do a code review, see [:Process/Code Review:Code Review Process]. BR Code Reviews are done in the review-tool Crucible: [http://kb-prod-udv-001.kb.dk:8060/ Link to Crucible], export is done via [http://kb-prod-udv-001.kb.dk:8060/plugins/servlet/export Crucible export function] BR Template for insertion of a new line in edit mode: BR || [http://kb-prod-udv-001.kb.dk:8060/cru/NS-XX NS-XX] || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || IssuesFromNsXX || INIT || - || - || - ||- ||- ||
Review Id BR NS-XX |
Task BR Bug/FR XXX |
Author(s)BR INIT |
Reviewer(s) BR INIT |
Review date BR YYYY/MM/DD |
Issues found BR IssuesFromNsXX |
Follow-up BR INIT |
Done BR -, OK, OK-wp |
Releasetest BR excl. or incl. TEST case.item and status |
Manuals BR Manual page(s)/chapter(s) |
FR 1654 |
Andreas |
SVC |
2009/06/19 |
Andreas |
OK |
- |
- |
||
Bug 1152 (reopened) |
CSR |
SVC |
2009/05/11 |
CSR |
OK |
- |
- |
||
FR 1628 |
N.Giraud |
SVC |
2009/06/11 |
NGIRAUD |
OK |
- |
- |
||
Bug 1694 |
N.Giraud |
SVC |
2009/06/11 |
NGIRAUD |
OK |
incl TEST1F? |
- |
||
Bug 1254 |
N.Giraud |
SVC |
2009/06/11 |
NGIRAUD |
OK |
- |
- |
||
Bug 1611, 1636, 1646 |
SVC |
JOLF |
2009/07/23 |
SVC |
OK |
No changes neeeded to releasetest |
- |
||
FR 1675 |
Andreas |
SVC |
2009/06/19 |
Andreas |
OK |
incl TEST1 item 4 ? |
|||
Bug 1711 |
KFC |
CSR |
2009/06/30 |
KFC |
OK |
- |
- |
||
Bug 1336 |
KFC |
HBK |
-- |
-- |
- |
- |
- |
||
Bug 1188 |
CSR |
JOLF |
2009/07/27 |
-- |
- |
- |
- |
||
Bug 1188, 1174, 1661 |
CSR |
JOLF |
2009/07/28 |
-- |
- |
- |
- |
||
Abandoned |
- |
- |
- |
- |
-- |
- |
- |
- |
|
Bug 928 |
HBK |
SVC |
2009/07/14 |
-- |
- |
- |
- |
||
FR 1014 |
HBK |
SVC |
2009/07/08 |
-- |
- |
- |
- |
||
Bug 1650 |
Eleonora |
SVC |
2009/07/15 |
-- |
OBSOLETE,See NS-62 |
- |
- |
||
FR 1709 |
Andreas |
SVC,HBK |
|
-- |
- |
- |
- |
||
FR 1716-1717, Bug 1644 |
SVC |
JOLF |
2009/07/23 |
SVC |
OK |
Releasetest, that uses template download pages should verify that these bugs are verified |
- |
||
Bug 1650 |
JOLF |
HBK |
|
-- |
- |
- |
- |
||
Bug 555 and related |
KFC* |
SVC,HBK |
|
-- |
- |
- |
- |
||
Bug 1722: Excessive logging in indexserver |
KFC* |
JOLF,SVC |
2009/07/22 |
-- |
OK |
Releasetest? |
- |
||
Bug 1547: Wrong synchronization ... |
KFC* |
SVC,JOLF |
2009/07/22 |
-- |
OK |
Releasetest? |
- |
||
Bug 1718: The link to monitor Heritrix process .. |
KFC* |
JOLF,SVC |
2009/07/22 |
-- |
OK |
Releasetest? |
- |
||
Bug 1712: Starting multiple applications .. |
KFC* |
JOLF,SVC |
2009/07/22 |
JOLF |
OK |
Releasetest? |
- |
||
Bug 1700: The WebProxy.handle() method creates CreateErrorResponse for null Uri |
KFC* |
JOLF,SVC |
2009/07/22 |
-- |
OK |
Releasetest? |
- |
||
Bug 1702 |
N.Giraud* |
JOLF, SVC |
2009/07/28 |
SVC |
- |
- |
- |
Authors marked with * have not been part of the review
Anchor(CodeReViewTplOrTable) Document ReviewsBR Description of columns can be found in section [#CodeReViewTplOrColumns Explanation of the columns in Document Review table].BR For details on how to other reviews, see [:Process/Document Review:Document Review Process].BR Template for insertion of a new line in edit mode:BR || [:DocumentReview/DocNameIdReview:DocName] || Version || Lines || Bug/FR XXX || INIT || INIT || YYYY/MM/DD || INIT || - ||
Document BR [:link:docName] |
Version BR X.Y/date |
Lines/parts BR All/x-y |
Task BR Bug/FR XXX |
Author(s)BR INIT |
Reviewer(s) BR INIT |
Review date BR YYYY/MM/DD |
Follow-up BR INIT |
Done BR -, OK, OK-wp |
Releasetest BR excl. or incl. TEST case.item and status |
Manuals BR Manual page(s)/chapter(s) |
Version |
Lines |
Bug/FR XXX |
INIT |
INIT |
YYYY/MM/DD |
INIT |
- |
- |
- |
Anchor(CodeReViewTplCrColumns) Explanation of the columns in Code Review table:
"Review Id": The Crucible Review Id which the review is created under in Crucible. Normally name is on form NS-XX, i.e. text to be inserted is [http://kb-prod-udv-001.kb.dk:8060/cru/NS-XX NS-XX]
- "Task": The assignment or tracker issue that the code has been updated for, e.g. Bug 1512.
- "Author(s)": The person(s) who have made changes or additions to the code. Only Initials are given, e.g. ELZI.
- "Reviewer(s)": The person(s) who have not been involved in coding these changes, who will participate in the review.
- "Review date": Date for review.
- "Issues found": Link to wiki page with issues found under review (and mandays of work). Normally on form IssuesFromNsXX.
- "Follow-up": The person who will do the follow-up on the review specified under 'Issues found'.
"Done": whether the review follow-up has been done. Has value "-" if new, "OK" if all follow-ups are done, "OK-wp" (with postpones) if follow-ups are done with exceptions that have been postponed.
"Release Test": Whether the changes are to be included in the Release test. Has values "-" if new, "incl." if it is to be included in the release test and "excl." if it cannot be included in the release test (e.g. if not testable).
"Manuals": The manuals that has to be/is updated as a concequence of the change. Has values "-" if new, "Quick" for [:Quick Start Manual devel:Quick Start Manual], "Deploy" for [:Deploy Manual devel:Deploy Manual], "Conf." for [:Configuration Manual devel:Configuration Manual], "Inst.." for [:Installation Manual devel:Installation Manual], "Dev." for [:Developer Manual devel:Developer Manual], "User" for [:User Manual devel:User Manual].
Anchor(CodeReViewTplOrColumns) Explanation of the columns in Document Review table:
"Document": Link to issue review page for document named with identification of the document, e.g. http://netarchive.dk/suite/AssignmentDeploy1 - See example in [Iteration 36].
- "Version": The SVN, CVS or date for revision of document/script to be reviewed.
- "Parts/lines": Specifies the parts of the document/script to review (if less that the whole file).
- "Task": The assignment or tracker issue that the code has been updated for, e.g. Bug 1512.
- "Author(s)": The person(s) who have made changes or additions to the code. Only Initials are given, e.g. ELZI.
- "Reviewer(s)": The person(s) who have not been involved in coding these changes, who will participate in the review.
- "Review date": Date for review.
- "Follow-up": The person who will do the follow-up on the review specified under 'Issues found'.
"Done": whether the review follow-up has been done. Has value "-" if new, "OK" if all follow-ups are done, "OK-wp" (with postpones) if follow-ups are done with exceptions that have been postponed.