- **status**: unassigned --> wontfix
- **Comment**:
This is a so called idempotency case.
In other words, everybody gains and nobody gets any problems due to the
relaxed handling of response to releasing an already released resource.
On the other hand "fixing" this defect of minor severity could cause major
problems for some legacy applications that simply have the benign "bug"
of sometimes redundantly requesting class-implementership to be released.
Yes this could formally be an "interface violation" by the user, but it is
a benign one since replying OK can not cause any problems for that client
or for any other IMM user. The case also probably meshes with imm handle
resurrect.
---
** [tickets:#1370] classimplementerrelease needs to return an error when the OI
is not an implementer for the class**
**Status:** wontfix
**Milestone:** future
**Created:** Tue May 12, 2015 09:43 AM UTC by Sirisha Alla
**Last Updated:** Tue May 12, 2015 09:43 AM UTC
**Owner:** nobody
1) Set an implementer for the class
2) Invoke classimplementerrelease from the OI twice on the same class.
API invocations return SA_AIS_OK even when the OI is no more implementer for
the class. There is no clarity on the error to be returned by the spec when the
OI is no more class implementer for a class and there are no objects for the
class
IMMND Trace:
May 12 12:16:25.523918 osafimmnd [2416:ImmModel.cc:12845] >> classImplementerSet
May 12 12:16:25.524114 osafimmnd [2416:ImmModel.cc:13070] NO implementer for
class 'DummyClassNameForOITesting' is OiClassImplementerRelease_twice => class
extent is safe.
May 12 12:16:25.524128 osafimmnd [2416:ImmModel.cc:13084] << classImplementerSet
May 12 12:16:25.525276 osafimmnd [2416:immnd_evt.c:8635] >> dequeue_outgoing
May 12 12:16:25.525299 osafimmnd [2416:immnd_evt.c:8641] TR Pending replies:0
space:16 out list?:(nil)
May 12 12:16:25.525309 osafimmnd [2416:immnd_evt.c:8670] << dequeue_outgoing
May 12 12:16:25.525317 osafimmnd [2416:immnd_evt.c:8754] <<
immnd_evt_proc_fevs_rcv
May 12 12:16:25.525338 osafimmnd [2416:immsv_evt.c:5414] T8 Received:
IMMND_EVT_A2ND_IMM_FEVS (14) from 2020f
May 12 12:16:25.525350 osafimmnd [2416:immnd_evt.c:2822] T2 sender_count: 1
size: 51
May 12 12:16:25.525361 osafimmnd [2416:immnd_evt.c:3103] >>
immnd_fevs_local_checks
May 12 12:16:25.525377 osafimmnd [2416:immnd_evt.c:3555] <<
immnd_fevs_local_checks
May 12 12:16:25.525388 osafimmnd [2416:immnd_evt.c:3021] T2 SENDING FEVS TO IMMD
May 12 12:16:25.525397 osafimmnd [2416:immsv_evt.c:5395] T8 Sending:
IMMD_EVT_ND2D_FEVS_REQ to 0
May 12 12:16:25.527599 osafimmnd [2416:immnd_evt.c:8693] >>
immnd_evt_proc_fevs_rcv
May 12 12:16:25.527623 osafimmnd [2416:immnd_evt.c:8709] T2 FEVS from myself,
still pending:0
May 12 12:16:25.527631 osafimmnd [2416:immsv_evt.c:5414] T8 Received:
IMMND_EVT_A2ND_OI_CL_IMPL_REL (43) from 0
May 12 12:16:25.527640 osafimmnd [2416:ImmModel.cc:13103] >>
classImplementerRelease
May 12 12:16:25.527761 osafimmnd [2416:ImmModel.cc:13265] NO implementer for
class 'DummyClassNameForOITesting' is released => class extent is UNSAFE
May 12 12:16:25.527777 osafimmnd [2416:ImmModel.cc:13272] <<
classImplementerRelease
May 12 12:16:25.527782 osafimmnd [2416:immnd_evt.c:9540] T2 SENDRSP OK
May 12 12:16:25.527986 osafimmnd [2416:immnd_evt.c:8635] >> dequeue_outgoing
May 12 12:16:25.527996 osafimmnd [2416:immnd_evt.c:8641] TR Pending replies:0
space:16 out list?:(nil)
May 12 12:16:25.528001 osafimmnd [2416:immnd_evt.c:8670] << dequeue_outgoing
May 12 12:16:25.528005 osafimmnd [2416:immnd_evt.c:8754] <<
immnd_evt_proc_fevs_rcv
May 12 12:16:25.528013 osafimmnd [2416:immsv_evt.c:5414] T8 Received:
IMMND_EVT_A2ND_IMM_FEVS (14) from 2020f
May 12 12:16:25.528019 osafimmnd [2416:immnd_evt.c:2822] T2 sender_count: 1
size: 51
May 12 12:16:25.528024 osafimmnd [2416:immnd_evt.c:3103] >>
immnd_fevs_local_checks
May 12 12:16:25.528031 osafimmnd [2416:immnd_evt.c:3555] <<
immnd_fevs_local_checks
May 12 12:16:25.528036 osafimmnd [2416:immnd_evt.c:3021] T2 SENDING FEVS TO IMMD
May 12 12:16:25.528041 osafimmnd [2416:immsv_evt.c:5395] T8 Sending:
IMMD_EVT_ND2D_FEVS_REQ to 0
May 12 12:16:25.529497 osafimmnd [2416:immnd_evt.c:8693] >>
immnd_evt_proc_fevs_rcv
May 12 12:16:25.529514 osafimmnd [2416:immnd_evt.c:8709] T2 FEVS from myself,
still pending:0
May 12 12:16:25.529528 osafimmnd [2416:immsv_evt.c:5414] T8 Received:
IMMND_EVT_A2ND_OI_CL_IMPL_REL (43) from 0
May 12 12:16:25.529537 osafimmnd [2416:ImmModel.cc:13103] >>
classImplementerRelease
May 12 12:16:25.529554 osafimmnd [2416:ImmModel.cc:13231] T7 Class
'DummyClassNameForOITesting' has no implementer
May 12 12:16:25.529563 osafimmnd [2416:ImmModel.cc:13272] <<
classImplementerRelease
May 12 12:16:25.529572 osafimmnd [2416:immnd_evt.c:9540] T2 SENDRSP OK
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/opensaf/tickets/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a
mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
Opensaf-tickets mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
