protect against nils

djeebus · djeebus · commit 835ee09fd8a7 · 2025-10-21T15:02:14.000-07:00
diff --git a/packages/orchestrator/internal/sharedstate/tracker.go b/packages/orchestrator/internal/sharedstate/tracker.go
@@ -36,7 +36,14 @@ func New(selfWriteInterval time.Duration) (*Manager, error) {
 }
 
 func (t *Manager) OnInsert(sandbox *sandbox.Sandbox) {
-	t.selfSandboxResources.Insert(sandbox.Metadata.Runtime.SandboxID, sandbox.Config)
+	metadata := sandbox.Metadata
+	if metadata == nil {
+		zap.L().Warn("Ignoring sandbox without metadata")
+
+		return
+	}
+
+	t.selfSandboxResources.Insert(metadata.Runtime.SandboxID, metadata.Config)
 }
 
 func (t *Manager) OnRemove(sandboxID string) {

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,14 @@ func New(selfWriteInterval time.Duration) (*Manager, error) {`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`func (t Manager) OnInsert(sandbox sandbox.Sandbox) {`
`39`		`- t.selfSandboxResources.Insert(sandbox.Metadata.Runtime.SandboxID, sandbox.Config)`
	`39`	`+ metadata := sandbox.Metadata`
	`40`	`+ if metadata == nil {`
	`41`	`+ zap.L().Warn("Ignoring sandbox without metadata")`
	`42`	`+`
	`43`	`+ return`
	`44`	`+ }`
	`45`	`+`
	`46`	`+ t.selfSandboxResources.Insert(metadata.Runtime.SandboxID, metadata.Config)`
`40`	`47`	`}`
`41`	`48`
`42`	`49`	`func (t *Manager) OnRemove(sandboxID string) {`