Harvester项目中的存储类默认设置冲突问题分析与解决方案

问题背景

在Harvester v1.4.0至v1.4.1-rc1版本的升级过程中，部分用户遇到了一个与存储类(StorageClass)相关的升级障碍。具体表现为当集群中存在非默认存储类设置时，升级过程会被webhook拦截，提示"default storage class already exists"错误。

问题现象

在3节点见证(witness)集群环境中，当用户创建了一个自定义存储类(如名为sc2的存储类)并将其设置为默认存储类后，尝试执行Harvester版本升级时会遇到以下错误：

admission webhook "validator.harvesterhci.io" denied the request: managed chart harvester is not ready, please wait for it to be ready

进一步检查ManagedChart状态时，会发现更详细的错误信息：

cannot patch "harvester-longhorn" with kind StorageClass: admission webhook "validator.harvesterhci.io" denied the request: default storage class %!s(MISSING) already exists, please reset it first

根本原因分析

经过深入调查，发现该问题由以下几个因素共同导致：

1. Harvester chart的默认行为：Harvester chart在部署时会自动将harvester-longhorn存储类标记为默认(default)存储类，通过设置storageclass.kubernetes.io/is-default-class: "true"注解。

2. 集群扩展时的自动部署：当集群处于扩展状态(如添加新节点)时，Harvester相关部署会自动扩展，这是由于management.cattle.io/scale-available注解的存在。

3. webhook验证冲突：当用户已经设置了其他存储类为默认存储类，而Harvester managedchart尝试重新部署并更新变更时，webhook会拦截这种冲突操作，导致升级失败。

解决方案

针对这一问题，开发团队提出了以下解决方案：

1. chart模板条件增强：在chart定义中添加更多条件判断，使用Helm的lookup功能检查是否已有其他存储类设置了默认注解。

2. 控制器逻辑优化：当任何非harvester-longhorn存储类被设置为默认时，Harvester控制器应自动将managedchart中的spec.values.storageClass.defaultStorageClass字段设置为false，避免冲突。

3. 临时解决方案：在升级前手动编辑managedchart，将spec.values.storageClass.defaultStorageClass设置为false。

技术实现细节

修复方案主要包含以下技术实现：

1. Helm模板条件判断：在harvester-storageclass.yaml模板中添加条件判断，确保只有在没有其他默认存储类时才设置harvester-longhorn为默认。

2. webhook验证逻辑优化：修改webhook验证逻辑，正确处理存储类默认状态的变更请求。

3. 超时设置修复：确保所有webhook配置都设置了适当的timeoutSeconds值，避免因超时导致的升级失败。

验证方案

为确保修复效果，建议进行以下验证步骤：

1. 创建一个新存储类并将其设置为默认存储类
2. 观察managedchart状态，确认没有出现存储类冲突错误
3. 执行版本升级操作，验证升级过程顺利完成
4. 检查升级后集群状态，确认所有组件正常运行

总结

Harvester项目中存储类默认设置的冲突问题是一个典型的配置管理挑战。通过增强chart的条件判断、优化控制器逻辑和webhook验证机制，开发团队有效解决了这一问题。这一修复不仅解决了当前版本的升级障碍，也为后续版本提供了更健壮的存储类管理机制。

对于正在使用v1.4.0版本且遇到此问题的用户，可以按照提供的临时解决方案进行操作，或者等待包含完整修复的v1.4.2版本发布。