Re: [devel] Review request for document OpenSAF_LOGSv_PR.odt Add info about new attribute [#578]

2013-10-04 Thread Mathivanan Naickan Palanivelu
Ack, with the changes suggested. Thanks, Mathi. From: Lennart Lund [mailto:lennart.l...@ericsson.com] Sent: Thursday, September 26, 2013 7:23 PM To: Mathivanan Naickan Palanivelu Cc: Lennart Lund; opensaf-devel@lists.sourceforge.net Subject: Review request for document

Re: [devel] [PATCH 1 of 1] logsv: Fix time formatting for logrecords, @Cd and @CD [#565]

2013-10-04 Thread Mathivanan Naickan Palanivelu
Lennart, Page 27 says the following: @Cd The first three letters (the first one capitalized)of the implied day of the week in English from logTimeStamp of type SaTimeT. Mon And another instance in page 31 (3.1.5.3), where it is referred as a number The default log record format expression for

Re: [devel] [PATCH 0 of 6] Review Request for : pre-review of 2PBE transaction payload handling (#21)

2013-10-04 Thread Anders Bjornerstedt
Ok thanks will look into your comments. That review request was a pre-review request sent in July, mostly to give you and any other reviewers or interested parties time to understand roughly where I was going with this. Improvements and tightening has been done on the 2PBE implementation since

[devel] [PATCH 0 of 1] Review Request for amfd: Stop cluster startup timer, if all configured nodes have joined [#76]

2013-10-04 Thread nagendra . k
Summary: amfd: Stop cluster startup timer, if all configured nodes have joined [#76] Review request for Trac Ticket(s): #76 Peer Reviewer(s): Hans F, Hans N, Praveen, Mathi Pull request to: Affected branch(es): Default Development branch: Default Impacted

[devel] [PATCH 1 of 1] amfd: Stop cluster startup timer, if all configured nodes have joined [#76]

2013-10-04 Thread nagendra . k
osaf/services/saf/amf/amfd/include/proc.h |1 + osaf/services/saf/amf/amfd/ndfsm.cc |4 + osaf/services/saf/amf/amfd/ndproc.cc | 105 ++ 3 files changed, 110 insertions(+), 0 deletions(-) diff --git a/osaf/services/saf/amf/amfd/include/proc.h

Re: [devel] [PATCH 0 of 4] Review Request for amf #574

2013-10-04 Thread Hans Feldt
Could I please have some feedback on this? /Hans On 09/25/2013 04:55 PM, Hans Feldt wrote: Summary: reduce IMM reads by amfnd Review request for Trac Ticket(s): 574 Peer Reviewer(s): Praveen, Nags, Hans N Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE Affected branch(es): all

Re: [devel] [PATCH 1 of 1] Amfd: Donot send empty susi to amfnd [#91]

2013-10-04 Thread Hans Feldt
Ack if you have tested that further admin ops on the SU to make sure it is manageable. Thanks, Hans On 09/24/2013 12:11 PM, nagendr...@oracle.com wrote: osaf/services/saf/amf/amfd/sgproc.cc | 9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git

Re: [devel] [PATCH 0 of 4] Review Request for amf #574

2013-10-04 Thread Anders Björnerstedt
The basic issue as I see it is that the AMF (AMFD or AMFND) must not put itself into a postion so that it must acutely read from the IMM as part of a time critical task (such as failover or switchover). The IMMND can crash and is restartable. While rare, when it does happen, local reads from IMM

Re: [devel] [PATCH 0 of 4] Review Request for amf #574

2013-10-04 Thread Anders Bjornerstedt
Anders Björnerstedt wrote: The basic issue as I see it is that the AMF (AMFD or AMFND) must not put itself into a postion so that it must acutely read from the IMM as part of a time critical task (such as failover or switchover). The IMMND can crash and is restartable. While rare, when it

[devel] [PATCH 1 of 1] uml: Support both TCP and TIPC transports in UML environment [#587]

2013-10-04 Thread Anders Widell
tools/cluster_sim_uml/archive/scripts/40opensaf.rc | 1 + tools/cluster_sim_uml/build_uml| 5 + 2 files changed, 6 insertions(+), 0 deletions(-) Add support for the TCP transport in the UML environment. Select TIPC transport when OpenSAF was built with TIPC enabled.

Re: [devel] [PATCH 0 of 2] Review Request for opensaf:make MDS/TCP as the default opensaf transport[#232]

2013-10-04 Thread Anders Widell
Hi! Ack from me, with comments: The pkill command should be replaced with killall, since killall is included in LSB. pkill and killall work nearly the same, except that killall by default does not take a regular expression as default (though there is an option for this). So the following

[devel] OpenSAF presentation at LinuxCON Europe

2013-10-04 Thread Anders Widell
The OpenSAF project will give a presentation at LinuxCON Europe on October 21. We will present an overview of OpenSAF, the current status of the project and the roadmap for the future. Don't miss out on this presentation if you are attending the LinuxCON! For more information about LinuxCON