simplify wal defaulting logic, fix docs

agent/consul/server.go

@@ -1055,18 +1055,23 @@ func (s *Server) setupRaft() error {
 			stable = wal
 			return nil
 		}
-		// Only use WAL if there is no existing raft.db, even if it's enabled.
-		if (s.config.LogStoreConfig.Backend == LogStoreBackendDefault || s.config.LogStoreConfig.Backend == LogStoreBackendWAL) && !boltFileExists {
+
+		// Default to WAL if there is no existing raft.db, even if it's enabled. Log a warning otherwise
+		if s.config.LogStoreConfig.Backend == LogStoreBackendDefault && !boltFileExists {
 			s.config.LogStoreConfig.Backend = LogStoreBackendWAL
-			if err = initWAL(); err != nil {
-				return err
-			}
-		} else {
-			if s.config.LogStoreConfig.Backend == LogStoreBackendWAL || s.config.LogStoreConfig.Backend == LogStoreBackendDefault {
+		} else if s.config.LogStoreConfig.Backend == LogStoreBackendWAL || s.config.LogStoreConfig.Backend == LogStoreBackendDefault {
 			// User configured the new storage, but still has old raft.db. Warn
 			// them!
 			s.logger.Warn("BoltDB file raft.db found, IGNORING raft_logstore.backend which is set to 'wal'")
 		}
+
+		if s.config.LogStoreConfig.Backend == LogStoreBackendWAL && !boltFileExists {
+			s.config.LogStoreConfig.Backend = LogStoreBackendWAL
+			if err = initWAL(); err != nil {
+				return err
+			}
+		} else {
+
 			s.config.LogStoreConfig.Backend = LogStoreBackendBoltDB
 			// Create the backend raft store for logs and stable storage.
 			store, err := raftboltdb.New(raftboltdb.Options{

website/content/docs/agent/wal-logstore/enable.mdx

@@ -7,7 +7,7 @@ description: >-
 
 # Enable the WAL LogStore backend
 
-WAL LogStore is now enabled by default
+WAL LogStore is now enabled by default unless a BoltDB database already exist.
 
 ## Known issues