<fix>[migration]: fix failed to start vm after ceph to ceph offline migration.

[ZStack-Robot]

support starting vm without nic in other clusters.

Resolves/Related: ZSTAC-80468

Change-Id: I6370646f7a796265677a6a656c716f6867706d69

sync from gitlab !8933

[coderabbitai]

概览

在 VmInstanceBase.java 的 getStartingCandidateHosts 方法中，构建 DesignatedAllocateHostMsg 时新增了 setAllowNoL3Networks(true) 配置，允许在主机分配干运行中考虑未配置 L3 网络的设计方案。

变更

组件 / 文件	变更摘要
虚拟机分配配置 compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java	在 getStartingCandidateHosts 方法中的 DesignatedAllocateHostMsg 构建阶段，新增 setAllowNoL3Networks(true) 调用，扩展允许不含 L3 网络配置的主机候选设计的处理范围

估算代码审查工作量

🎯 2 (简单) | ⏱️ ~5 分钟

• 需要确认 setAllowNoL3Networks(true) 的行为变更对现有主机分配逻辑的影响范围
• 验证该配置是否可能导致选择不符合网络需求的主机

诗

一行代码悄悄改，
网络约束轻轻拜，
主机选择更灵活，
设计方案展新彩，🐰
分配之路宽且快！

Pre-merge checks and finishing touches

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error, 1 warning)

Check name	Status	Explanation	Resolution
Title check	❌ Error	标题超过72字符限制（78字符），不符合格式要求。	缩短标题至72字符以内。建议改为：[migration]: Fix VM start after ceph offline migration.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	描述内容与变更集相关，涉及VM启动和集群迁移，符合PR的实际修改内容。

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sync/shan.wu/fix-ZSTAC-80468@@2

---

📜 Recent review details

Configuration used: Path: http://open.zstack.ai:20001/code-reviews/zstack-cloud.yaml (via .coderabbit.yaml)

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2241420 and 3547791.

📒 Files selected for processing (1)

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

🧰 Additional context used

📓 Path-based instructions (2)

**/*.*

⚙️ CodeRabbit configuration file

**/*.*: - 代码里不应当有有中文，包括报错、注释等都应当使用正确的、无拼写错误的英文来写

Files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

**/*.java

⚙️ CodeRabbit configuration file

**/*.java: ## 1. API 设计要求

• API 命名：
  • API 名称必须唯一，不能重复。
  • API 消息类需要继承 APIMessage；其返回类必须继承 APIReply 或 APIEvent，并在注释中用 @RestResponse 进行标注。
  • API 消息上必须添加注解 @RestRequest，并满足如下规范：
    • path:
      • 针对资源使用复数形式。
      • 当 path 中引用消息类变量时，使用 {variableName} 格式。
    • HTTP 方法对应：
      • 查询操作 → HttpMethod.GET
      • 更新操作 → HttpMethod.PUT
      • 创建操作 → HttpMethod.POST
      • 删除操作 → HttpMethod.DELETE
  • API 类需要实现 __example__ 方法以便生成 API 文档，并确保生成对应的 Groovy API Template 与 API Markdown 文件。

---

2. 命名与格式规范

• 类名：

  • 使用 UpperCamelCase 风格。
  • 特殊情况：
    • VO/AO/EO 类型类除外。
    • 抽象类采用 Abstract 或 Base 前缀/后缀。
    • 异常类应以 Exception 结尾。
    • 测试类需要以 Test 或 Case 结尾。

• 方法名、参数名、成员变量和局部变量：

  • 使用 lowerCamelCase 风格。

• 常量命名：

  • 全部大写，使用下划线分隔单词。
  • 要求表达清楚，避免使用含糊或不准确的名称。

• 包名：

  • 统一使用小写，使用点分隔符，每个部分应是一个具有自然语义的英文单词（参考 Spring 框架的结构）。

• 命名细节：

  • 避免在父子类或同一代码块中出现相同名字的成员或局部变量，防止混淆。
  • 命名缩写：
    • 不允许使用不必要的缩写，如：AbsSchedulerJob、condi、Fu 等。应使用完整单词提升可读性。

---

3. 编写自解释代码

• 意图表达：

  • 避免使用布尔型参数造成含义不明确。例如：
    • 对于 stopAgent(boolean ignoreError)，建议拆分为不同函数（如 stopAgentIgnoreError()），或使用枚举表达操作类型。
  • 命名应尽量用完整的单词组合表达意图，并在名称中体现数据类型或用途（例如在常量与变量名称中，将类型词放在末尾）。
  • 避免使用魔法值（Magic Value）：

    • 直接使用未经定义的数值或字符串（如 if (status == 5)）应替换为枚举或常量。

    • 示例：

    • // 错误示例：魔法值

    • if (user.getStatus() == 5) { ... }

    • // 正确示例：常量或枚举

    • public static final int STATUS_ACTIVE = 5;

    • if (user.getStatus() == STATUS_ACTIVE) { ... }

    • // 或使用枚举

    • enum UserStatus { ACTIVE, INACTIVE }

