Skip to content

<feat>[utils]: add GPU pre-check error code ORG_ZSTACK_AI_VM_10012#3774

Open
MatheMatrix wants to merge 1 commit into5.5.16from
sync/xinhao.huang/5.5.16@@2
Open

<feat>[utils]: add GPU pre-check error code ORG_ZSTACK_AI_VM_10012#3774
MatheMatrix wants to merge 1 commit into5.5.16from
sync/xinhao.huang/5.5.16@@2

Conversation

@MatheMatrix
Copy link
Copy Markdown
Owner

Summary

  • 新增 GPU 资源预检查错误码 ORG_ZSTACK_AI_VM_10012
  • 新增中英文 i18n 错误消息映射

关联: ZSTAC-84176

sync from gitlab !9644

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent 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: d296391a-83a3-4d99-9493-5093efe64d65

📥 Commits

Reviewing files that changed from the base of the PR and between 3b4d8ab and 976a819.

⛔ Files ignored due to path filters (2)
  • conf/i18n/globalErrorCodeMapping/global-error-en_US.json is excluded by !**/*.json
  • conf/i18n/globalErrorCodeMapping/global-error-zh_CN.json is excluded by !**/*.json
📒 Files selected for processing (1)
  • utils/src/main/java/org/zstack/utils/clouderrorcode/CloudOperationsErrorCode.java
✅ Files skipped from review due to trivial changes (1)
  • utils/src/main/java/org/zstack/utils/clouderrorcode/CloudOperationsErrorCode.java

Walkthrough

在云操作错误代码常量类中新增常量 ORG_ZSTACK_AI_VM_10012,其值为字符串 "ORG_ZSTACK_AI_VM_10012"。仅添加常量声明,无其他逻辑或控制流改动。

Changes

Cohort / File(s) Summary
错误代码常量
utils/src/main/java/org/zstack/utils/clouderrorcode/CloudOperationsErrorCode.java
新增公共静态常量 ORG_ZSTACK_AI_VM_10012 = "ORG_ZSTACK_AI_VM_10012",仅声明新增,无行为或控制流改动。

预估代码审查工作量

🎯 1 (Trivial) | ⏱️ ~2 minutes

诗歌

🐰 一个常数悄悄落地,
云端错误有了新名字,
行间轻轻添一行,
静候日志轻声诉事情。

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR标题遵循指定的格式 [scope]: ,长度66个字符(在72字符限制内),内容与变更集相关。
Description check ✅ Passed PR描述与变更集相关,说明了新增GPU预检查错误码和国际化消息映射的目的,并关联了相应的JIRA工单。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch sync/xinhao.huang/5.5.16@@2

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.42.1)
utils/src/main/java/org/zstack/utils/clouderrorcode/CloudOperationsErrorCode.java

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

@MatheMatrix MatheMatrix force-pushed the sync/xinhao.huang/5.5.16@@2 branch 2 times, most recently from fa6bdba to 3b4d8ab Compare April 16, 2026 10:34
Add error code and i18n messages for GPU resource availability pre-check
during model service deployment.

Resolves: ZSTAC-84176

Change-Id: I676f6e77696d6f7577726170756f6878647a716f
@MatheMatrix MatheMatrix force-pushed the sync/xinhao.huang/5.5.16@@2 branch from 3b4d8ab to 976a819 Compare April 17, 2026 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant