Identification of the deliverable or milestone | |
---|---|
Project: EMI | Deliverable or milestone identifier: DJRA1.7.1 |
Title: DJRA1.7.1 – Software Development Quality Control Report | Doc. identifier: DJRA1.7.1_Software_Development_Quality_ControlReport_latest.odt |
Author(s): Andrea Ceccanti | Due date: __ |
Identification of the reviewer | ||
---|---|---|
Name: F. Giacomini | Affiliation: INFN | EMI Activity/External project or Institute: SA1 |
Review date | 2010-11-4 |
Author(s) revision date | mm/dd/yyyy |
Reviewer acceptance date | 2010-11-23 |
N° | Page | Section | Observations | Is Addressed? |
1 | 5 | 1.1 | Second paragraph: what's the matter with deliverables here? they are not mentioned anywhere else in the document. -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
2 | 5 | 1.1 | The first part of the third paragraph is a repetition of what already said in the first paragraph and can be removed. -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
3 | 5 | 1.3 | The application area should include also SA1, which is affected because, for example, some types of tests need to be run every time there is a release (minor or revision). -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
4 | 5 | 1.4 | References to deliverables should point to the official pages (either TWiki or CDS), not to JRA1 internal pages, e.g. R7 and R8. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
5 | 6 | 1.6 | Many acronyms and terms used throughout the document are not mentioned in the glossary, e.g. all the names of components. -- FrancescoGiacomini - 04-Nov-2010 The acronyms table has been expanded. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
6 | 7 | 2 | What does the JRA1 objective "Follow and anticipate the needs of the growing infrastructure usage by investigating and adopting technologies to improve scalability, reliability and performance of the grid services." have to do with QC? -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
7 | 7 | 2 | In the paragraph starting with "In order to explore": - "we created a survey" -> "a survey was created" - the last period, starting with "The survey objective", is a repetition -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
8 | 8 | 3 | Use a reference rather than a footnote for the SQAP. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
9 | 8 | 3 | Third paragraph: the assessment of the PTs by the QC task leader must happen only by means of automated procedures and checks. This has to be explicitly and clearly stated. -- FrancescoGiacomini - 04-Nov-2010 Agreed. The above point has been clarified. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
10 | 9 | 3.3 | Fifth bullet: what does "your software" refer to? -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
11 | 9 | The JRA1 development and test plan in context | What do "these points" in the first paragraph refer to? -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
12 | 9 | The JRA1 development and test plan in context | Third paragraph: survey results cannot be started. -- FrancescoGiacomini - 04-Nov-2010 Addressed by changes in the text. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
13 | 10 | 4 | First bullet: the reference is missing. References to Wikipedia should be avoided, given that the content can change. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
14 | 12 | 4.2 | The structure of the document would improve if the title of this and following sections mirrored the categories of the survey. -- FrancescoGiacomini - 04-Nov-2010 The survey results presentation has been restructured considering this and other remarks. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
15 | 13 | 4.3 | Why are ARC and gLite in the same section? -- FrancescoGiacomini - 04-Nov-2010 The survey results presentation has been restructured considering this and other remarks. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
16 | 16 | 4.5 | What is the S2 testsuite. Put at least a reference. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
17 | 16 | 4.5 | I doubt BES is a serious reference standard for job submission. Better use another example. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |
18 | 16 | 4.7 | First paragraph: "ensure" is a bit too strong; moreover it's the task leader that reports, not the task itself. -- FrancescoGiacomini - 04-Nov-2010 Done. -- AndreaCeccanti - 18-Nov-2010 |
![]() |