<feature>[errorcode]: ZSTAC-86666 update backup conflict error mappings - #4678
<feature>[errorcode]: ZSTAC-86666 update backup conflict error mappings#4678zstack-robot-2 wants to merge 2 commits into
Conversation
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 40 minutes Limit details: You’ve used the included review currently available. Your 96 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (10)
📒 Files selected for processing (5)
Warning
|
8ee8597 to
25006c8
Compare
|
Comment from yingzhe.hu: Code Review代码量不大,但两个契约问题都不能带过去:i18n 会静默吞掉冲突资源信息,MR 又把包含 13 个字段的 schema migration 写成 JSON-only。
PonytailLean already. Ship. Findings
结论REVISION_REQUIRED 🚫 回归风险:高。错误定位信息退化,且 schema 发布证据缺失。
Review by Codex + Ponytail Review Tracker: ZSTAC-86666Rounds: R1=block
Progress: ░░░░░░░░░░ 0/8 resolved (0%) |
c7341a3 to
b7306aa
Compare
Keep snapshot execution lock-free and add the error mappings required by premium backup conflict checks. Resolves: ZSTAC-86666 Change-Id: Ib5d2ad686fa92394a771980c1d1425adea6ca02f
2b6a93d to
327cab4
Compare
Route chain snapshot admission by vmUuid and use the shared per-VM backup operation queue. Resolves: ZSTAC-86666 Change-Id: Idfd94627a050fab22032f6d66f243ba3e4926320
Summary
Update global error-code mappings for backup/CBT/CDP scheduler conflict messages on 5.5.38.
Changes
Testing
Notes
Resolves: ZSTAC-86666
sync from gitlab !10732