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: Enol Fernandez Affiliation: CSIC EMI Activity/External project or Institute: JRA1

Review date 05/17/2011
Author(s) revision date 05/17/2011
Reviewer acceptance date mm/dd/yyyy

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

General comments

Document is clearly written, most of my comments are on formatting or typos.

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 2 At the end of the 2nd paragraph I don't understand the sentence: This is in order to report the work that has taken place outside that for direct achievement of the objectives. Outside what? Could you clarify?    
      I see your point... I have changed the last two sentences to "The last sub-section of Section 3 gives an overview of the work performed in the past year of the EMI project within each EMI Security Area product team. The work reported in this last sub-section is work performed that is not directly aimed at the achievement of the objectives in DNA1.3.1. eg. user requests for improvements."    
2 18 4.3.1 Bullet points are missing (shown as asterisks)    
Done. Changed to bullets.
3 18 4.3.1 Last sentence substitute the second and by end    
4 18 Argus integration with HED section is under 4.3.2 that relates to UNICORE, but HED is ARC, isnt it?    
Ah yes... I saw this as well. It's now section 4.3.3
5 19 4.5 For me the section title suggest not just the support of some platforms, but all the changes that will be included in future releases.    
Aha. Yes. I have changed the title to "Work plan for releases on other platforms" I have also added the sentence "The components listed above will be further developed according to user requirements and released on the listed platforms as well as the current SL5/64." in order to clarify that this is not just a static porting exercise.
6 19 4.5 Instead of "All EMI Security..." it should be "all EMI Security"    
7 20 4.7 affected components are all components or just VOMS, Hydra, Argus and UNICORE?    
      The affected components are those services that publish to the information system. In security something like glexec does not do this or security libraries like LCAS etc. These four are the services that may be affected.    
8 23 s/complete d/completed/    
Thanks. Corrected
9 25 4.13 s/Ses/SEs/    
Thanks. That's openoffice/word!
10 27 4.15 s/EMI stack.,/EMI stack,/    
11 7 1.5 Add AuthZ and AuthN to terminology    
Done. I have added AuthN "Common short-hand for Authentication" AuthZ "Common short-hand for Authorization"

Some paragraphs are not justified:

  • whole page 9
  • page 20, section 4.8, first paragraph
  • page 22, section
  • page 22, section 4.9
  • page 23, section 4.10, first paragraph
  • page 24, section
  • page 25, section 4.12
  • page 25, section 4.13
  • whole pages 26 & 27
  • page 27, until section

Yup. Thanks for spotting this. Don't know how that happened. I usually use LaTeX. -- JohnWhite - 17-May-2011

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.

-- AlbertoDiMeglio - 12-Jul-2010

-- FloridaEstrella - 29-Apr-2011

Edit | Attach | Watch | Print version | History: r4 < r3 < r2 < r1 | Backlinks | Raw View | WYSIWYG | More topic actions
Topic revision: r4 - 2011-05-20 - JohnWhite
    • 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-2021 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