kvserver: partially clarify StoreIdentKey

Epic: none
Release note: none

Author: pav-kv

pkg/cli/debug_recover_loss_of_quorum.go

@@ -825,6 +825,7 @@ func applyRecoveryToLocalStore(
 		defer store.Close() //nolint:deferloop
 		defer batch.Close() //nolint:deferloop
 
+		// TODO(sep-raft-log): StoreIdent is in the LogEngine.
 		storeIdent, err := kvstorage.ReadStoreIdent(ctx, store)
 		if err != nil {
 			return err
@@ -844,6 +845,7 @@ func applyRecoveryToLocalStore(
 	}
 
 	updateTime := timeutil.Now()
+	// TODO(sep-raft-log): batches need to work with the split log/state engines.
 	prepReport, err := loqrecovery.PrepareUpdateReplicas(
 		ctx, nodeUpdates, uuid.DefaultGenerator, updateTime, localNodeID, batches)
 	if err != nil {

pkg/kv/kvserver/store.go

@@ -2147,8 +2147,7 @@ func (s *Store) Start(ctx context.Context, stopper *stop.Stopper) error {
 
 	// Populate the store ident. If not bootstrapped, ReadStoreIntent will
 	// return an error.
-	// TODO(sep-raft-log): which engine holds the ident?
-	ident, err := kvstorage.ReadStoreIdent(ctx, s.TODOEngine())
+	ident, err := kvstorage.ReadStoreIdent(ctx, s.LogEngine())
 	if err != nil {
 		return err
 	}