Move rtarget_table_test to controller/src/bgp/rtarget:flaky-test

Change-Id: Iff5ec3a383567e6d5523b2fad8e61d59b5d424bf
Fix issues in bgp_config_listener_test.cc.

Change-Id: I1e3a3818b3b52fc45e7aa6a2b1358f33cb9f6550
Merge "Move rtarget_table_test to controller/src/bgp/rtarget:flaky-test"
q_api_data is not used in 1.1

Change-Id: I4b4257c9148c12a6c71b0aed3476bfe0e59b2f76
Merge "Fix issues in bgp_config_listener_test.cc."
Merge "q_api_data is not used in 1.1"
Fix bug that caused double free of BgpInstanceConfig.

https://bugs.launchpad.net/juniperopenstack/+bug/1334510

BgpPeeringConfig has a back pointer to BgpInstanceConfig but does
not take a reference on the BgpInstanceConfig.  Further, the code
can delete the BgpInstanceConfig when a BgpPeeringConfig is being
deleted. If all references to BgpInstanceConfig have already gone
and it has been deleted, this results in a double free.

Fix is to keep a map of BgpPeeringConfigs in an BgpInstanceConfig
and to modify DeleteIfEmpty to check the PeeringMap.

Add logs for BgpInstanceConfig create/delete in a few places that
were overlooked previously.

Change-Id: I8579d534390b1aba7f1b14a591f51013d2ca731d