Harvester升级控制器在等待镜像差异列表时点击忽略导致清理跳过的技术分析

问题背景

在Harvester系统的升级过程中，升级控制器负责管理整个升级流程，包括准备升级环境、执行升级操作以及最后的清理工作。其中，清理阶段涉及删除不再需要的镜像资源，这对系统资源的回收至关重要。

问题现象

当升级过程进入清理阶段时，控制器会调用cleanupImages函数，该函数通过retry.OnError机制尝试获取当前版本与目标版本的镜像差异列表。在此期间，系统日志会显示类似"Trying to get 98a6e4ae image list"的信息，表明系统正在尝试获取镜像列表。

此时，用户界面会显示"忽略"按钮，因为升级的完成条件harvesterv1.UpgradeCompleted.IsTrue(upgrade)已经满足。如果用户在此阶段点击"忽略"，系统会重置升级状态，导致清理流程被意外中断。

技术原理分析

升级控制器的清理流程设计如下：

1. 首先尝试获取镜像差异列表
2. 无论成功获取列表还是超时失败，都会进入正常的清理路径
3. 清理完成后，系统会标记升级为完成状态

问题出现的根本原因是状态管理的不一致性：

• 前端界面根据升级完成条件显示"忽略"按钮
• 后端控制器仍在执行清理操作
• 用户操作"忽略"会重置升级状态，打断了正在进行的清理流程

影响范围

该问题会导致以下不良后果：

1. 升级日志对象(upgradeLog)未被正确清理，残留系统中
2. 其他相关资源可能处于非预期状态
3. 系统资源未能完全回收，可能影响后续操作

解决方案

该问题已通过修复"忽略"功能得到解决。具体改进包括：

1. 确保升级状态重置不会重新触发状态机
2. 保证清理流程在任何情况下都能完整执行
3. 加强前后端状态同步机制

最佳实践建议

对于系统升级操作，建议：

1. 在升级过程中避免进行管理操作，特别是清理阶段
2. 监控升级日志，确认所有流程完成后再进行其他操作
3. 对于长时间处于"获取镜像列表"的状态，应先检查系统日志而非直接中断

总结

Harvester升级控制器的这一问题展示了分布式系统中状态管理的重要性。通过修复前后端交互逻辑，确保了升级流程的完整性和可靠性。这为类似系统的状态机设计提供了有价值的参考案例。