Comment 15 for bug 1686817

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/39176
Committed: http://github.com/Juniper/contrail-controller/commit/2a8f57c59408e7e80cd8dd510dd1a72157a685cd
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 2a8f57c59408e7e80cd8dd510dd1a72157a685cd
Author: Naren Karjala <email address hidden>
Date: Tue Jan 23 14:02:22 2018 -0800

Retain Routing Instances Tracebuf upon deletion

- Retain the Routing Instance Trace buff even when the instance is deleted.
- Maintain two map list for proper bookeeping of the Active Routing instance
Tracebuf and one for the deleted Routing instance.
- When new routing instance is created, the trace buf is looked up in Active
map and if not found will look into Dormant map. If not found in these
two lists, a new trace buf is created and added to Active map list.
- Upon deletion of RI, the tracebuf in Active list is moved to Dormant list.
- Ran s/w unit test to verify the tracebuf chnages.

- taken care of the review comments (2/13).
- Added support upto 64K (by default) routinginstance trace buffers retention
Test results are available at:
Server: [nkarjala@ubuntu-build04]$
/build/nkarjala/SB/mainline/build_test_dormant.log
/build/nkarjala/SB/mainline/build_xmpp_dormant_test.log

Note:
The target was built and tested with LOG_DISABLE to false (made hardcoded changes to the src)

export TASK_UTIL_WAIT_TIME=10000 TASK_UTIL_RETRY_COUNT=250000 BGP_STRESS_TEST_SUITE=1 NO_HEAPCHECK=TRUE; scons controller/src/bgp:test |& tee > build_test_dormant.log
unset BUILD_ONLY; export TASK_UTIL_WAIT_TIME=10000 TASK_UTIL_RETRY_COUNT=250000 BGP_STRESS_TEST_SUITE=1 NO_HEAPCHECK=TRUE; scons controller/src/xmpp:test control-node |& tee > build_xmpp_dormant_test.log

Change-Id: I10b80975247d289a66836edacd13a73b2a8e42ea
Partial-Bug: #1686817