TESTcase-simplekvm

[ZStack-Robot]

testcase for kvm running status

Resolves: ZSTAC-1234

Change-Id: I6f6f686fffffffffffff6b6966786175766b62

sync from gitlab !9334

[coderabbitai]

分析摘要

概览

此PR新增了一个VM生命周期集成测试用例，用于验证虚拟机创建后的运行状态；同时启用了zstack.properties配置文件中之前被注释的大量属性设置。

变更汇总

文件分组	摘要
新增测试用例 test/src/test/groovy/org/zstack/test/integration/kvm/vm/SimpleVmLifeCycleCase.groovy	新增KVM虚拟机生命周期测试类，包含环境设置、VM创建和状态验证逻辑。测试流程：创建双主机NFS环境 → 创建虚拟机 → 断言VM运行状态，共51行新代码。
配置文件更新 test/src/test/resources/zstack.properties	取消注释并启用CloudBus、vmTracer、Simulator、Ansible、Cassandra等功能块的配置属性，88行修改，激活之前被禁用的运行时设置。

代码评审工作量

🎯 2 (Simple) | ⏱️ ~12 分钟

诗

🐰 小小测试入库来，
VM生命周期验证在，
配置属性齐解封，
系统功能全启动，
代码质量步步高！

---

Important

Pre-merge checks failed

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

❌ Failed checks (1 error)

Check name	Status	Explanation	Resolution
Title check	❌ Error	PR title does not follow the required format '[scope]: ' and appears to be an informal abbreviation rather than a proper commit title.	Rewrite title to follow the format, e.g., 'test[kvm]: Add simple VM lifecycle test case' or 'test[vm]: Verify KVM VM running status'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	PR description is related to the changeset, mentioning a KVM test case and including proper tracking references.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch sync/tian.huang/huangtian_5.5.12

📝 Coding Plan

• Generate coding plan for human review comments

---

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

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In
`@test/src/test/groovy/org/zstack/test/integration/kvm/vm/SimpleVmLifeCycleCase.groovy`:
- Around line 36-38: Replace the Chinese Javadoc comment in
SimpleVmLifeCycleCase.groovy with an English equivalent; specifically update the
comment block (the /** ... */ above the test) to read something like "Test: VM
should be in Running state after creation" so the file contains no Chinese text.
- Around line 11-14: The top-of-file Chinese comment block in
SimpleVmLifeCycleCase.groovy contains non-compliant Chinese text; replace the
Chinese comments with equivalent English wording (e.g., "Minimal VM test case"
and "Only verifies VM state after creation") so the header comment above the
class SimpleVmLifeCycleCase uses English only and complies with the coding
standard.

---

ℹ️ Review info

⚙️ Run configuration

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

Review profile: CHILL

Plan: Pro

Run ID: 2b9347d8-a0b0-49f4-9da4-2fe677e579ed

📥 Commits

Reviewing files that changed from the base of the PR and between 44c191f and 1eae100.

📒 Files selected for processing (2)

• test/src/test/groovy/org/zstack/test/integration/kvm/vm/SimpleVmLifeCycleCase.groovy
• test/src/test/resources/zstack.properties

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

注释中包含中文，违反编码规范。

根据编码规范，代码中不应包含中文（包括注释）。请将中文注释改为英文。

📝 建议修改

 /**
- * 最小化虚拟机测试用例
- * 仅验证虚拟机创建后的状态
+ * Minimal VM test case
+ * Only verifies the VM state after creation
  */
 class SimpleVmLifeCycleCase extends SubCase {

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	/**
	* 最小化虚拟机测试用例
	* 仅验证虚拟机创建后的状态
	*/
	/**
	* Minimal VM test case
	* Only verifies the VM state after creation
	*/

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@test/src/test/groovy/org/zstack/test/integration/kvm/vm/SimpleVmLifeCycleCase.groovy`
around lines 11 - 14, The top-of-file Chinese comment block in
SimpleVmLifeCycleCase.groovy contains non-compliant Chinese text; replace the
Chinese comments with equivalent English wording (e.g., "Minimal VM test case"
and "Only verifies VM state after creation") so the header comment above the
class SimpleVmLifeCycleCase uses English only and complies with the coding
standard.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

方法注释中包含中文，违反编码规范。

根据编码规范，代码中不应包含中文（包括注释）。请将中文注释改为英文。

📝 建议修改

     /**
-     * 测试：创建后虚拟机应处于 Running 状态
+     * Test: VM should be in Running state after creation
      */
     void testVmIsRunningAfterCreation() {

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	/**
	* 测试：创建后虚拟机应处于 Running 状态
	*/
	/**
	* Test: VM should be in Running state after creation
	*/

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In
`@test/src/test/groovy/org/zstack/test/integration/kvm/vm/SimpleVmLifeCycleCase.groovy`
around lines 36 - 38, Replace the Chinese Javadoc comment in
SimpleVmLifeCycleCase.groovy with an English equivalent; specifically update the
comment block (the /** ... */ above the test) to read something like "Test: VM
should be in Running state after creation" so the file contains no Chinese text.