Hi Anders, The test is about testing IMM OpenSAF 4.5 protocol, and the test disable and enable IMM OpenSAF 4.5 features. So, there should be admin ops on IMM objects.
'saImmService' admin owner name is used for disabling and enabling IMM features in the test, which is standard IMM admin owner name used on IMM object There is no need for releasing 'saImmService' admin owner. Same works for immadm on IMM object. 'saImmService' admin owner is not released after calling admin op on IMM objects. BR, Zoran -----Original Message----- From: [email protected] [mailto:[email protected]] Sent: Tuesday, February 09, 2016 12:47 PM To: Zoran Milinkovic; Hung Duc Nguyen Cc: [email protected] Subject: Re: [devel] [PATCH 1 of 1] imm: Release adminowner of IMM object for immoitest [#1667] Hi Zoran and Hung, Just a small comment on this. Note that an IMMA OM client can get admin-owner released automatically at admo-handle-close or process termination, if you set the parameter "ReleaseOwnershipOnFinalize" to SA_TRUE in the adminOwnerInitialize API. In general the imma tests should be fixed to always use their own test objects (and never (ab)use regular/normal IMM objects for the testing). Such test objects are created by the test framework and destroyed by the test framework. /AndersBj >----Ursprungligt meddelande---- >Från : [email protected] >Datum : 2016-02-09 - 10:19 (GMT) >Till : [email protected], [email protected] Kopia >: [email protected] >Ämne : Re: [devel] [PATCH 1 of 1] imm: Release adminowner of IMM object >for immoitest [#1667] > >Hi Hung, > >Was there any problem with not releasing admin owner on IMM object ? > >IMM test should not release admin owner on IMM object. It may accidentally >release admin owner on ongoing operation on IMM object. > >I have to NACK this patch. >I think that the ticket is not valid, and if there is no valid reason for >releasing admin owner, the ticket can be closed with status 'invalid'. > >Thanks, >Zoran > >-----Original Message----- >From: Hung Nguyen [mailto:[email protected]] >Sent: Thursday, February 04, 2016 8:59 AM >To: Zoran Milinkovic; [email protected] >Cc: [email protected] >Subject: [PATCH 1 of 1] imm: Release adminowner of IMM object for >immoitest [#1667] > > tests/immsv/implementer/test_saImmOiImplementerSet.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > >Release adminowner of IMM object for immoitest. > >diff --git a/tests/immsv/implementer/test_saImmOiImplementerSet.c >b/tests/immsv/implementer/test_saImmOiImplementerSet.c >--- a/tests/immsv/implementer/test_saImmOiImplementerSet.c >+++ b/tests/immsv/implementer/test_saImmOiImplementerSet.c >@@ -368,6 +368,7 @@ void saImmOiImplementerSet_10(void) > safassert(saImmOmAdminOwnerInitialize(immHandle, "safImmService", > SA_FALSE, &ownerHandle), SA_AIS_OK); > safassert(saImmOmAdminOwnerSet(ownerHandle, immObjs, SA_IMM_ONE), > SA_AIS_OK); > safassert(saImmOmAdminOperationInvoke_2(ownerHandle, &immObj, 1, 2, >params, &err, 100), SA_AIS_OK); >+ safassert(saImmOmAdminOwnerRelease(ownerHandle, immObjs, SA_IMM_ONE), >+SA_AIS_OK); > safassert(saImmOmAdminOwnerFinalize(ownerHandle), SA_AIS_OK); > > safassert(saImmOmAdminOwnerInitialize(immHandle, (const >SaImmAdminOwnerNameT)__FUNCTION__, SA_TRUE, &ownerHandle), SA_AIS_OK); @@ >-388,6 +389,7 @@ void saImmOiImplementerSet_10(void) > safassert(saImmOmAdminOwnerInitialize(immHandle, "safImmService", > SA_FALSE, &ownerHandle), SA_AIS_OK); > safassert(saImmOmAdminOwnerSet(ownerHandle, immObjs, SA_IMM_ONE), > SA_AIS_OK); > safassert(saImmOmAdminOperationInvoke_2(ownerHandle, &immObj, 1, 1, >params, &err, 100), SA_AIS_OK); >+ safassert(saImmOmAdminOwnerRelease(ownerHandle, immObjs, SA_IMM_ONE), >+SA_AIS_OK); > safassert(saImmOmAdminOwnerFinalize(ownerHandle), SA_AIS_OK); > > safassert(saImmOmFinalize(immHandle), SA_AIS_OK); > >----------------------------------------------------------------------- >------- >Site24x7 APM Insight: Get Deep Visibility into Application Performance >APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month >Monitor end-to-end web transactions and take corrective actions now >Troubleshoot faster and improve end-user experience. Signup Now! >http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140 >_______________________________________________ >Opensaf-devel mailing list >[email protected] >https://lists.sourceforge.net/lists/listinfo/opensaf-devel > ------------------------------------------------------------------------------ Site24x7 APM Insight: Get Deep Visibility into Application Performance APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month Monitor end-to-end web transactions and take corrective actions now Troubleshoot faster and improve end-user experience. Signup Now! http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140 _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
