[host]: SystemTagCreator ignoreIfExisting to prevent duplicate tags

[MatheMatrix]

Resolves: ZSTAC-58237

sync from gitlab !9841

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• conf/errorElaborations/Elaboration.json is excluded by !**/*.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ 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: 96ca0319-e372-4af3-94a5-9a79b2894b3f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

Walkthrough

该 PR 在 HostBase.updateHostConnectedTime 中将创建 HostSystemTags.HOST_CONNECTED_TIME 的 SystemTagCreator 配置为 ignoreIfExisting = true，避免在标签已存在时创建失败。

Changes

Host System Tag Configuration

Layer / File(s)	Summary
Host Connected Time Tag Creation compute/src/main/java/org/zstack/compute/host/HostBase.java	updateHostConnectedTime 方法中的 SystemTagCreator 配置添加 ignoreIfExisting = true 标志，允许在 HostSystemTags.HOST_CONNECTED_TIME 标签已存在时安全跳过创建。

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 一行代码轻轻改，
主机标签不再抛错，
若已存在则默默越过，
连接时间静好如初，
小改动，大安心。

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The pull request title follows the required [scope]: format with '[host]' as scope and is 67 characters, within the 72-character limit.
Description check	✅ Passed	The pull request description is related to the changeset, mentioning the JIRA issue ZSTAC-58237 and referencing the gitlab sync, which aligns with the SystemTagCreator change.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch sync/songtao.liu/fix/ZSTAC-58237-v2

---

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

[zstack-robot-1]

Comment from yaohua.wu:

Review: MR !9841 — ZSTAC-58237

总结

本次修复针对重连物理机时 HOST_CONNECTED_TIME system tag 被反复删除再创建（即使值未变化）导致 mysql-bin 快速增长的问题。修改点为在 HostBase.updateHostConnectedTime 的 SystemTagCreator 上设置 ignoreIfExisting = true，当同一 tag（UUID 由 resourceUuid + resourceType + tag 内容确定性生成）已存在时直接跳过写入，避免不必要的 DB 操作。

从 SystemTag.doCreate() 实现来看，ignoreIfExisting = true 会在两处截断：①先查 count 发现 uuid 已存在则返回 null；② persist 时捕获 Duplicate entry 异常静默返回 null。结合调用侧已有的前置守卫 if (hostStatus == HostStatus.Connected && !tagExists) 可知，正常路径下该 tag 不存在时才会走到 creator；ignoreIfExisting 起到并发竞态下的兜底保护作用，设计合理。

recreate = false 已经确保不会先删再建；connectedTime 是 System.currentTimeMillis()，每次重连值不同，因此此处 ignoreIfExisting 的语义是"同一主机本次连接时间 tag 已写入则跳过"，符合幂等预期。

Jira 中原始 issue 描述的是 iommuStatus tag 在 premium 模块（已由 MR !7980 修复），本 MR 修复的是同类问题在 HOST_CONNECTED_TIME tag 上的对应分支（5.5.22 版本），修复逻辑一致、覆盖完整。

无阻塞性问题，建议合并。

Verdict: APPROVED

---

🤖 Robot Reviewer