go/worker/storage: Disable storage worker when no runtimes configured

oasisprotocol · Oct 11, 2023 · 3edf6f3 · 3edf6f3
1 parent f83b69d
commit 3edf6f3
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/.changelog/5394.bugfix.md b/.changelog/5394.bugfix.md
@@ -0,0 +1 @@
+go/worker/storage: Disable storage worker when no runtimes configured
diff --git a/go/config/config.go b/go/config/config.go
@@ -56,7 +56,7 @@ func (m NodeMode) IsClientOnly() bool {
 // HasLocalStorage returns true iff the mode is one that has local storage.
 func (m NodeMode) HasLocalStorage() bool {
 	switch m {
-	case ModeClient, ModeCompute:
+	case ModeClient, ModeCompute, ModeArchive:
 		return true
 	}
 	return false

diff --git a/go/worker/storage/worker.go b/go/worker/storage/worker.go
@@ -38,10 +38,7 @@ func New(
 	commonWorker *workerCommon.Worker,
 	registration *registration.Worker,
 ) (*Worker, error) {
-	enabled := config.GlobalConfig.Mode.HasLocalStorage()
-	if config.GlobalConfig.Mode == config.ModeArchive && len(commonWorker.GetRuntimes()) > 0 {
-		enabled = true
-	}
+	enabled := config.GlobalConfig.Mode.HasLocalStorage() && len(commonWorker.GetRuntimes()) > 0
 
 	s := &Worker{
 		enabled:      enabled,
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		go/worker/storage: Disable storage worker when no runtimes configured