feat: 更新前后调用ostree的backup和deploy

[fly602]

更新前后调用ostree的backup和deploy

Log: 更新前后调用ostree的backup和deploy
pms:

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 常量定义：

   • 在manager_upgrade.go文件中新增了DEEPIN_IMMUTABLE_CTL常量，这是一个好的做法，因为它提高了代码的可读性和可维护性。

2. 错误处理：

   • 在distUpgrade函数中，当osTreeDeploy失败时，错误被记录为警告日志，而不是返回一个system.JobError。这可能不是最佳实践，因为调用者可能需要根据错误类型采取不同的行动。建议返回错误以便调用者处理。
   • 在preRunningHook函数中，当osTreeBackUp失败时，错误被记录为警告日志，并且状态被设置为BackupFailed。这可能会导致更新流程中断，建议在失败时返回错误，并允许调用者决定如何处理。

3. 日志记录：

   • 在osTreeCmd函数中，当DEEPIN_IMMUTABLE_CTL不存在时，记录了一个警告日志。这是一个好的做法，因为它可以帮助开发者了解问题所在。但是，如果这个函数被频繁调用，可能会产生大量的日志，建议添加一个条件来限制日志的频率。

4. 代码重复：

   • osTreeBackUp和osTreeDeploy函数中存在代码重复，建议将公共部分提取到一个单独的函数中，以减少代码重复和提高可维护性。

5. 错误信息：

   • 在osTreeCmd函数中，当命令执行失败时，错误信息被格式化为%v，这可能会导致错误信息不清晰。建议使用更具体的格式化字符串来提供更详细的错误信息。

6. 注释：

   • 在preRunningHook函数中，注释TODO: 当前控制中心只会调用distupgrade，且是全量更新，每次更新都进行备份可能不再准确，因为代码已经修改为支持部分更新。建议更新注释以反映最新的实现。

7. 状态管理：

   • 在preRunningHook函数中，状态管理逻辑可能需要进一步优化，以确保状态的一致性和正确性。例如，如果osTreeBackUp失败，应该立即停止更新流程，而不是继续执行后续操作。

8. 代码风格：

   • 在osTreeCmd函数中，cmd.Stdout和cmd.Stderr的缓冲区应该使用os.Stdout和os.Stderr，而不是bytes.Buffer。这是因为bytes.Buffer不会将输出显示给用户，而os.Stdout和os.Stderr会将输出显示给用户。

综上所述，代码在错误处理、日志记录、代码重复、错误信息、注释、状态管理和代码风格方面存在一些改进空间。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602, robertkill

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment