go/runtime/bundle: Don't abort when some manifests cannot be loaded

When upgrading from earlier versions the exploded bundle directory may contain non-bundle subdirectories which should not cause the loader to fail.
oasisprotocol · Feb 7, 2025 · 7a3d8fc · 7a3d8fc
1 parent 6609237
commit 7a3d8fc
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/.changelog/6044.trivial.md b/.changelog/6044.trivial.md
diff --git a/go/runtime/bundle/manager.go b/go/runtime/bundle/manager.go
@@ -508,12 +508,20 @@ func (m *Manager) loadManifests() ([]*ExplodedManifest, error) {
 
 		b, err := os.ReadFile(filepath.Join(dir, manifestName))
 		if err != nil {
-			return nil, fmt.Errorf("failed to read manifest: %w", err)
+			m.logger.Warn("skipping unreadable manifest",
+				"path", dir,
+				"err", err,
+			)
+			continue
 		}
 
 		var manifest Manifest
 		if err = json.Unmarshal(b, &manifest); err != nil {
-			return nil, fmt.Errorf("failed to parse manifest: %w", err)
+			m.logger.Warn("skipping malformed manifest",
+				"path", dir,
+				"err", err,
+			)
+			continue
 		}
 
 		m.logger.Info("manifest loaded",