<fix>[vm_plugin]: fix detach vm nic without alias

[zstack-robot-2]

Resolves: ZSV-5276

Change-Id: I6868776c3d9b3aae39344e3fa9b1b9def4cf8223
(cherry picked from commit 26ae530)

sync from gitlab !4610

Summary by CodeRabbit

• Bug Fixes
  • 优化了虚拟机插件中的条件判断，以确保在非“Detach”操作时正确执行特定代码块。

[coderabbitai]

Walkthrough

本次更改主要在vm_plugin.py文件中增加了一个条件检查，当action不等于'Detach'以及之前的'Update'时，才执行特定的代码块。这是对虚拟机插件逻辑的一个小幅调整，旨在细化操作的条件，以便更精确地控制代码执行流程。

Changes

文件路径	更改摘要
kvmagent/kvmagent/plugins/.../vm_plugin.py	添加条件检查，除了'Update'外，action不等于'Detach'也会执行特定代码块。

🐰✨

在代码的海洋里，小兔跳跃，
'Detach'和'Update'，轻轻一躲，
逻辑更严谨，bug远离，
小兔庆祝，代码更美妙。
🌟📜✨

---

Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>.
  • Generate unit-tests for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit tests for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai generate interesting stats about this repository and render them as a table.
  • @coderabbitai show all the console.log statements in this repository.
  • @coderabbitai read src/utils.ts and generate unit tests.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger a review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• The JSON schema for the configuration file is available here.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/coderabbit-overrides.v2.json

CodeRabbit Discord Community

Join our Discord Community to get help, request features, and share feedback.

[coderabbitai]

Review Status

Actionable comments generated: 0

Configuration used: CodeRabbit UI

Commits

Files that changed from the base of the PR and between 980adcd and 43d3420.

Files selected for processing (1)

• kvmagent/kvmagent/plugins/vm_plugin.py (1 hunks)

Additional comments (1)

kvmagent/kvmagent/plugins/vm_plugin.py (1)

• 6024-6024: 代码更改引入了一个条件检查，以确保在执行特定代码块之前，action 变量不等于 'Detach' 以及 'Update'。这是一个重要的改进，旨在解决没有别名时分离 VM NIC 的问题。建议进行额外的测试，以确保此更改不会意外影响其他功能。