- **status**: review --> fixed
- **Comment**:
develop(5.17.10):
commit 3be8e9adb4670607a93907a886b0cf301570d65a
Author: Zoran Milinkovic <[email protected]>
Date: Tue Jul 11 11:19:31 2017 +0200
clm: make CLM tests independent of other CLM tests [#2520]
The patch removes dependencies between CLM tests. CLM tests can be run more
times now.
Duplicated CLM tests are removed from clmtest.
-----
release(5.17.8):
commit 89ba594771730a88b6584af19e6ce629bbd8fdbb
Author: Zoran Milinkovic <[email protected]>
Date: Tue Jul 11 11:19:31 2017 +0200
clm: make CLM tests independent of other CLM tests [#2520]
The patch removes dependencies between CLM tests. CLM tests can be run more
times now.
Duplicated CLM tests are removed from clmtest.
---
** [tickets:#2520] clm: make CLM tests more independent from other CLM tests**
**Status:** fixed
**Milestone:** 5.17.08
**Created:** Tue Jul 04, 2017 01:50 PM UTC by Zoran Milinkovic
**Last Updated:** Wed Jul 05, 2017 02:56 PM UTC
**Owner:** Zoran Milinkovic
Today's CLM tests depend on other tests, request them to revert CLM state to
the starting state.
If CLM tests are done manually, then they must be done in an order. Also, some
tests cannot be called twice.
For example test 7 21:
~~~
$ clmtest 7 21
Suite 7: Test case for saClmClusterTrack. ** For all tests to pass, Run a
payload with node_name PL-3 **
waiting on poll
Inside TrackCallback4
invocation : 0
Step : 4
error = 1
numberOfMembers = 2
No of items = 1
Value of i = 0
Cluster Change = 3
Node Name length = 36, value = safNode=PL-3,safCluster=myClmCluster
Node Member = 0
Node view number = 5
Node eename length = 0,value =
Node boottimestamp = 1499175648545988548
Node nodeAddress family = 1,node address length = 0, node address value =
Node nodeid = 131855
21 PASSED saClmClusterTrack_4 with SA_TRACK_CHANGES_ONLY track flags -
admin lock
=====================================================================================
Test Result:
Total: 1
Passed: 1
Failed: 0
~~~
And if we repeat the test, it fails:
~~~
$ clmtest 7 21
Suite 7: Test case for saClmClusterTrack. ** For all tests to pass, Run a
payload with node_name PL-3 **
error - saImmOmAdminOperationInvoke_2 admin-op RETURNED: SA_AIS_ERR_NO_OP (28)
clmtest: src/clm/apitest/tet_saClmClusterTrack.c:604: saClmClusterTrack_21:
Assertion `ret == 1' failed.
waiting on pollAborted
~~~
Or if we call another test after executing test 7 21:
~~~
$ clmtest 7 23
Suite 7: Test case for saClmClusterTrack. ** For all tests to pass, Run a
payload with node_name PL-3 **
error - saImmOmAdminOperationInvoke_2 admin-op RETURNED: SA_AIS_ERR_NO_OP (28)
clmtest: src/clm/apitest/tet_saClmClusterTrack.c:672: saClmClusterTrack_23:
Assertion `ret == 1' failed.
Aborted
~~~
---
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.------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-tickets mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