• 注释：

  • 代码应尽量做到自解释，对少于两行的说明可以直接写在代码中。
  • 对于较长的注释，需要仔细校对并随代码更新，确保内容正确。
  • 接口方法不应有多余的修饰符（例如 public），且必须配有有效的 Javadoc 注释。

---

4. 流程控制和结构优化

• if...else 的使用：

  • 应尽量减少 if...else 结构的使用，建议：
    • 限制嵌套层级最多为两层，且内层不应再出现 else 分支。
    • 尽早返回（Early Return），将条件判断中的处理逻辑提前结束或抽成独立方法。
    • 使用 Java Stream 或 Lambda 表达式代替冗长的循环与条件判断。

• 条件判断：

  • if 条件表达不宜过长或过于复杂，必要时可以将条件抽成 boolean 变量描述。

• 代码块长度：
  ...

Files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

🧠 Learnings (7)

📓 Common learnings

Learnt from: zstack-robot-1
Repo: MatheMatrix/zstack PR: 2360
File: network/src/main/java/org/zstack/network/l3/L3BasicNetwork.java:449-490
Timestamp: 2025-08-04T04:48:19.103Z
Learning: ZStack项目在cherry-pick操作中，即使发现了性能优化机会（如IP地址批量保存的内存优化），也严格遵循不做额外修改的政策，优先保证cherry-pick的完整性和一致性。

Learnt from: ZStack-Robot
Repo: MatheMatrix/zstack PR: 2489
File: storage/src/main/java/org/zstack/storage/snapshot/VolumeTree.java:471-545
Timestamp: 2025-08-22T05:36:40.467Z
Learning: In ZStack项目中，对于核心逻辑相关的代码，团队倾向于暂时不进行重构，即使代码复杂度较高，优先保证核心功能的稳定性。

Learnt from: ZStack-Robot
Repo: MatheMatrix/zstack PR: 2325
File: compute/src/main/java/org/zstack/compute/vm/VmMigrateCallExtensionFlow.java:29-29
Timestamp: 2025-07-24T05:53:10.246Z
Learning: 在ZStack项目的重构过程中，团队采用务实的优先级策略：优先完成影响运行时功能的代码重构，对于注释中的旧引用等非功能性问题可以延后处理，因为这些不会影响系统的实际运行。

Learnt from: zstack-robot-1
Repo: MatheMatrix/zstack PR: 2360
File: compute/src/main/java/org/zstack/compute/vm/StaticIpOperator.java:364-501
Timestamp: 2025-08-04T03:19:07.454Z
Learning: In ZStack project, developers may prefer to keep related validation logic together in a single method (like validateStaticIpTagsInApiMessage with 130+ lines) when the architecture is clear and there's no code reuse, rather than splitting into smaller methods based purely on single responsibility principle.

Learnt from: ZStack-Robot
Repo: MatheMatrix/zstack PR: 2419
File: plugin/sdnController/src/main/java/org/zstack/sdnController/SdnControllerBase.java:986-1067
Timestamp: 2025-08-13T02:31:40.023Z
Learning: 在ZStack项目中，团队倾向于保持统一的业务流程方法不进行拆分，即使方法较长也会维持在单个方法中以保持业务逻辑的完整性。

Learnt from: zstack-robot-2
Repo: MatheMatrix/zstack PR: 2496
File: plugin/sharedMountPointPrimaryStorage/src/main/java/org/zstack/storage/primary/smp/KvmBackend.java:2545-2566
Timestamp: 2025-08-24T07:35:42.793Z
Learning: 在 ZStack 代码库中，当响应类包含数值字段（如 size、actualSize）时，优先使用原始类型（long）而不是包装类型（Long），以避免 NPE 风险和不必要的装箱/拆箱操作。如果 Agent 端可能不设置该字段，应在 Agent 端确保设置默认值，而不是在使用端做 null 检查。

📚 Learning: 2025-08-12T05:39:14.846Z

Learnt from: ZStack-Robot
Repo: MatheMatrix/zstack PR: 2418
File: compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java:3856-3865
Timestamp: 2025-08-12T05:39:14.846Z
Learning: 在 compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java 的 setNoIpAllocationStaticIp 流程中，从数据库加载的 VmNicVO 实例其 getUsedIps() 为空时也不会为 null（返回空集合）。因此无需对 getUsedIps() 再做 Optional/空列表归一化的空指针保护；若找不到对应 NIC，使用 orElse(new VmNicVO()) 的约定允许后续逻辑通过 NPE 暴露问题，与项目既有约定一致。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

📚 Learning: 2025-08-12T05:52:18.323Z

