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
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel