[FAB-18194] Fix service discovery for legacy installed chaincodes

Service discovery endorsers query fails with errors:
"failed constructing descriptor for chaincodes"
"required chaincodes are not installed on sufficient peers"

The chaincodes installed with legacy chaincode lifecycle were not getting recognized
after peer restart since the path to installed chaincodes was set after
the legacy chaincode cache initialized.
Setting the installed chaincode path earlier in peer start sequence resolves the issue.

Signed-off-by: David Enyeart <enyeart@us.ibm.com>

integration/gossip/gossip_test.go

@@ -125,6 +125,14 @@ var _ = Describe("Gossip State Transfer and Membership", func() {
 		basePeerForTransactions := peer0Org1
 		nwo.DeployChaincodeLegacy(network, channelName, orderer, chaincode, basePeerForTransactions)
 
+		By("verifying peer0Org1 discovers all the peers and the legacy chaincode before starting the tests")
+		Eventually(nwo.DiscoverPeers(network, peer0Org1, "User1", "testchannel"), network.EventuallyTimeout).Should(ConsistOf(
+			network.DiscoveredPeer(peer0Org1, "_lifecycle", "mycc"),
+			network.DiscoveredPeer(peer1Org1, "_lifecycle"),
+			network.DiscoveredPeer(peer0Org2, "_lifecycle"),
+			network.DiscoveredPeer(peer1Org2, "_lifecycle"),
+		))
+
 		By("STATE TRANSFER TEST 1: newly joined peers should receive blocks from the peers that are already up")
 
 		// Note, a better test would be to bring orderer down before joining the two peers.
@@ -142,6 +150,14 @@ var _ = Describe("Gossip State Transfer and Membership", func() {
 		basePeerForTransactions = peer1Org1
 		nwo.InstallChaincodeLegacy(network, chaincode, basePeerForTransactions)
 
+		By("verifying peer0Org1 discovers all the peers and the additional legacy chaincode installed on peer1Org1")
+		Eventually(nwo.DiscoverPeers(network, peer0Org1, "User1", "testchannel"), network.EventuallyTimeout).Should(ConsistOf(
+			network.DiscoveredPeer(peer0Org1, "_lifecycle", "mycc"),
+			network.DiscoveredPeer(peer1Org1, "_lifecycle", "mycc"),
+			network.DiscoveredPeer(peer0Org2, "_lifecycle"),
+			network.DiscoveredPeer(peer1Org2, "_lifecycle"),
+		))
+
 		By("stopping peer0Org1 (currently elected leader in Org1) and peer1Org2 (static leader in Org2)")
 		stopPeers(nwprocs, peer0Org1, peer1Org2)
 
@@ -152,6 +168,15 @@ var _ = Describe("Gossip State Transfer and Membership", func() {
 		// This effectively tests leader election as well, since the newly elected leader in Org1 (peer1Org1) will be the only peer
 		// that receives blocks from orderer and will therefore serve as the provider of blocks to all other peers.
 		sendTransactionsAndSyncUpPeers(nwprocs, orderer, basePeerForTransactions, channelName, peer0Org1, peer1Org2)
+
+		By("verifying peer0Org1 can still discover all the peers and the legacy chaincode after it has been restarted")
+		Eventually(nwo.DiscoverPeers(network, peer0Org1, "User1", "testchannel"), network.EventuallyTimeout).Should(ConsistOf(
+			network.DiscoveredPeer(peer0Org1, "_lifecycle", "mycc"),
+			network.DiscoveredPeer(peer1Org1, "_lifecycle", "mycc"),
+			network.DiscoveredPeer(peer0Org2, "_lifecycle"),
+			network.DiscoveredPeer(peer1Org2, "_lifecycle"),
+		))
+
 	})
 
 	When("gossip connection is lost and restored", func() {

internal/peer/node/start.go

@@ -367,6 +367,10 @@ func serve(args []string) error {
 		LegacyDeployedCCInfoProvider: &lscc.DeployedCCInfoProvider{},
 	}
 
+	// Configure CC package storage before ccInfoFSImpl.ListInstalledChaincodes() gets called
+	lsccInstallPath := filepath.Join(coreconfig.GetPath("peer.fileSystemPath"), "chaincodes")
+	ccprovider.SetChaincodesPath(lsccInstallPath)
+
 	ccInfoFSImpl := &ccprovider.CCInfoFSImpl{GetHasher: factory.GetDefault()}
 
 	// legacyMetadataManager collects metadata information from the legacy
@@ -465,10 +469,6 @@ func serve(args []string) error {
 
 	peerInstance.GossipService = gossipService
 
-	// Configure CC package storage
-	lsccInstallPath := filepath.Join(coreconfig.GetPath("peer.fileSystemPath"), "chaincodes")
-	ccprovider.SetChaincodesPath(lsccInstallPath)
-
 	if err := lifecycleCache.InitializeLocalChaincodes(); err != nil {
 		return errors.WithMessage(err, "could not initialize local chaincodes")
 	}