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]<[email protected]>
Sent: Wednesday, September 26, 2018 6:12 PM
To: 'Jones, Alex' [2]<[email protected]>; [3][email protected];
'Gary Lee' [4]<[email protected]>;
[5][email protected]; [6][email protected]
Cc: [7][email protected]
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]<[email protected]>
Sent: Wednesday, September 26, 2018 12:10 AM
To: Tran Thuan [9]<[email protected]>;
[10][email protected]; 'Gary Lee' [11]<[email protected]>;
[12][email protected]; [13][email protected]
Cc: [14][email protected]
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:[email protected]> [16]<[email protected]>
Sent: Saturday, September 22, 2018 1:01 AM
To: Tran Thuan [17]<mailto:[email protected]>
[18]<[email protected]>; [19][email protected]
[20]<mailto:[email protected]> ; 'Gary Lee'
[21]<mailto:[email protected]> [22]<[email protected]>;
[23][email protected]
[24]<mailto:[email protected]> ;
[25][email protected]
[26]<mailto:[email protected]>
Cc: [27][email protected]
[28]<mailto:[email protected]>
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][email protected] [30]<mailto:[email protected]>
[31]<mailto:[email protected]> [32]<[email protected]>
Sent: Friday, September 7, 2018 12:57 PM
To: Alex Jones [33]<mailto:[email protected]> [34]<[email protected]>; Gary
Lee [35]<mailto:[email protected]> [36]<[email protected]>;
[37][email protected]
[38]<mailto:[email protected]> ;
[39][email protected]
[40]<mailto:[email protected]>
Cc: [41][email protected]
[42]<mailto:[email protected]>
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:[email protected]> [46]<[email protected]>
Date: 9/6/18 8:52 pm
To: [47][email protected] [48]<mailto:[email protected]> ,
"Gary Lee" [49]<mailto:[email protected]>
[50]<[email protected]>, [51][email protected]
[52]<mailto:[email protected]> ,
[53][email protected]
[54]<mailto:[email protected]>
Cc: [55][email protected]
[56]<mailto:[email protected]>
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][email protected]
[58]<mailto:[email protected]> 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:[email protected]> [62]<[email protected]>
Date: 8/29/18 9:29 pm
To: [63][email protected] [64]<mailto:[email protected]> ,
"Gary Lee" [65]<mailto:[email protected]>
[66]<[email protected]>, [67][email protected]
[68]<mailto:[email protected]> ,
[69][email protected]
Cc: [70][email protected]
[71]<mailto:[email protected]>
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][email protected]
[73]<mailto:[email protected]> 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:[email protected]> [77]<[email protected]>
Date: 8/15/18 11:10 pm
To: "Gary Lee" [78]<mailto:[email protected]>
[79]<[email protected]>, [80][email protected]
[81]<mailto:[email protected]> ,
[82][email protected]
[83]<mailto:[email protected]> , [84][email protected]
[85]<mailto:[email protected]>
Cc: [86][email protected]
[87]<mailto:[email protected]>
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][email protected]
[91]<mailto:[email protected]>
[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][email protected]
[95]https://lists.sourceforge.net/lists/listinfo/opensaf-devel
References
1. mailto:[email protected]
2. mailto:[email protected]
3. mailto:[email protected]
4. mailto:[email protected]
5. mailto:[email protected]
6. mailto:[email protected]
7. mailto:[email protected]
8. mailto:[email protected]
9. mailto:[email protected]
10. mailto:[email protected]
11. mailto:[email protected]
12. mailto:[email protected]
13. mailto:[email protected]
14. mailto:[email protected]
15. mailto:[email protected]
16. mailto:[email protected]
17. mailto:[email protected]
18. mailto:[email protected]
19. mailto:[email protected]
20. mailto:[email protected]
21. mailto:[email protected]
22. mailto:[email protected]
23. mailto:[email protected]
24. mailto:[email protected]
25. mailto:[email protected]
26. mailto:[email protected]
27. mailto:[email protected]
28. mailto:[email protected]
29. mailto:[email protected]
30. mailto:[email protected]
31. mailto:[email protected]
32. mailto:[email protected]
33. mailto:[email protected]
34. mailto:[email protected]
35. mailto:[email protected]
36. mailto:[email protected]
37. mailto:[email protected]
38. mailto:[email protected]
39. mailto:[email protected]
40. mailto:[email protected]
41. mailto:[email protected]
42. mailto:[email protected]
43.
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
44.
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
45. mailto:[email protected]
46. mailto:[email protected]
47. mailto:[email protected]
48. mailto:[email protected]
49. mailto:[email protected]
50. mailto:[email protected]
51. mailto:[email protected]
52. mailto:[email protected]
53. mailto:[email protected]
54. mailto:[email protected]
55. mailto:[email protected]
56. mailto:[email protected]
57. mailto:[email protected]
58. mailto:[email protected]
59.
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
60.
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
61. mailto:[email protected]
62. mailto:[email protected]
63. mailto:[email protected]
64. mailto:[email protected]
65. mailto:[email protected]
66. mailto:[email protected]
67. mailto:[email protected]
68. mailto:[email protected]
69. mailto:[email protected]
70. mailto:[email protected]
71. mailto:[email protected]
72. mailto:[email protected]
73. mailto:[email protected]
74.
https://protect-us.mimecast.com/s/ODVpCo2O52iYo4xcOczy8?domain=hasolutions.in
75.
https://protect-us.mimecast.com/s/vp4zCpYz3YiolDXu2ZV6t?domain=hasolutions.in
76. mailto:[email protected]
77. mailto:[email protected]
78. mailto:[email protected]
79. mailto:[email protected]
80. mailto:[email protected]
81. mailto:[email protected]
82. mailto:[email protected]
83. mailto:[email protected]
84. mailto:[email protected]
85. mailto:[email protected]
86. mailto:[email protected]
87. mailto:[email protected]
88. https://protect-us.mimecast.com/s/MlQdCqx23xSN4WmHqvqOZ?domain=sdm.link
89. https://protect-us.mimecast.com/s/MlQdCqx23xSN4WmHqvqOZ?domain=sdm.link
90. mailto:[email protected]
91. mailto:[email protected]
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:[email protected]
95.
https://protect-us.mimecast.com/s/ILyKCrkY3kiZN9LUQIG-p?domain=lists.sourceforge.net
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
