Re: [devel] [PATCH 1 of 1] amfnd: donot read configuration for mw comp [#369]

[Gary Lee]

Hi Nagu

ack (reviewed & tested)

> On 12 Jul 2016, at 9:13 PM, nagendr...@oracle.com wrote:
> 
> osaf/services/saf/amf/amfnd/imm.cc |  7 ++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
> 
> 
> Amf shouldn't read configuration for ncs component.
> this case is for SU instantiation.
> 
> diff --git a/osaf/services/saf/amf/amfnd/imm.cc 
> b/osaf/services/saf/amf/amfnd/imm.cc
> --- a/osaf/services/saf/amf/amfnd/imm.cc
> +++ b/osaf/services/saf/amf/amfnd/imm.cc
> @@ -111,7 +111,12 @@ void ImmReader::ir_process_event(AVND_EV
>   goto done;
>   }
> 
> - res = avnd_comp_config_get_su(su);
> + if(false == su->is_ncs) {
> + res = avnd_comp_config_get_su(su);
> + } else {
> + res = NCSCC_RC_FAILURE;
> + }
> +
>   {
>   AVND_EVT *evt_ir = 0;
>   TRACE("Sending to main thread.");


--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity planning
reports.http://sdm.link/zohodev2dev
___
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Re: [devel] [PATCH 1 of 1] amfnd: donot read configuration for mw comp [#369]

[Gary Lee]

Hi Nagu

ack (reviewed & tested)

> On 12 Jul 2016, at 9:13 PM, nagendr...@oracle.com wrote:
> 
> osaf/services/saf/amf/amfnd/imm.cc |  7 ++-
> 1 files changed, 6 insertions(+), 1 deletions(-)
> 
> 
> Amf shouldn't read configuration for ncs component.
> this case is for SU instantiation.
> 
> diff --git a/osaf/services/saf/amf/amfnd/imm.cc 
> b/osaf/services/saf/amf/amfnd/imm.cc
> --- a/osaf/services/saf/amf/amfnd/imm.cc
> +++ b/osaf/services/saf/amf/amfnd/imm.cc
> @@ -111,7 +111,12 @@ void ImmReader::ir_process_event(AVND_EV
>   goto done;
>   }
> 
> - res = avnd_comp_config_get_su(su);
> + if(false == su->is_ncs) {
> + res = avnd_comp_config_get_su(su);
> + } else {
> + res = NCSCC_RC_FAILURE;
> + }
> +
>   {
>   AVND_EVT *evt_ir = 0;
>   TRACE("Sending to main thread.");


--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity planning
reports.http://sdm.link/zohodev2dev
___
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

[devel] [PATCH 1 of 1] amfnd: donot read configuration for mw comp [#369]

[nagendra . k]

 osaf/services/saf/amf/amfnd/imm.cc |  7 ++-
 1 files changed, 6 insertions(+), 1 deletions(-)


Amf shouldn't read configuration for ncs component.
this case is for SU instantiation.

diff --git a/osaf/services/saf/amf/amfnd/imm.cc 
b/osaf/services/saf/amf/amfnd/imm.cc
--- a/osaf/services/saf/amf/amfnd/imm.cc
+++ b/osaf/services/saf/amf/amfnd/imm.cc
@@ -111,7 +111,12 @@ void ImmReader::ir_process_event(AVND_EV
goto done;
}
 
-   res = avnd_comp_config_get_su(su);
+   if(false == su->is_ncs) {
+   res = avnd_comp_config_get_su(su);
+   } else {
+   res = NCSCC_RC_FAILURE;
+   }
+
{
AVND_EVT *evt_ir = 0;
TRACE("Sending to main thread.");

--
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity planning
reports.http://sdm.link/zohodev2dev
___
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel