Hi Vu Wrong document! I don't think the information about the new global configuration possibilities in OpenSAF should be described in the extension document. It is better to update the overview document. It can be placed as a sub section in the "Tunable OpenSAF configuration parameters" section.
There is also some suggested changes and comments in the attached document Thanks Lennart From: Vu Minh Nguyen [mailto:[email protected]] Sent: den 7 april 2016 09:03 To: 'Mathivanan Naickan Palanivelu'; Lennart Lund Cc: [email protected] Subject: RE: Review Request for log: update PR document for node originator [#1688] Hi all, Thanks for your comments. Please see the update for both docs. Regarding OpenSAF_extensions.doc: 1) Add new chapter "5 Others" 2) Update the cover & Content tables If you have any more comments, please let me know. Otherwise, Lennart could help to push the docs. Regards, Vu. From: Mathivanan Naickan Palanivelu [mailto:[email protected]] Sent: Thursday, April 07, 2016 1:06 PM To: Lennart Lund; Vu Minh Nguyen Cc: [email protected]<mailto:[email protected]> Subject: RE: Review Request for log: update PR document for node originator [#1688] Hi Vu, I have my suggestions in the attached doc. And I agree with Lennart , but in the OpenSAF_extensions.doc and update the doc as suggested by him. Thanks, Mathi. From: Lennart Lund [mailto:[email protected]] Sent: Monday, April 04, 2016 5:07 PM To: Vu Minh Nguyen; Mathivanan Naickan Palanivelu Cc: [email protected]<mailto:[email protected]>; Lennart Lund Subject: RE: Review Request for log: update PR document for node originator [#1688] Hi Vu, Ack for this document (log). Comment: A description of the new OpenSAF configuration object is needed. However it is correct to not have this description in the log document. I suggest that the "OpenSAF_Overview_PR" document is updated with this information. Information to add: * Description of the class e.g. create a table describing the name of the class, describe that a default object of this class is created and the name of this object, all attributes and their default value (so far we have only one attribute but others may be added in the future) Thanks Lennart From: Vu Minh Nguyen [mailto:[email protected]] Sent: den 29 mars 2016 10:47 To: [email protected]<mailto:[email protected]>; Lennart Lund Cc: [email protected]<mailto:[email protected]> Subject: Review Request for log: update PR document for node originator [#1688] Summary: log: update PR document for node originator [#1688] Trac Ticket(s): #1688 Peer Reviewer(s): Lennart, Mathi -------------------------------- Impacted area Impact y/n -------------------------------- Docs y Build system n RPM/packaging n Configuration files n Startup scripts n SAF services n OpenSAF services n Core libraries n Samples n Tests n Other n Comments (indicate scope for each "y" above): --------------------------------------------- Update Programmers Reference with the changes introduced in [#1480] Marked document is attached. Regards, Vu.
OpenSAF_Extensions_PR_1688_Lennart_comments.odt
Description: OpenSAF_Extensions_PR_1688_Lennart_comments.odt
------------------------------------------------------------------------------
_______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
