clean up

[bytom/vapor.git] / toolbar / precog / monitor / monitor.go

diff --git a/toolbar/precog/monitor/monitor.go b/toolbar/precog/monitor/monitor.go
index cb6ebd2..b0de9a2 100644 (file)
--- a/toolbar/precog/monitor/monitor.go
+++ b/toolbar/precog/monitor/monitor.go
@@ -2,12 +2,10 @@ package monitor
 
 import (
        "fmt"
-       "io/ioutil"
        "os"
        "os/user"
        "strings"
-       // "sync"
-       // "time"
+       "sync"
 
        "github.com/jinzhu/gorm"
        log "github.com/sirupsen/logrus"
@@ -18,9 +16,8 @@ import (
        "github.com/vapor/event"
        "github.com/vapor/netsync/chainmgr"
        "github.com/vapor/netsync/consensusmgr"
-       "github.com/vapor/p2p"
-       // msgs "github.com/vapor/netsync/messages"
        "github.com/vapor/netsync/peers"
+       "github.com/vapor/p2p"
        "github.com/vapor/p2p/discover/dht"
        "github.com/vapor/p2p/discover/mdns"
        "github.com/vapor/p2p/signlib"
@@ -29,18 +26,18 @@ import (
 )
 
 type monitor struct {
-       // *sync.RWMutex
-       cfg           *config.Config
-       db            *gorm.DB
-       nodeCfg       *vaporCfg.Config
-       sw            *p2p.Switch
-       discvCh       chan *dht.Node
-       privKey       chainkd.XPrv
-       chain         *mock.Chain
-       txPool        *mock.Mempool
+       *sync.RWMutex
+       cfg     *config.Config
+       db      *gorm.DB
+       nodeCfg *vaporCfg.Config
+       sw      *p2p.Switch
+       privKey chainkd.XPrv
+       chain   *mock.Chain
+       txPool  *mock.Mempool
+       // discvMap maps a node's public key to the node itself
+       discvMap      map[string]*dht.Node
        dialCh        chan struct{}
        checkStatusCh chan struct{}
-       // peers         *peers.PeerSet
 }
 
 // TODO: set myself as SPV?
@@ -60,7 +57,6 @@ func NewMonitor(cfg *config.Config, db *gorm.DB) *monitor {
        }
        nodeCfg.DBPath = dbPath
        nodeCfg.ChainID = "mainnet"
-       discvCh := make(chan *dht.Node)
        privKey, err := signlib.NewPrivKey()
        if err != nil {
                log.Fatal(err)
@@ -72,14 +68,14 @@ func NewMonitor(cfg *config.Config, db *gorm.DB) *monitor {
        }
 
        return &monitor{
-               // RWMutex:       &sync.RWMutex{},
+               RWMutex:       &sync.RWMutex{},
                cfg:           cfg,
                db:            db,
                nodeCfg:       nodeCfg,
-               discvCh:       discvCh,
                privKey:       privKey.(chainkd.XPrv),
                chain:         chain,
                txPool:        txPool,
+               discvMap:      make(map[string]*dht.Node),
                dialCh:        make(chan struct{}, 1),
                checkStatusCh: make(chan struct{}, 1),
        }
@@ -96,17 +92,10 @@ func makePath() (string, error) {
                return "", err
        }
 
-       tmpDir, err := ioutil.TempDir(dataPath, "")
-       if err != nil {
-               return "", err
-       }
-
-       return tmpDir, nil
+       return dataPath, nil
 }
 
 func (m *monitor) Run() {
-       defer os.RemoveAll(m.nodeCfg.DBPath)
-
        var seeds []string
        for _, node := range m.cfg.Nodes {
                seeds = append(seeds, fmt.Sprintf("%s:%d", node.Host, node.Port))
@@ -121,7 +110,6 @@ func (m *monitor) Run() {
 
        m.dialCh <- struct{}{}
        go m.discoveryRoutine()
-       go m.collectDiscoveredNodes()
        go m.connectNodesRoutine()
        go m.checkStatusRoutine()
 }
@@ -156,7 +144,7 @@ func (m *monitor) prepareReactors(peers *peers.PeerSet) error {
        }
 
        for label, reactor := range m.sw.GetReactors() {
-               log.Debug("start reactor: (%s:%v)", label, reactor)
+               log.Debugf("start reactor: (%s:%v)", label, reactor)
                if _, err := reactor.Start(); err != nil {
                        return nil
                }
@@ -164,7 +152,6 @@ func (m *monitor) prepareReactors(peers *peers.PeerSet) error {
 
        m.sw.GetSecurity().RegisterFilter(m.sw.GetNodeInfo())
        m.sw.GetSecurity().RegisterFilter(m.sw.GetPeers())
-
        return m.sw.GetSecurity().Start()
 }
 
@@ -174,20 +161,14 @@ func (m *monitor) checkStatusRoutine() {
                log.Fatal(err)
        }
 
-       protocolReactor, ok := m.sw.GetReactors()["PROTOCOL"]
-       if !ok {
-               log.Fatal("protocolReactor not found")
-       }
-
        bestHeight := uint64(0)
        for range m.checkStatusCh {
-               // m.Lock()
-
                for _, peer := range m.sw.GetPeers().List() {
                        peer.Start()
-                       protocolReactor.AddPeer(peer)
+                       peers.AddPeer(peer)
                }
                log.Infof("%d connected peers: %v", len(m.sw.GetPeers().List()), m.sw.GetPeers().List())
+
                for _, peer := range m.sw.GetPeers().List() {
                        p := peers.GetPeer(peer.ID())
                        if p == nil {
@@ -204,16 +185,9 @@ func (m *monitor) checkStatusRoutine() {
                        if peerInfo.Height > bestHeight {
                                bestHeight = peerInfo.Height
                        }
-
-                       m.savePeerInfo(peerInfo)
                }
                log.Info("bestHeight: ", bestHeight)
-
-               // TODO:
-               // msg := struct{ msgs.BlockchainMessage }{&msgs.GetBlockMessage{Height: bestHeight + 1}}
-               // for _, peer := range m.sw.GetPeers().List() {
-               //      peers.SendMsg(peer.ID(), msgs.BlockchainChannel, msg)
-               // }
+               m.processPeerInfos(peers.GetPeerInfos())
 
                for _, peer := range m.sw.GetPeers().List() {
                        p := peers.GetPeer(peer.ID())
@@ -224,7 +198,7 @@ func (m *monitor) checkStatusRoutine() {
                        peers.RemovePeer(p.ID())
                }
                log.Info("Disonnect all peers.")
-               // m.Unlock()
+               m.Unlock()
                m.dialCh <- struct{}{}
        }
 }