Learnt from: MatheMatrix
Repo: MatheMatrix/zstack PR: 2418
File: compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java:3873-3885
Timestamp: 2025-08-12T05:52:18.323Z
Learning: 在 compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java 的 setNoIpAllocationStaticIp() 流程中，静态 IP 标签删除已统一在 change-ip-in-database 步骤对 voRemoveSet 循环处理：对收集到的旧 IPv4/IPv6 UsedIpVO 均调用 ipOperator.deleteStaticIpByVmUuidAndL3Uuid(vmUuid, l3Uuid, IPv6NetworkUtils.ipv6AddessToTagValue(ip)) 清理对应的 static-ip 标签。因此无需在分支里分别删除 IPv4/IPv6 的标签。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

📚 Learning: 2025-10-09T02:23:30.305Z

Learnt from: MatheMatrix
Repo: MatheMatrix/zstack PR: 2709
File: compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java:5142-5142
Timestamp: 2025-10-09T02:23:30.305Z
Learning: 在 compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java 的 getAttachableL3Network() 逻辑中，项目约束是：同一 VmNic 可能同时存在两条 UsedIpVO，但它们只代表同一 L3 的 IPv4/IPv6 双栈，UsedIpVO.l3NetworkUuid 与 VmNicVO.l3NetworkUuid 保持一致；因此使用 VmNicVO_.l3NetworkUuid 列表判断 IPAM 关闭网络是否已附加是正确的，无需通过 UsedIpVO 或 VmNicHelper.getL3Uuids() 额外聚合。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

📚 Learning: 2025-09-02T02:18:34.660Z

Learnt from: MatheMatrix
Repo: MatheMatrix/zstack PR: 2549
File: network/src/main/java/org/zstack/network/l3/L3NetworkApiInterceptor.java:553-567
Timestamp: 2025-09-02T02:18:34.660Z
Learning: ZStack 中 enableIPAM=false 的 L3 网络在系统设计上不会存在 ipRangeUuid。这通过多层约束实现：1) API 拦截器层面 - APIAddIpRangeMsg 和 APIAddIpRangeByNetworkCidrMsg 都会在 enableIPAM=false 时抛出异常阻止 IP 范围添加；2) Manager 层面 - L3NetworkManagerImpl 的 SQL 查询使用 enableIPAM=true 过滤条件。因此在 APIGetIpAddressCapacityMsg 验证中仅需 L3NetworkUuids 过滤，无需额外的 ipRangeUuids IPAM 检查。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

📚 Learning: 2025-04-21T03:16:40.427Z

Learnt from: MatheMatrix
Repo: MatheMatrix/zstack PR: 2024
File: network/src/main/java/org/zstack/network/service/DnsUtils.java:53-77
Timestamp: 2025-04-21T03:16:40.427Z
Learning: DnsUtils.getVmDnsListOnL3() 方法中不需要考虑 ipVersion 参数为 null 的情况，此参数在调用时总是有值。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

📚 Learning: 2025-09-02T02:18:34.660Z

Learnt from: MatheMatrix
Repo: MatheMatrix/zstack PR: 2549
File: network/src/main/java/org/zstack/network/l3/L3NetworkApiInterceptor.java:553-567
Timestamp: 2025-09-02T02:18:34.660Z
Learning: 在 ZStack 的 L3NetworkApiInterceptor 中，enableIPAM=false 的 L3 网络不会有关联的 ipRangeUuid，因为在 validate(APIAddIpRangeMsg) 和 validate(APIAddIpRangeByNetworkCidrMsg) 方法中有业务逻辑强制约束，当 enableIPAM=false 时会抛出异常阻止 IP 范围的添加。因此在 APIGetIpAddressCapacityMsg 验证中不需要对 ipRangeUuids 进行额外的 IPAM 过滤。

Applied to files:

• compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

🔇 Additional comments (1)

compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java (1)

918-931: 在候选宿主机查询中开放“无 L3 网络”场景的改动是合理的

这里给 DesignatedAllocateHostMsg 增加 amsg.setAllowNoL3Networks(true);，只会影响 VmNicHelper.getL3Uuids(...) 结果为空的场景（例如 VM 当前没有 NIC，或 NIC 不挂任何 L3），允许调度在无 L3 约束下返回候选宿主机，和 getVmMigrationTargetHost() 中的用法保持一致。对已有 NIC/L3 的 VM，L3 列表非空时该开关不会放宽原有约束，兼容性风险可接受，符合“迁移后在其他集群启动无 NIC VM”的需求定位。建议在以下两种场景各做一次回归验证：1）无 NIC VM 获取候选宿主机并启动；2）有多 L3 NIC 的 VM，确认候选宿主机仍受 L3 网络约束。

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ast-grep (0.40.0)

compute/src/main/java/org/zstack/compute/vm/VmInstanceBase.java

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}