diff --git a/core/deliverservice/client.go b/core/deliverservice/client.go index fcbcef2932d..37688a1ae96 100644 --- a/core/deliverservice/client.go +++ b/core/deliverservice/client.go @@ -38,7 +38,7 @@ import ( var logger *logging.Logger // package-level logger func init() { - logger = logging.MustGetLogger("noopssinglechain.client") + logger = logging.MustGetLogger("deliveryService") } // DeliverService used to communicate with orderers to obtain diff --git a/gossip/state/state.go b/gossip/state/state.go index 03d2fc91ea4..2d79d0a093b 100644 --- a/gossip/state/state.go +++ b/gossip/state/state.go @@ -126,7 +126,7 @@ func NewGossipStateProvider(chainID string, g gossip.Gossip, committer committer stopFlag: 0, // Create a queue for payload received - payloads: NewPayloadsBuffer(height + 1), + payloads: NewPayloadsBuffer(height), committer: committer, @@ -329,17 +329,12 @@ func (s *GossipStateProviderImpl) antiEntropy() { } } } + s.logger.Debug("[XXX]: Stateprovider stopped, stoping anti entropy procedure.") if current == max { - // No messages in the buffer or there are no gaps - //s.logger.Debugf("Current ledger height is the same as ledger height on other peers.") continue } - if current > 0 { - current = current + 1 - } - //s.logger.Debugf("Requesting new blocks in range [%d...%d].", current+1, max) s.requestBlocksInRange(uint64(current), uint64(max)) } s.logger.Debug("[XXX]: Stateprovider stopped, stoping anti entropy procedure.") diff --git a/gossip/state/state_test.go b/gossip/state/state_test.go index 7443cda6979..e46f5e1f40e 100644 --- a/gossip/state/state_test.go +++ b/gossip/state/state_test.go @@ -25,6 +25,7 @@ import ( "time" pb "github.com/golang/protobuf/proto" + "github.com/hyperledger/fabric/common/configtx/test" "github.com/hyperledger/fabric/common/util" "github.com/hyperledger/fabric/core/committer" "github.com/hyperledger/fabric/core/ledger/ledgermgmt" @@ -123,7 +124,7 @@ func (*naiveCryptoService) ValidateIdentity(peerIdentity api.PeerIdentityType) e func bootPeers(ids ...int) []string { peers := []string{} for _, id := range ids { - peers = append(peers, fmt.Sprintf("localhost:%d", (id+portPrefix))) + peers = append(peers, fmt.Sprintf("localhost:%d", id+portPrefix)) } return peers } @@ -172,6 +173,8 @@ func newGossipInstance(config *gossip.Config) gossip.Gossip { // Create new instance of KVLedger to be used for testing func newCommitter(id int) committer.Committer { ledger, _ := ledgermgmt.CreateLedger(strconv.Itoa(id)) + cb, _ := test.MakeGenesisBlock(util.GetTestChainID()) + ledger.Commit(cb) return committer.NewLedgerCommitter(ledger, &validator.MockValidator{}) } @@ -334,8 +337,8 @@ func TestNewGossipStateProvider_SendingManyMessages(t *testing.T) { logger.Debug("[*****]: Trying to see all peers get all blocks") for _, p := range peersSet { height, err := p.commit.LedgerHeight() - if height != uint64(msgCount) || err != nil { - logger.Debug("[XXXXXXX]: Ledger height is at: ", height) + if height != uint64(msgCount+1) || err != nil { + //logger.Debug("[XXXXXXX]: Ledger height is at: ", height) return false } }