Hi Thuan,

       I can't seem to reproduce this. Can you tell me the exact steps for
   how you are setting it up?

   Alex

   On 09/27/2018 06:55 AM, Tran Thuan wrote:
     __________________________________________________________________

   NOTICE: This email was received from an EXTERNAL sender
     __________________________________________________________________

   Hi Alex,
   When I try admin restart component, it got error and state is stuck in
   RESTARTING.
   Please check.
   root@SC-1:/# amf-adm restart
   safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   2018-09-27 17:52:51.808 SC-1 osafamfnd[515]: NO Admin restart requested
   for
   'safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_2N'
   2018-09-27 17:52:51.810 SC-1 osafamfnd[515]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' Presence State
   INSTANTIATED => RESTARTING
   2018-09-27 17:52:51.811 SC-1 amf_container_demo[1074]: =====Contained
   Terminate Callback====>
   2018-09-27 17:52:51.811 SC-1 amf_container_demo[1074]:
   comp:'safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained
   _2N'
   2018-09-27 17:52:51.815 SC-1 amf_container_demo[1074]:
   <=====================================
   2018-09-27 17:52:51.815 SC-1 amf_container_demo[1074]: =====Contained
   Instantiate Callback====>
   2018-09-27 17:52:51.815 SC-1 amf_container_demo[1074]:
   comp:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_
   2N
   2018-09-27 17:52:51.817 SC-1 amf_container_demo[1074]:
   <===========================================
   2018-09-27 17:52:51.817 SC-1 osafamfnd[515]: ER nodeid_mdsdest rec
   doesn't exist, Rec get failed: NodeId:0
   2018-09-27 17:52:51.817 SC-1 osafamfnd[515]: ER
   avnd_comp_cbq_rec_send:Msg Send to AvND
   Failed:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Containe
   d_2N, 0
   root@SC-1:/# amf-state su | grep -A 4 SU1.*Contain
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   saAmfSUAdminState=UNLOCKED(1)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=RESTARTING(5)
   saAmfSUReadinessState=IN-SERVICE(2)
   --
   safSu=SU1,safSg=Container,safApp=Container
   saAmfSUAdminState=UNLOCKED(1)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=INSTANTIATED(3)
   saAmfSUReadinessState=IN-SERVICE(2)
   Best Regards,
   Thuan
   -----Original Message-----
   From: Tran Thuan [1]<thuan.t...@dektech.com.au>
   Sent: Wednesday, September 26, 2018 6:12 PM
   To: 'Jones, Alex' [2]<ajo...@rbbn.com>; [3]nagen...@hasolutions.in;
   'Gary Lee' [4]<gary....@dektech.com.au>;
   [5]hans.nordeb...@ericsson.com; [6]ravisekhar.ko...@oracle.com
   Cc: [7]opensaf-devel@lists.sourceforge.net
   Subject: Re: [devel] [PATCH 1/1] amf: add support for
   container/contained [#70]
   Hi Alex,
   I can fetch your code. Will try it and reply if any concern...
   * 430dbfdd4 - (HEAD -> ticket-70) amf: fix crash of amfd [#70] (5 days
   ago) <Alex Jones>
   * bf998b543 - amfd: don't start contained if it is in locked-in state
   [#70] (3 weeks ago) <Alex Jones>
   * 8b8f2fe30 - amf: fix restart of container app and check container
   model [#70] (6 weeks ago) <Alex Jones>
   * 9c9f7e04c - amf: add support for container/contained [#70] (6 weeks
   ago) <Alex Jones>
   * cf9d75653 - amf: add support for container/contained [#70] (6 weeks
   ago) <Alex Jones>
   * 199d81e0e - amf: add support for container/contained [#70] (6 weeks
   ago) <Alex Jones>
   * d53a063d3 - amfnd: add support for container/contained [#70] (6 weeks
   ago) <Alex Jones>
   * 308b8d733 - amfd: add support for container/contained [#70] (6 weeks
   ago) <Alex Jones>
   Best Regards,
   Thuan
   From: Jones, Alex [8]<ajo...@rbbn.com>
   Sent: Wednesday, September 26, 2018 12:10 AM
   To: Tran Thuan [9]<thuan.t...@dektech.com.au>;
   [10]nagen...@hasolutions.in; 'Gary Lee' [11]<gary....@dektech.com.au>;
   [12]hans.nordeb...@ericsson.com; [13]ravisekhar.ko...@oracle.com
   Cc: [14]opensaf-devel@lists.sourceforge.net
   Subject: Re: [devel] [PATCH 1/1] amf: add support for
   container/contained [#70]
   Hi Thuan,
   I pushed the latest ticket-70 to my review repo.
   For these concerns you have try cleaning out your code, and pull the
   latest from my review repo for ticket-70. Contained should not get
   instantiated until you unlock-in the contained su. I'm guessing you are
   missing some patches...
   Let me know if you still see these issues.
   Alex
   On 09/24/2018 12:24 AM, Tran Thuan wrote:
   _____
   NOTICE: This email was received from an EXTERNAL sender
   _____
   Hi Alex,
   Can you send out new version of review?
   Then I can fetch latest version from your review repo.
   Some more concerns without using your additional patch plm-70 yet.
   1. After unlock-in, unlock container, contained got instantiated but
   admin state is still locked-in
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   saAmfSUAdminState=LOCKED-INSTANTIATION(3)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=INSTANTIATED(3)
   saAmfSUReadinessState=OUT-OF-SERVICE(1)
   --
   safSu=SU1,safSg=Container,safApp=Container
   saAmfSUAdminState=UNLOCKED(1)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=INSTANTIATED(3)
   saAmfSUReadinessState=IN-SERVICE(2)
   2. Also, I don't see how to make contained get assignment?
   root@SC-1:~# amf-adm unlock
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   error - saImmOmAdminOperationInvoke_2 admin-op RETURNED:
   SA_AIS_ERR_BAD_OPERATION (20)
   error-string: State transition invalid, state 3, op 1
   root@SC-1:~# amf-adm unlock-in
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   error - saImmOmAdminOperationInvoke_2 admin-op RETURNED:
   SA_AIS_ERR_BAD_OPERATION (20)
   error-string: Can't instantiate
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N', whose presence
   state is '3'
   3. When I kill -9 `pidof amf_container_demo`, it seems recovery fail?
   2018-09-24 10:59:59.273 SC-1 osafamfnd[478]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' component restart
   probation timer started (timeout: 4000000000 ns)
   2018-09-24 10:59:59.273 SC-1 osafamfnd[478]: NO Restarting a component
   of 'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' (comp restart
   count: 1)
   2018-09-24 10:59:59.273 SC-1 osafamfnd[478]: NO
   'safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_2N'
   faulted due to 'avaDown' : Recovery is 'componentRestart'
   2018-09-24 10:59:59.274 SC-1 osafamfnd[478]: ER ncsmds_api for 0
   FAILED, dest=2010f00000280
   2018-09-24 10:59:59.274 SC-1 osafamfnd[478]: NO Component CLC fsm
   exited with error for
   comp:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_
   2N
   2018-09-24 11:00:03.295 SC-1 osafamfnd[478]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' Component or SU
   restart probation timer expired
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   saAmfSUAdminState=LOCKED-INSTANTIATION(3)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=INSTANTIATED(3)
   saAmfSUReadinessState=OUT-OF-SERVICE(1)
   --
   safSu=SU1,safSg=Container,safApp=Container
   saAmfSUAdminState=UNLOCKED(1)
   saAmfSUOperState=ENABLED(1)
   saAmfSUPresenceState=INSTANTIATED(3)
   saAmfSUReadinessState=IN-SERVICE(2)
   safSi=Contained_2N_1,safApp=Contained_2N
   saAmfSIAdminState=UNLOCKED(1)
   saAmfSIAssignmentState=UNASSIGNED(1)
   safSi=Container,safApp=Container
   saAmfSIAdminState=UNLOCKED(1)
   saAmfSIAssignmentState=PARTIALLY_ASSIGNED(3)
   Best Regards,
   Thuan
   From: Jones, Alex [15]<mailto:ajo...@rbbn.com> [16]<ajo...@rbbn.com>
   Sent: Saturday, September 22, 2018 1:01 AM
   To: Tran Thuan [17]<mailto:thuan.t...@dektech.com.au>
   [18]<thuan.t...@dektech.com.au>; [19]nagen...@hasolutions.in
   [20]<mailto:nagen...@hasolutions.in> ; 'Gary Lee'
   [21]<mailto:gary....@dektech.com.au> [22]<gary....@dektech.com.au>;
   [23]hans.nordeb...@ericsson.com
   [24]<mailto:hans.nordeb...@ericsson.com> ;
   [25]ravisekhar.ko...@oracle.com
   [26]<mailto:ravisekhar.ko...@oracle.com>
   Cc: [27]opensaf-devel@lists.sourceforge.net
   [28]<mailto:opensaf-devel@lists.sourceforge.net>
   Subject: Re: [devel] [PATCH 1/1] amf: add support for
   container/contained [#70]
   Hi Tran,
   Here is a patch you can overlay to fix this crash.
   Alex
   On 09/21/2018 06:31 AM, Tran Thuan wrote:
   _____
   NOTICE: This email was received from an EXTERNAL sender
   _____
   Hi Alex,
   I think you need update samples/amf/container/README.
   Also when I try following steps, AMFD crash.
   root@SC-1:/opt/amf_demo# immcfg -f AppConfig-container.xml
   root@SC-1:/opt/amf_demo# immcfg -f AppConfig-contained-2N.xml
   root@SC-1:/opt/amf_demo# amf-adm unlock-in
   safSu=SU1,safSg=Container,safApp=Container
   root@SC-1:/opt/amf_demo# amf-adm unlock
   safSu=SU1,safSg=Container,safApp=Container
   root@SC-1:/opt/amf_demo# amf-adm lock
   safSu=SU1,safSg=Container,safApp=Container
   2018-09-21 16:58:38.338 SC-1 osafamfnd[512]: NO Assigning
   'safSi=Container,safApp=Container' ACTIVE to
   'safSu=SU1,safSg=Container,safApp=Container'
   2018-09-21 16:58:38.339 SC-1 amf_container_demo[739]: csi set callback
   for
   comp: safComp=Container,safSu=SU1,safSg=Container,safApp=Container
   2018-09-21 16:58:38.339 SC-1 amf_container_demo[739]: CSI Set - add
   'safCsi=Container1,safSi=Container,safApp=Container' HAState Active
   2018-09-21 16:58:38.339 SC-1 amf_container_demo[739]: name: Contained1,
   value: AAAA
   2018-09-21 16:58:38.339 SC-1 amf_container_demo[739]: name: Contained1,
   value: BBBB
   2018-09-21 16:58:38.340 SC-1 osafamfnd[512]: NO Assigned
   'safSi=Container,safApp=Container' ACTIVE to
   'safSu=SU1,safSg=Container,safApp=Container'
   2018-09-21 16:58:38.413 SC-1 osafamfnd[512]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' Presence State
   UNINSTANTIATED => INSTANTIATING
   2018-09-21 16:58:38.414 SC-1 amf_container_demo[739]: =====Contained
   Instantiate Callback====>
   2018-09-21 16:58:38.414 SC-1 amf_container_demo[739]:
   comp:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_
   2N
   2018-09-21 16:58:38.414 SC-1 amf_container_demo[739]: responding with
   TRY_AGAIN
   2018-09-21 16:58:38.417 SC-1 amf_container_demo[739]:
   <===========================================
   2018-09-21 16:58:38.418 SC-1 amf_container_demo[739]: =====Contained
   Clean Up Callback====>
   2018-09-21 16:58:38.418 SC-1 amf_container_demo[739]:
   comp:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_
   2N
   2018-09-21 16:58:38.418 SC-1 amf_container_demo[739]:
   <===========================================
   2018-09-21 16:58:38.422 SC-1 amf_container_demo[739]: =====Contained
   Instantiate Callback====>
   2018-09-21 16:58:38.422 SC-1 amf_container_demo[739]:
   comp:safComp=Contained_1,safSu=SU1,safSg=Contained_2N,safApp=Contained_
   2N
   2018-09-21 16:58:38.422 SC-1 amf_container_demo[739]:
   <===========================================
   2018-09-21 16:58:38.424 SC-1 osafamfnd[512]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' Presence State
   INSTANTIATING => INSTANTIATED
   2018-09-21 17:19:07.776 SC-1 osafamfnd[512]: ER AMFD has unexpectedly
   crashed. Rebooting node
   2018-09-21 17:19:07.776 SC-1 osafamfnd[512]: Rebooting OpenSAF NodeId =
   131343 EE Name = , Reason: AMFD has unexpectedly crashed. Rebooting
   node, OwnNodeId = 131343, SupervisionTime = 60
   2018-09-21 17:19:07.778 SC-1 osafimmnd[438]: NO Implementer locally
   disconnected. Marking it as doomed 5 <23, 2010f> (safAmfService)
   2018-09-21 17:19:07.799 SC-1 opensaf_reboot: Rebooting local node;
   timeout=60
   Best Regards,
   Thuan
   -----Original Message-----
   From: [29]nagen...@hasolutions.in [30]<mailto:nagen...@hasolutions.in>
   [31]<mailto:nagen...@hasolutions.in> [32]<nagen...@hasolutions.in>
   Sent: Friday, September 7, 2018 12:57 PM
   To: Alex Jones [33]<mailto:ajo...@rbbn.com> [34]<ajo...@rbbn.com>; Gary
   Lee [35]<mailto:gary....@dektech.com.au> [36]<gary....@dektech.com.au>;
   [37]hans.nordeb...@ericsson.com
   [38]<mailto:hans.nordeb...@ericsson.com> ;
   [39]ravisekhar.ko...@oracle.com
   [40]<mailto:ravisekhar.ko...@oracle.com>
   Cc: [41]opensaf-devel@lists.sourceforge.net
   [42]<mailto:opensaf-devel@lists.sourceforge.net>
   Subject: Re: [devel] [PATCH 1/1] amf: add support for
   container/contained [#70]
   Hi Alex,
   Thanks for the patch.
   From my side Ack. I wish that I could have tested the following area (I
   assume you would have covered it):
   - Headless enabled test cases
   - CSI Dep, SI Dep testing(in 2N red model)
   - Combinations of Admin operations on Container and contained (in all 5
   red models for contained) with fault scenarios.
   - Escalations of contained components.
   Thanks,
   Nagendra, 91-9866424860
   High Availability Solutions Pvt. Ltd. ([43]www.hasolutions.in
   <[44]https://protect-us.mimecast.com/s/EO45CQWOyWhZZ5Wcx7jbh?domain=has
   olutions.in> )
   - OpenSAF Support and Services
   --------- Original Message --------- Subject: Re: [PATCH 1/1] amf: add
   support for container/contained [#70]
   From: "Alex Jones" [45]<mailto:ajo...@rbbn.com> [46]<ajo...@rbbn.com>
   Date: 9/6/18 8:52 pm
   To: [47]nagen...@hasolutions.in [48]<mailto:nagen...@hasolutions.in> ,
   "Gary Lee" [49]<mailto:gary....@dektech.com.au>
   [50]<gary....@dektech.com.au>, [51]hans.nordeb...@ericsson.com
   [52]<mailto:hans.nordeb...@ericsson.com> ,
   [53]ravisekhar.ko...@oracle.com
   [54]<mailto:ravisekhar.ko...@oracle.com>
   Cc: [55]opensaf-devel@lists.sourceforge.net
   [56]<mailto:opensaf-devel@lists.sourceforge.net>
   Hi Nagu,
   Here's a patch that fixes your issue in test #1.
   For the other code review issues, is it OK if I just add them when I
   push the final patch. Or do you want to review them now?
   Alex
   On 08/30/2018 01:44 AM, [57]nagen...@hasolutions.in
   [58]<mailto:nagen...@hasolutions.in> wrote:
   NOTICE: This email was received from an EXTERNAL sender
   Hi Alex,
   Thanks for your response.
   For Test #2, I had configured all SUs on the single node SC-1. So, 2
   container SUs and 2 contained SUs are on the same node. In such cases,
   we can have the implementation as having only one SU of that
   node(higher rank SUs may be) to be the container for all the contained
   SUs of that node.
   Thanks,
   Nagendra, 91-9866424860
   High Availability Solutions Pvt. Ltd. ([59]www.hasolutions.in
   <[60]https://protect-us.mimecast.com/s/EO45CQWOyWhZZ5Wcx7jbh?domain=has
   olutions.in> )
   - OpenSAF Support and Services
   --------- Original Message --------- Subject: Re: [PATCH 1/1] amf: add
   support for container/contained [#70]
   From: "Alex Jones" [61]<mailto:ajo...@rbbn.com> [62]<ajo...@rbbn.com>
   Date: 8/29/18 9:29 pm
   To: [63]nagen...@hasolutions.in [64]<mailto:nagen...@hasolutions.in> ,
   "Gary Lee" [65]<mailto:gary....@dektech.com.au>
   [66]<gary....@dektech.com.au>, [67]hans.nordeb...@ericsson.com
   [68]<mailto:hans.nordeb...@ericsson.com> ,
   [69]ravisekhar.ko...@oracle.com
   Cc: [70]opensaf-devel@lists.sourceforge.net
   [71]<mailto:opensaf-devel@lists.sourceforge.net>
   Hi Nagu,
   I have a fix for your issue test #1. I will send out a patch along with
   changes for code review #1 and #2.
   For issue test #2, I think this needs to be handled in the
   configuration. In this case because there is no explicit node set for
   the contained SUs, su.cc:map_su_to_node will assign a node in the node
   group.
   The code is assigning it to SC-2 in this case, because another SU has
   been assigned to SC-1, even though there is no container on SC-2. I'm
   not sure how we can get around this without explicitly setting the
   contained host node in the configuration. Since the container csi has
   not yet been assigned, we can't map it to a container, and so we can't
   figure out which container we should be on the same node as. Am I right
   here?
   Alex
   On 08/28/2018 09:56 AM, [72]nagen...@hasolutions.in
   [73]<mailto:nagen...@hasolutions.in> wrote:
   NOTICE: This email was received from an EXTERNAL sender
   Hi Alex,
   Code review:
   1. Header for few functions are missing.
   2. Clc.cc: Need to add '0' in place avnd_comp_clc_inst_try_again_hdler
   in other fsm states.
   Testing:
   1. Uploaded AppConfig-container.xml and AppConfig-contained-2N.xml
   Performed:
   amf-adm unlock-in safSu=SU1,safSg=Container,safApp=Container
   amf-adm unlock safSu=SU1,safSg=Container,safApp=Container
   Even I don't perform the following, the contained components are
   instantiated.
   amf-adm unlock-in safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   amf-adm unlock safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   Aug 28 19:15:11 nags-VirtualBox osafamfnd[28278]: NO
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N' Presence State
   UNINSTANTIATED => INSTANTIATING
   immlist safSu=SU1,safSg=Contained_2N,safApp=Contained_2N will show
   saAmfSUPresenceState 3(instantiated) and saAmfSUAdminState 3(locked-in)
   Now further admin operation on
   safSu=SU1,safSg=Contained_2N,safApp=Contained_2N will fail:
   root@nags-VirtualBox:/home/nags/views/ajones-review/samples/amf/contain
   er#
   amf-adm unlock-in safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   error - saImmOmAdminOperationInvoke_2 admin-op RETURNED:
   SA_AIS_ERR_BAD_OPERATION (20)
   error-string: Can't instantiate
   'safSu=SU1,safSg=Contained_2N,safApp=Contained_2N', whose presence
   state is '3'
   2.This is related to Specs 6.2.2 Assignment of the Container CSI: "If
   there are multiple container components on a node which have the active
   HA state for a particular container CSI, and one or more service units
   on the same node whose contained components are configured with the
   same container CSI, it is implementation- defined how the Availability
   Management Framework selects container components to handle the life
   cycle of the contained components of these service units.
   However, all contained components of a service unit must have the same
   associated container component."
   Uploaded AppConfig-container.xml and AppConfig-contained-2N.xml with
   once difference that all SUs of container and contained are configured
   on SC-1.
   Perform the following operations, but
   safSu=SU2,safSg=Contained_2N,safApp=Contained_2N will not get
   assignments.
   amf-adm unlock-in safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   amf-adm unlock safSu=SU1,safSg=Contained_2N,safApp=Contained_2N
   amf-adm unlock-in safSu=SU2,safSg=Contained_2N,safApp=Contained_2N
   amf-adm unlock safSu=SU2,safSg=Contained_2N,safApp=Contained_2N
   amf-adm unlock-in safSu=SU1,safSg=Container,safApp=Container
   amf-adm unlock safSu=SU1,safSg=Container,safApp=Container
   amf-adm unlock-in safSu=SU2,safSg=Container,safApp=Container
   amf-adm unlock safSu=SU2,safSg=Container,safApp=Container
   root@nags-VirtualBox:/home/nags/views/ajones-review/samples/amf/contain
   er#
   amf-state siass
   safSISU=safSu=SC-1\,safSg=NoRed\,safApp=OpenSAF,safSi=NoRed1,safApp=Ope
   nSAF
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   safSISU=safSu=SC-1\,safSg=2N\,safApp=OpenSAF,safSi=SC-2N,safApp=OpenSAF
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   safSISU=safSu=SU1\,safSg=Contained_2N\,safApp=Contained_2N,safSi=Contai
   ned_2
   N_1,safApp=Contained_2N
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   safSISU=safSu=SU1\,safSg=Container\,safApp=Container,safSi=Container,sa
   fApp=
   Container
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   safSISU=safSu=SU2\,safSg=Container\,safApp=Container,safSi=Container,sa
   fApp=
   Container
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   I will do further testing.
   The documentation need to be done if you haven't tested :
   - Headless enabled
   - CSI Dep, SI Dep testimg
   - Etc.
   Thanks,
   Nagendra, 91-9866424860
   High Availability Solutions Pvt. Ltd. ([74]www.hasolutions.in
   <[75]https://protect-us.mimecast.com/s/EO45CQWOyWhZZ5Wcx7jbh?domain=has
   olutions.in> )
   - OpenSAF Support and Services
   --------- Original Message --------- Subject: Re: [PATCH 1/1] amf: add
   support for container/contained [#70]
   From: "Alex Jones" [76]<mailto:ajo...@rbbn.com> [77]<ajo...@rbbn.com>
   Date: 8/15/18 11:10 pm
   To: "Gary Lee" [78]<mailto:gary....@dektech.com.au>
   [79]<gary....@dektech.com.au>, [80]hans.nordeb...@ericsson.com
   [81]<mailto:hans.nordeb...@ericsson.com> ,
   [82]ravisekhar.ko...@oracle.com
   [83]<mailto:ravisekhar.ko...@oracle.com> , [84]nagen...@hasolutions.in
   [85]<mailto:nagen...@hasolutions.in>
   Cc: [86]opensaf-devel@lists.sourceforge.net
   [87]<mailto:opensaf-devel@lists.sourceforge.net>
   G'day Gary,
   I see you were adding the XML file dynamically with "immcfg -f". I
   hadn't tried that. I hadn't tried killing the sample app, either.
   Here is a patch that should fix both issues. Apply it on top of the
   latest big one I sent.
   Alex
   On 08/13/2018 10:37 PM, Gary Lee wrote:
   NOTICE: This email was received from an EXTERNAL sender
   Hi Alex
   I modified AppConfig-container.xml and changed saAmfSgtRedundancyModel
   from
   4 (NwayAct) to 1 (2N).
   The xml still loads and I could unlock, resulting in:
   root@SC-1:/var/log# immlist safVersion=1,safSgType=Container Name Type
   Value(s)
   =======================================================================
   =
   safVersion SA_STRING_T
   safVersion=1
   saAmfSgtValidSuTypes SA_NAME_T
   safVersion=1,safSuType=Container (32)
   saAmfSgtRedundancyModel SA_UINT32_T 1 (0x1)
   safSISU=safSu=SU2\,safSg=Container\,safApp=Container,safSi=Container,sa
   fApp=
   Container
   saAmfSISUHAState=STANDBY(2)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   safSISU=safSu=SU1\,safSg=Container\,safApp=Container,safSi=Container,sa
   fApp=
   Container
   saAmfSISUHAState=ACTIVE(1)
   saAmfSISUHAReadinessState=READY_FOR_ASSIGNMENT(1)
   Also, have you tried killing the amf_container_demo binary?
   Thanks
   Gary
   On 14/08/18 05:00, Alex Jones wrote:
   Hi Gary,
   I just resubmitted a new patch which breaks out the different
   components, and addresses the other comments here. But, #2 (rejecting
   all but NWay-active for container) should already be in there. Is there
   a specific test you ran that didn't work?
   Alex
   On 08/13/2018 02:43 AM, Gary Lee wrote:
   NOTICE: This email was received from an EXTERNAL sender
   Hi Alex
   Some initial comments:
   0. Is it possible to split up the patch into amfd / amfnd / common /
   samples. Just makes it easier to reply inline.
   1. Please compile the container demo by default, and make
   amf_container_script world executable.
   Eg.
   diff --git a/samples/amf/Makefile.am b/samples/amf/Makefile.am index
   447dedd..7ebf9c3 100644
   --- a/samples/amf/Makefile.am
   +++ b/samples/amf/Makefile.am
   @@ -19,5 +19,5 @@ include $(top_srcdir)/Makefile.common
   MAINTAINERCLEANFILES = Makefile.in
   -SUBDIRS = sa_aware non_sa_aware wrapper proxy api_demo +SUBDIRS =
   sa_aware non_sa_aware wrapper proxy api_demo container
   diff --git a/samples/amf/container/amf_container_script
   b/samples/amf/container/amf_container_script
   old mode 100644
   new mode 100755
   diff --git a/samples/configure.ac b/samples/configure.ac index
   7cf803e..9765d54 100644
   --- a/samples/configure.ac
   +++ b/samples/configure.ac
   @@ -67,6 +67,7 @@ AC_CONFIG_FILES([ \
   amf/wrapper/Makefile \
   amf/proxy/Makefile \
   amf/api_demo/Makefile \
   + amf/container/Makefile \
   cpsv/Makefile \
   cpsv/ckpt_demo/Makefile \
   cpsv/ckpt_track_demo/Makefile \
   2. We should probably reject CCBs that set saAmfSgtRedundancyModel to
   anything other than NWayActive, for Containers.
   3. Do we need to bump the msg format version to
   AVSV_AVD_AVND_MSG_FMT_VER_8? An old amfnd will assert if it gets an
   AVSV_D2N_CONTAINED_SU_MSG_INFO msg.
   Thanks
   Gary
   -----------------------------------------------------------------------
   -----
   --
   Check out the vibrant tech community on one of the world's most
   engaging tech sites, Slashdot.org! [88]http://sdm.link/slashdot
   <[89]https://protect-us.mimecast.com/s/qqUZCR6Lz6CZZALcN2_lT?domain=sdm
   .link>
   _______________________________________________
   Opensaf-devel mailing list
   [90]Opensaf-devel@lists.sourceforge.net
   [91]<mailto:Opensaf-devel@lists.sourceforge.net>
   [92]https://lists.sourceforge.net/lists/listinfo/opensaf-devel
   <[93]https://protect-us.mimecast.com/s/rLi0CVONEOUQQDAtyfptK?domain=lis
   ts.sourceforge.net>
   _______________________________________________
   Opensaf-devel mailing list
   [94]Opensaf-devel@lists.sourceforge.net
   [95]https://lists.sourceforge.net/lists/listinfo/opensaf-devel

References

   1. mailto:thuan.t...@dektech.com.au
   2. mailto:ajo...@rbbn.com
   3. mailto:nagen...@hasolutions.in
   4. mailto:gary....@dektech.com.au
   5. mailto:hans.nordeb...@ericsson.com
   6. mailto:ravisekhar.ko...@oracle.com
   7. mailto:opensaf-devel@lists.sourceforge.net
   8. mailto:ajo...@rbbn.com
   9. mailto:thuan.t...@dektech.com.au
  10. mailto:nagen...@hasolutions.in
  11. mailto:gary....@dektech.com.au
  12. mailto:hans.nordeb...@ericsson.com
  13. mailto:ravisekhar.ko...@oracle.com
  14. mailto:opensaf-devel@lists.sourceforge.net
  15. mailto:ajo...@rbbn.com
  16. mailto:ajo...@rbbn.com
  17. mailto:thuan.t...@dektech.com.au
  18. mailto:thuan.t...@dektech.com.au
  19. mailto:nagen...@hasolutions.in
  20. mailto:nagen...@hasolutions.in
  21. mailto:gary....@dektech.com.au
  22. mailto:gary....@dektech.com.au
  23. mailto:hans.nordeb...@ericsson.com
  24. mailto:hans.nordeb...@ericsson.com
  25. mailto:ravisekhar.ko...@oracle.com
  26. mailto:ravisekhar.ko...@oracle.com
  27. mailto:opensaf-devel@lists.sourceforge.net
  28. mailto:opensaf-devel@lists.sourceforge.net
  29. mailto:nagen...@hasolutions.in
  30. mailto:nagen...@hasolutions.in
  31. mailto:nagen...@hasolutions.in
  32. mailto:nagen...@hasolutions.in
  33. mailto:ajo...@rbbn.com
  34. mailto:ajo...@rbbn.com
  35. mailto:gary....@dektech.com.au
  36. mailto:gary....@dektech.com.au
  37. mailto:hans.nordeb...@ericsson.com
  38. mailto:hans.nordeb...@ericsson.com
  39. mailto:ravisekhar.ko...@oracle.com
  40. mailto:ravisekhar.ko...@oracle.com
  41. mailto:opensaf-devel@lists.sourceforge.net
  42. mailto:opensaf-devel@lists.sourceforge.net
  43. 
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
  44. 
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
  45. mailto:ajo...@rbbn.com
  46. mailto:ajo...@rbbn.com
  47. mailto:nagen...@hasolutions.in
  48. mailto:nagen...@hasolutions.in
  49. mailto:gary....@dektech.com.au
  50. mailto:gary....@dektech.com.au
  51. mailto:hans.nordeb...@ericsson.com
  52. mailto:hans.nordeb...@ericsson.com
  53. mailto:ravisekhar.ko...@oracle.com
  54. mailto:ravisekhar.ko...@oracle.com
  55. mailto:opensaf-devel@lists.sourceforge.net
  56. mailto:opensaf-devel@lists.sourceforge.net
  57. mailto:nagen...@hasolutions.in
  58. mailto:nagen...@hasolutions.in
  59. 
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
  60. 
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
  61. mailto:ajo...@rbbn.com
  62. mailto:ajo...@rbbn.com
  63. mailto:nagen...@hasolutions.in
  64. mailto:nagen...@hasolutions.in
  65. mailto:gary....@dektech.com.au
  66. mailto:gary....@dektech.com.au
  67. mailto:hans.nordeb...@ericsson.com
  68. mailto:hans.nordeb...@ericsson.com
  69. mailto:ravisekhar.ko...@oracle.com
  70. mailto:opensaf-devel@lists.sourceforge.net
  71. mailto:opensaf-devel@lists.sourceforge.net
  72. mailto:nagen...@hasolutions.in
  73. mailto:nagen...@hasolutions.in
  74. 
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
  75. 
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
  76. mailto:ajo...@rbbn.com
  77. mailto:ajo...@rbbn.com
  78. mailto:gary....@dektech.com.au
  79. mailto:gary....@dektech.com.au
  80. mailto:hans.nordeb...@ericsson.com
  81. mailto:hans.nordeb...@ericsson.com
  82. mailto:ravisekhar.ko...@oracle.com
  83. mailto:ravisekhar.ko...@oracle.com
  84. mailto:nagen...@hasolutions.in
  85. mailto:nagen...@hasolutions.in
  86. mailto:opensaf-devel@lists.sourceforge.net
  87. mailto:opensaf-devel@lists.sourceforge.net
  88. https://protect-us.mimecast.com/s/MlQdCqx23xSN4WmHqvqOZ?domain=sdm.link
  89. https://protect-us.mimecast.com/s/MlQdCqx23xSN4WmHqvqOZ?domain=sdm.link
  90. mailto:Opensaf-devel@lists.sourceforge.net
  91. mailto:Opensaf-devel@lists.sourceforge.net
  92. 
https://protect-us.mimecast.com/s/ILyKCrkY3kiZN9LUQIG-p?domain=lists.sourceforge.net
  93. 
https://protect-us.mimecast.com/s/ILyKCrkY3kiZN9LUQIG-p?domain=lists.sourceforge.net
  94. mailto:Opensaf-devel@lists.sourceforge.net
  95. 
https://protect-us.mimecast.com/s/ILyKCrkY3kiZN9LUQIG-p?domain=lists.sourceforge.net

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to