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) -5/20/2011 (Sections 1,2,4,5) |
Reviewer acceptance date | mm/dd/yyyy |
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 |
yes |
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 |
yes |
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 |
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 |
yes | |
I have changed the sentence to "Other technical highlights are the agreements on common EMI profiles for SAML and XACML and the common authentication library APIs." -- JohnWhite - 20-May-2011 | |||||
2 | 9 | 2 | Missing full stop at 'objectives in DNA1.3.1. eg. user ' ... e.g. user ... -- AndreGiesler - 19-May-20111 |
yes | |
Corrected. Also picked up about four more! | |||||
3 | 9 | 2 | Two spaces at 'for a common set ' -- AndreGiesler - 19-May-20111 |
yes | |
Corrected | |||||
4 | 16 | 4 | Two spaces at 'objectives, giving ' -- AndreGiesler - 19-May-20111 |
yes | |
Corrected | |||||
4 | 17 | 4.3 | Spelling: dicsussions -- AndreGiesler - 19-May-20111 |
yes | |
Corrected | |||||
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 |
yes | |
Yup. Changed the sentence in 4.3 to end as: "The affected components are Argus , UNICORE XACML entity (UNICORE PDP), ARC HED and A-REX." I have added A-REX to the terminology table as well. | |||||
4 | 20 | 4.8 | Correct the double quotes format at to access a ``Grid''”. -- AndreGiesler - 19-May-20111 |
yes | |
I have changed "Grid" to 'Grid'. I need to quote the objectives verbatim in order to make clear these are not my words. We based our plans on the literal statements of these objectives as interpretation and changes only leads to trouble. | |||||
4 | 20 | 4.8 | Two spaces at by most Grid infrastructures -- AndreGiesler - 19-May-20111 |
yes | |
Corrected | |||||
4 | 20 | 4.8 | Correct sentence: there are still identity federation requirement-gathering exercises still ongoing. -- AndreGiesler - 19-May-20111 |
||
4 | 21 | 4.8.2.2 | 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 |
yes | |
I will confirm with Krzysztof Benedyczak, will remove URC. Will add definitions for Hila and UCC to table. -- JohnWhite - 20-May-2011 | |||||
In fact URC gets mentioned in 4.10.2.2 as well. Have corrected the sentences there to: "The development of a STS/AAI client-side library for UNICORE, useful for both HiLA and UCC (URC will use this library as well, but outside the EMI project)." -- JohnWhite - 20-May-2011 | |||||
4 | 22 | 4.9.3.1 | Correct order "It is not defined yet the if C++ library will be implemented " -- AndreGiesler - 19-May-20111 |
yes | |
Yup. Bad sentence. I have changed it to "Presently, it is not decided if the C++ library will be implemented on top of the C version or will use an independent code-base." | |||||
4 | 23 | 4.10 | Something is missing here "planned and delivered by EMI the SAML group project month 4 " -- AndreGiesler - 19-May-20111 |
yes | |
Yup, you're right. I have changed the sentence to "In order to achieve this target, an agreement on common set of SAML authorization attributes was planned and delivered by EMI the SAML group in the EMI PM4." | |||||
4 | 25 | 4.13 | The risk... "The risks associated to the tasks in this objective is that " -- AndreGiesler - 19-May-20111 |
yes | |
Corrected | |||||
4 | 25 | 4.18.1.2 | refer.. "Resources refers " -- AndreGiesler - 19-May-20111 |
yes | |
I have changed the start of the sentence to "In this context, “resources” refer to:" |
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 |
||
Yeah... another effect of using crappy typesetting software. I will remove all spaces manually between headings and paragraph ones -- JohnWhite - 20-May-2011 | |||||
OK... mega-amounts of clickings and I think I've removed all the spaces. It's better now but not LaTeX. |