Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Mathivanan Naickan Palanivelu
More comments inline: > -Original Message- > From: praveen malviya > Sent: Tuesday, June 25, 2013 5:14 PM > To: Mathivanan Naickan Palanivelu > Cc: Hans Feldt; Nagendra Kumar; opensaf-devel@lists.sourceforge.net > Subject: Re: [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM >

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Hans Feldt
On 06/25/2013 01:43 PM, praveen malviya wrote: > Thanks for the comments and discussion. I will respond for other > comments soon, as of now starting with this mail. > Please see the response below. > > One more point can be brought into discussion here. The case is when > admin restart operation

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread praveen malviya
On 25-Jun-13 4:24 PM, Hans Feldt wrote: > > On 06/25/2013 11:50 AM, Mathivanan Naickan Palanivelu wrote: >>> -Original Message- >>> From: Hans Feldt [mailto:hans.fe...@ericsson.com] > - I'm sure you would have had a reason to do it this way, but I thought >>> alternatively perfo

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread praveen malviya
Thanks for the comments and discussion. I will respond for other comments soon, as of now starting with this mail. Please see the response below. One more point can be brought into discussion here. The case is when admin restart operation is invoked on the component and saAmfDisableRestart is t

Re: [devel] [PATCH 1 of 1] amfnd: fix admin comp restart to terminate [#7]

2013-06-25 Thread Nagendra Kumar
Ack -Nagu -Original Message- From: Nagendra Kumar Sent: 31 May 2013 17:16 To: Hans Feldt; Praveen Malviya Cc: opensaf-devel@lists.sourceforge.net Subject: Re: [devel] [PATCH 1 of 1] amfnd: fix admin comp restart to terminate [#7] Few early comment: 1. Please change the name of com

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Hans Feldt
On 06/25/2013 11:50 AM, Mathivanan Naickan Palanivelu wrote: >> -Original Message- >> From: Hans Feldt [mailto:hans.fe...@ericsson.com] >>> - I'm sure you would have had a reason to do it this way, but I thought >> alternatively performing the repair at the AMFND itself(instead of notifyi

[devel] Announcement: OpenSAF policy on limits

2013-06-25 Thread Anders Widell
I would like to announce that OpenSAF now has a policy regarding limits: limits should, in general, be possible to configure without recompiling the source code. This means that we would like to move away from using hard-coded limits, and we would like the code to be designed in scalable way so tha

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Mathivanan Naickan Palanivelu
To add on, my comments at a high level, is also to nitpick any possible problems born out of parallelism. -Mathi. > -Original Message- > From: Mathivanan Naickan Palanivelu > Sent: Tuesday, June 25, 2013 3:20 PM > To: Hans Feldt > Cc: opensaf-devel@lists.sourceforge.net > Subject: Re: [d

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Mathivanan Naickan Palanivelu
> -Original Message- > From: Hans Feldt [mailto:hans.fe...@ericsson.com] > Sent: Tuesday, June 25, 2013 3:01 PM > To: Mathivanan Naickan Palanivelu > Cc: Praveen Malviya; Nagendra Kumar; opensaf-devel@lists.sourceforge.net > Subject: Re: [PATCH 6 of 6] amf: handle sufailover in SU FSM and C

Re: [devel] [PATCH 6 of 6] amf: handle sufailover in SU FSM and Comp FSM at amfnd [#98]

2013-06-25 Thread Hans Feldt
On 06/24/2013 04:39 PM, Mathivanan Naickan Palanivelu wrote: > Hi Praveen, > > Thanks for the clarification. > Please find some other comments below: > > - I'm sure you would have had a reason to do it this way, but I thought > alternatively performing the repair at the AMFND itself(instead of no