Skip to content

Commit 609d764

Browse files
committed
error handling
1 parent cb41fe2 commit 609d764

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

docs/pod-sandbox-lifecycle.md

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ Plugins subscribe to these events during the Configure phase by returning the ap
134134
- `Event_STOP_POD_SANDBOX` (1 << 1) for StopPodSandbox
135135
- `Event_REMOVE_POD_SANDBOX` (1 << 2) for RemovePodSandbox
136136

137-
These events are notified via the StateChange RPC call.
137+
These events are delivered to plugins using the RunPodSandbox, StopPodSandbox and RemovePodSandbox event handlers.
138138

139139
### Timeout Handling
140140

@@ -146,11 +146,12 @@ All plugin processing must complete within the configured request timeout. Plugi
146146

147147
### Error Handling
148148

149-
Plugins should handle errors gracefully and avoid leaving the pod or system in an inconsistent state. Error recovery strategies:
149+
On the teardown path, plugin errors MUST NOT prevent the operation from proceeding. Runtimes MUST ensure that a failing plugin cannot block pod or container teardown:
150+
151+
- **RunPodSandbox errors**: A plugin error may prevent the pod from being created, depending on runtime policy. Plugins bear responsibility for errors they return at this phase.
152+
- **StopPodSandbox errors**: A plugin error MUST NOT prevent the sandbox from being stopped. The runtime MUST proceed with teardown regardless of plugin failures.
153+
- **RemovePodSandbox errors**: A plugin error MUST NOT prevent the sandbox from being removed. The runtime MUST proceed with removal regardless of plugin failures.
150154

151-
- **RunPodSandbox errors**: Problematic; may block pod creation depending on failure severity and runtime policy
152-
- **StopPodSandbox errors**: May not prevent scenario termination depending on runtime policy
153-
- **RemovePodSandbox errors**: Should not prevent sandbox removal
154155

155156
### Multi-Plugin Coordination
156157

0 commit comments

Comments
 (0)