docs: propose precise sandbox lifecycle states#653
Conversation
Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #653 +/- ##
==========================================
- Coverage 80.06% 80.05% -0.02%
==========================================
Files 229 229
Lines 17861 17862 +1
==========================================
- Hits 14300 14299 -1
- Misses 2976 2978 +2
Partials 585 585
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
Signed-off-by: AiRanthem <zhongtianyun.zty@alibaba-inc.com>
What changed
Adds the technical proposal for replacing the derived five-state Sandbox model with an eleven-state lifecycle through three ordered changes:
Why
The current
deadstate conflates completion, failure, controller progress, and temporary unavailability. This can produce false 404s, premature route deletion, and duplicated CRD-condition interpretation in sandbox-manager.Review focus
Impact
Documentation only. No code, CRD schema, generated artifact, or public state enum changes are included in this PR.
Validation
feature/extend-states-260715: 3/3 changes pass strict validation.git diff --checkpasses.