Merge pull request #5361 from oasisprotocol/kostko/fix/rt-status-failed

go/worker/common: Reset runtime provisioned status on failure
oasisprotocol · Sep 11, 2023 · 1c00e97 · 1c00e97
2 parents 6925b49 + a012888
commit 1c00e97
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.changelog/5361.bugfix.md b/.changelog/5361.bugfix.md
@@ -0,0 +1 @@
+go/worker/common: Reset runtime provisioned status on failure
diff --git a/go/worker/common/committee/node.go b/go/worker/common/committee/node.go
@@ -604,9 +604,13 @@ func (n *Node) handleNewEventLocked(ev *roothash.Event) {
 
 // Guarded by n.CrossNode.
 func (n *Node) handleRuntimeHostEventLocked(ev *host.Event) {
-	if ev.Started != nil {
+	switch {
+	case ev.Started != nil:
 		atomic.StoreUint32(&n.hostedRuntimeProvisioned, 1)
+	case ev.FailedToStart != nil, ev.Stopped != nil:
+		atomic.StoreUint32(&n.hostedRuntimeProvisioned, 0)
 	}
+
 	for _, hooks := range n.hooks {
 		hooks.HandleRuntimeHostEventLocked(ev)
 	}
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		go/worker/common: Reset runtime provisioned status on failure