feat(provider): info log AcceleratedDHTClient crawl (#11143)

* feat(provider): log fullrt crawl
* fix(provider): improve AcceleratedDHTClient log messages

use config option name instead of internal "fullrt" jargon,
align terminology with daemon startup message, add time estimate

---------

Co-authored-by: Marcin Rataj <lidel@lidel.org>

core/node/provider.go

@@ -16,6 +16,7 @@ import (
 	"github.com/ipfs/go-datastore"
 	"github.com/ipfs/go-datastore/namespace"
 	"github.com/ipfs/go-datastore/query"
+	log "github.com/ipfs/go-log/v2"
 	"github.com/ipfs/kubo/config"
 	"github.com/ipfs/kubo/repo"
 	irouting "github.com/ipfs/kubo/routing"
@@ -53,6 +54,8 @@ const (
 	keystoreDatastorePath = "keystore"
 )
 
+var errAcceleratedDHTNotReady = errors.New("AcceleratedDHTClient: routing table not ready")
+
 // Interval between reprovide queue monitoring checks for slow reprovide alerts.
 // Used when Provide.DHT.SweepEnabled=true
 const reprovideAlertPollInterval = 15 * time.Minute
@@ -325,13 +328,34 @@ type dhtImpl interface {
 
 type fullrtRouter struct {
 	*fullrt.FullRT
+	ready  bool
+	logger *log.ZapEventLogger
+}
+
+func newFullRTRouter(fr *fullrt.FullRT, loggerName string) *fullrtRouter {
+	return &fullrtRouter{
+		FullRT: fr,
+		ready:  true,
+		logger: log.Logger(loggerName),
+	}
 }
 
 // GetClosestPeers overrides fullrt.FullRT's GetClosestPeers and returns an
 // error if the fullrt's initial network crawl isn't complete yet.
 func (fr *fullrtRouter) GetClosestPeers(ctx context.Context, key string) ([]peer.ID, error) {
-	if !fr.Ready() {
-		return nil, errors.New("fullrt: initial network crawl still running")
+	if fr.ready {
+		if !fr.Ready() {
+			fr.ready = false
+			fr.logger.Info("AcceleratedDHTClient: waiting for routing table initialization (5-10 min, depends on DHT size and network) to complete before providing")
+			return nil, errAcceleratedDHTNotReady
+		}
+	} else {
+		if fr.Ready() {
+			fr.ready = true
+			fr.logger.Info("AcceleratedDHTClient: routing table ready, providing can begin")
+		} else {
+			return nil, errAcceleratedDHTNotReady
+		}
 	}
 	return fr.FullRT.GetClosestPeers(ctx, key)
 }
@@ -382,6 +406,9 @@ func SweepingProviderOpt(cfg *config.Config) fx.Option {
 			// provides happen as fast as possible via a dedicated worker that continuously
 			// processes the queue regardless of this timing.
 			bufferedIdleWriteTime = time.Minute
+
+			// loggerName is the name of the go-log logger used by the provider.
+			loggerName = dhtprovider.DefaultLoggerName
 		)
 
 		bufferedProviderOpts := []buffered.Option{
@@ -411,6 +438,8 @@ func SweepingProviderOpt(cfg *config.Config) fx.Option {
 					ddhtprovider.WithDedicatedPeriodicWorkers(int(cfg.Provide.DHT.DedicatedPeriodicWorkers.WithDefault(config.DefaultProvideDHTDedicatedPeriodicWorkers))),
 					ddhtprovider.WithDedicatedBurstWorkers(int(cfg.Provide.DHT.DedicatedBurstWorkers.WithDefault(config.DefaultProvideDHTDedicatedBurstWorkers))),
 					ddhtprovider.WithMaxProvideConnsPerWorker(int(cfg.Provide.DHT.MaxProvideConnsPerWorker.WithDefault(config.DefaultProvideDHTMaxProvideConnsPerWorker))),
+
+					ddhtprovider.WithLoggerName(loggerName),
 				)
 				if err != nil {
 					return nil, nil, err
@@ -419,7 +448,7 @@ func SweepingProviderOpt(cfg *config.Config) fx.Option {
 			}
 		case *fullrt.FullRT:
 			if inDht != nil {
-				impl = &fullrtRouter{inDht}
+				impl = newFullRTRouter(inDht, loggerName)
 			}
 		}
 		if impl == nil {
@@ -454,6 +483,8 @@ func SweepingProviderOpt(cfg *config.Config) fx.Option {
 			dhtprovider.WithDedicatedPeriodicWorkers(int(cfg.Provide.DHT.DedicatedPeriodicWorkers.WithDefault(config.DefaultProvideDHTDedicatedPeriodicWorkers))),
 			dhtprovider.WithDedicatedBurstWorkers(int(cfg.Provide.DHT.DedicatedBurstWorkers.WithDefault(config.DefaultProvideDHTDedicatedBurstWorkers))),
 			dhtprovider.WithMaxProvideConnsPerWorker(int(cfg.Provide.DHT.MaxProvideConnsPerWorker.WithDefault(config.DefaultProvideDHTMaxProvideConnsPerWorker))),
+
+			dhtprovider.WithLoggerName(loggerName),
 		}
 
 		prov, err := dhtprovider.New(opts...)