code review updates

Signed-off-by: Merlin Ran <merlinran@gmail.com>

main.go

@@ -135,9 +135,10 @@ Also take the file system overhead into consideration when calculating the limit
 No limit by default.`,
 		},
 		{
-			Name:        "sealing-sectors-limit",
-			DefValue:    0,
-			Description: "If set, stop bidding if the number of Lotus sealing sectors exceeds this limit.",
+			Name:     "sealing-sectors-limit",
+			DefValue: 0,
+			Description: `If bigger than zero, stop bidding if the number of Lotus sealing sectors exceeds this limit.
+Zero means no limits`,
 		},
 		{
 			Name:        "deal-data-fetch-attempts",

service/lotusclient/lotusclient.go

@@ -105,9 +105,11 @@ func (c *Client) CurrentSealingSectors() (int, error) {
 
 	ctx, cancel := context.WithTimeout(c.ctx, requestTimeout)
 	defer cancel()
+	start := time.Now()
 	sectorsByState, err := c.c.SectorsSummary(ctx)
+	log.Debugf("SectorsSummary() call took %.2f seconds", time.Since(start).Seconds())
 	if err != nil {
-		return 0, err
+		return 0, fmt.Errorf("getting lotus sectors summary: %s", err)
 	}
 	total := 0
 	for _, i := range sectorsByState {

service/service.go

@@ -378,7 +378,7 @@ func (s *Service) makeBid(a *pb.Auction, from core.ID) error {
 		if err != nil {
 			return fmt.Errorf("getting number of sealing sectors: %v", err)
 		}
-		if n >= s.sealingSectorsLimit {
+		if n > s.sealingSectorsLimit {
 			log.Infof("not bidding: lotus already have %d sealing sectors", n)
 			return nil
 		}