fix(sandbox): redefine deletion semantics around VMM stop and resource cleanup-Conch-AtomGit

Closed

fix(sandbox): redefine deletion semantics around VMM stop and resource cleanup #190

ytluoAdd on Sep 2Closed at Sep 11Merged fromytluo/Conch： fix_issue_202 into openeuler/Conch： dev

fix(sandbox): redefine deletion semantics around VMM stop and resource cleanup #190

Closed

ytluoAdd on Sep 2Closed at Sep 11

Merged fromytluo/Conch fix_issue_202intoopeneuler/Conch dev

ytluo

Sep 2

当前PR是否有AI参与:

[x] 否

[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :

参考：openEuler社区《生成式AI工具使用与开源贡献政策》

PR功能描述 / 为什么需要这个合入**:

 修正了沙箱删除流程中的契约边界：
  - VMM 停止失败时终止删除，保留 in-memory Manager 与持久化记录；
  - VMM 停止成功后继续尝试资源清理，无论后续资源是否清理成功都将删除记录，已删除沙箱将不会残留在查询结果中；
  - 如果资源清理失败，则仅记录日志，并不打断上述流程，交由人工维护进行处理；
  - 持久化记录的删除失败返回同样的错误，并不做修改，防止其不一致而被静默忽略；
该修改解决了 issue #202 中 VMM 实际已停止但因后续资源清理失败而残留在 conch sandbox ls 中的问题。

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #
#202

希望检视人员了解:

The current Pull Request has been closed by openeuler-ci-bot.

Y ytluo

Sep 2 Created pull request, commit b88aedc5

openeuler-ci-botCollaborators

Sep 2 Set weibaohui,yangzhao_kl,jimmieme,duguhaotian,wonleing,zmr_2020,gaodaweiky,lu-wei-army,zhaoyifan28 as approver

openeuler-ci-botCollaborators

Sep 2 Added label: sig/sig-CloudNative

openeuler-ci-botCollaborators

Sep 2 Comments：

Welcome To openEuler Community

Hey @ytluo , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands. You can self-configure the PR merge rules for this repository. For more details, please refer to Here.

Contact Guide

If you have any questions, please contact the SIG: sig-CloudNative ,
and any of the maintainers: @duguhaotian, @gaodaweiky, @jimmieme, @lu-wei-army, @weibaohui, @wonleing, @yangzhao_kl, @zhaoyifan28, @zmr_2020 ,
and any of the committers: @Yekelu .

openeuler-ci-botCollaborators

Sep 2 Added label: openeuler-cla/yes

74 messages are collapsed here。 See more

openeuler-ci-botCollaborators

Sep 7 Removed label: openeuler-cla/yes

openeuler-ci-botCollaborators

Sep 7 Added label: openeuler-cla/yes

openeuler-ci-botCollaborators

Sep 7 Comments：

CLA Signature Pass

ytluo, thanks for your pull request. All authors of the commits have signed the CLA. 👍

zhaoyifan28Collaborators

Sep 11 Comments：

/close

openeuler-ci-botCollaborators

Sep 11 Close pull request