SA1/JRA1 Deliverable Review Form

Identification of the deliverable or milestone
Project: EMI Deliverable or milestone identifier: D5.3.2
Title: DJRA1.3.2 Security Area Work Plan and Status Report Doc. identifier: EMI-DJRA1.3.2_v0.3.odt
Author(s): John White Due date: __

Identification of the reviewer
Name: Andre Giesler Affiliation: JUELICH EMI Activity/External project or Institute: SA1/JRA1

Review date 05/03/2011 (Section 3) and 05/20/2011
Author(s) revision date 05/04/2011 (Section 3)
Reviewer acceptance date mm/dd/yyyy

Attach the reviewed document to the deliverable page, put here a link

General comments

The reviewed chapter 3 is structured very clearly and can also be read very fluently. All in all, there are almost no comments from my side.

It is clearly outlined in the workplan who has to do what, until when , and which risks can influent the result. I have noted only some spelling and a few other comments in my review. If my comments (almost only formatting and typos) are adressed I would accept the document.

Additional recommendations (not affecting the document content, e.g. recommendation for future work)

Detailed comments on the content

Note 1: The reviewers must list here any observation they want to track explicitly and that require interaction with the authors
Alternatively all changes must be listed in the document itself using Word change tracking features (if you use Word)
Note 2: These comments have to be explicitly addressed by the authors and the action taken must be clearly described

N Page Section Observations and Replies Is Addressed?
1 9 3.3 Missing bullet at The API definitions in C/C++
-- AndreGiesler - 02-May-20111
Added. -- JohnWhite - 04-May-2011
2 10 3.5 The planning of the replacement of Unicore Uvos by VOMS-SAML is mentioned as an objective both in chapter 3.5 (Common attribute authority) and 3.6 (Reduction in components). Could be reduced to be mentioned in only one chapter.
-- AndreGiesler - 02-May-20111
I have added the text to 3.6 "(as mentioned above as part of the Common Attribute Authority objective)" to point out the fact we realise this is reported in two places. -- JohnWhite - 04-May-2011
3     As the work in the area goes this UVOS->VOMS gets looked at both from the point of view of a technical advance (common SAML profile everywhere) and a more "political" decision to reduce components to make the EU feel happy. Is this OK with you? -- JohnWhite - 04-May-2011 yes

Review of sections 1,2,4,5

N Page Section Observations and Replies Is Addressed?
1 9 2 Maybe I'm overcorrect here, but the sentence 'Also the agreement on common SAML...' doesn't contain a verb
-- AndreGiesler - 19-May-20111
2 9 2 Missing full stop at 'objectives in DNA1.3.1. eg. user ' ... e.g. user ...
-- AndreGiesler - 19-May-20111
3 9 2 Two spaces at 'for a common set '
-- AndreGiesler - 19-May-20111
4 16 4 Two spaces at 'objectives, giving '
-- AndreGiesler - 19-May-20111
4 17 4.3 Spelling: dicsussions
-- AndreGiesler - 19-May-20111
4 18 4.3.3 A minor issue, but since ARGUS integration in A-REX is mentioned in 4.3.3, A-REX could be also mentioned in introducing paragraph 4.3 (only HED is there mentioned)
-- AndreGiesler - 19-May-20111
4 20 4.8 Correct the double quotes format at to access a ``Grid''.
-- AndreGiesler - 19-May-20111
4 20 4.8 Two spaces at by most Grid infrastructures
-- AndreGiesler - 19-May-20111
4 20 4.8 Correct sentence: there are still identity federation requirement-gathering exercises still ongoing.
-- AndreGiesler - 19-May-20111
4 21 As far as I know, the URC is not part of EMI and therefore not known in that document. I think only Hila and UCC should be mentioned here. "The development of a STS/AAI client-side library for UNICORE, useful for both HiLA, UCC and URC."
-- AndreGiesler - 19-May-20111
4 22 Correct order "It is not defined yet the if C++ library will be implemented "
-- AndreGiesler - 19-May-20111
4 23 4.10 Something is missing here "planned and delivered by EMI the SAML group project month 4 "
-- AndreGiesler - 19-May-20111
4 25 4.13 The risk... "The risks associated to the tasks in this objective is that "
-- AndreGiesler - 19-May-20111
4 25 refer.. "Resources refers "
-- AndreGiesler - 19-May-20111

x xx xxx Minor commont: In general, the space between the paragraphs could be checked again. Between the paragraphs in one chapter are sometimes two lines(e.g. in 4.8), and between two chapters is one line (4.8. and 4.8.1)
-- AndreGiesler - 19-May-20111

Any other modification, spelling or grammatical corrections, etc must be done directly in the document using tracked changes or similar mechanisms that allows the authors to identify which correction is suggested.

Note 3: Hello Andre, I am just letting you know that I went through section 3 again myself and picked up a few errors and missing references. I haven't touched any parts corrected by you. I have put these into v0.5. No need to review yet until Morris has made the Gantt chart.I've also done all the other stuff like Exec Summary, Abstract, Conclusions etc. Cheers, John.

-- AlbertoDiMeglio - 12-Jul-2010

-- FloridaEstrella - 29-Apr-2011

Edit | Attach | Watch | Print version | History: r14 < r13 < r12 < r11 < r10 | Backlinks | Raw View | Raw edit | More topic actions...
Topic revision: r11 - 2011-05-20 - AGieslerExternal
    • Cern Search Icon Cern Search
    • TWiki Search Icon TWiki Search
    • Google Search Icon Google Search

    EMI All webs login

This site is powered by the TWiki collaboration platform Powered by PerlCopyright & 2008-2022 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
or Ideas, requests, problems regarding TWiki? use Discourse or Send feedback