fix(ci): 修复deploy脚本语法错误（多余双引号） #612

Merged

xiaoxia merged 2 commits from fix/deploy-script-syntax into develop 2026-07-19 20:45:21 +08:00

Conversation 4 Commits 2 Files Changed 3

+2 -6

xiaoxia commented 2026-07-19 20:38:33 +08:00

Owner

Copy Link

Copy Source

#611 合入时引入的笔误，echo "All images pulled."" 末尾多了一个双引号，导致SSH远端sh执行时脚本报 Syntax error: "(" unexpected，Deploy Staging步骤失败。

修复：移除两个脚本都修掉多余的双引号。

#611 合入时引入的笔误，`echo "All images pulled.""` 末尾多了一个双引号，导致SSH远端sh执行时脚本报 `Syntax error: "(" unexpected`，Deploy Staging步骤失败。 修复：移除两个脚本都修掉多余的双引号。

xiaoxia added 1 commit 2026-07-19 20:38:33 +08:00

fix(ci): 修复deploy脚本多余双引号导致的语法错误 ... fad12efd72

#611 合入时引入的笔误，echo行末尾多了一个双引号，导致
SSH远端用sh执行时脚本报 Syntax error: "(" unexpected，
Deploy Staging步骤失败，staging无法自动更新。

修复：移除多余的双引号，staging和production两个脚本都修。

xiaoxia added 1 commit 2026-07-19 20:39:31 +08:00

fix(ci): 修复ACR cleanup错误的命令行参数 ... f7c59b6b44

acr_cleanup.py通过环境变量(ACR_REGISTRY/ACR_NAMESPACE/ACR_USERNAME/ACR_PASSWORD)
读取配置，不接受--registry/--namespace/--username/--password参数。
CI重构时错误地把这些当作命令行参数传入，导致job失败。

移除错误参数，保留正确的--keep/--pr-days/--execute参数。

auto-approve-bot commented 2026-07-19 20:41:52 +08:00

Collaborator

Copy Link

Copy Source

📊 审查概览

• 整体评价：需修改
• 严重问题数量：1 个
• 一般建议数量：0 个

❌ 需修改的问题（严重）

1. [.gitea/workflows/ci-pipeline.yml: 931-936] 移除关键运行参数导致功能失效
   • 问题描述：移除了 python3 scripts/ci/acr_cleanup.py 命令中的 --registry、--namespace、--username 和 --password 参数。除非 acr_cleanup.py 脚本逻辑在本次 PR 中同步修改为从环境变量或其他默认配置读取这些值，否则该脚本将因缺少必要的连接信息和认证凭据而无法正常运行，导致镜像清理任务失败。
   • 修改建议：请回滚这些参数的删除，或者确认 acr_cleanup.py 已更新为不再依赖这些 CLI 参数。如果脚本已支持环境变量，请确保 CI 环境中已正确配置对应变量。

💡 改进建议（一般）

无

✅ 良好实践

• 修复了 scripts/ci_production_deploy.sh 和 scripts/ci_staging_deploy.sh 中 echo 语句末尾多余的引号，消除了潜在的 Shell 语法错误。

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-19 20:41:52 | 模型:}

### 📊 审查概览 - 整体评价：需修改 - 严重问题数量：1 个 - 一般建议数量：0 个 ### ❌ 需修改的问题（严重） 1. **[.gitea/workflows/ci-pipeline.yml: 931-936] 移除关键运行参数导致功能失效** - 问题描述：移除了 `python3 scripts/ci/acr_cleanup.py` 命令中的 `--registry`、`--namespace`、`--username` 和 `--password` 参数。除非 `acr_cleanup.py` 脚本逻辑在本次 PR 中同步修改为从环境变量或其他默认配置读取这些值，否则该脚本将因缺少必要的连接信息和认证凭据而无法正常运行，导致镜像清理任务失败。 - 修改建议：请回滚这些参数的删除，或者确认 `acr_cleanup.py` 已更新为不再依赖这些 CLI 参数。如果脚本已支持环境变量，请确保 CI 环境中已正确配置对应变量。 ### 💡 改进建议（一般） 无 ### ✅ 良好实践 - 修复了 `scripts/ci_production_deploy.sh` 和 `scripts/ci_staging_deploy.sh` 中 echo 语句末尾多余的引号，消除了潜在的 Shell 语法错误。 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-19 20:41:52 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-07-19 20:44:14 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-19 20:44:15 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

xiaoxia merged commit 9dee33c54d into develop 2026-07-19 20:45:21 +08:00

xiaoxia deleted branch fix/deploy-script-syntax 2026-07-19 20:45:21 +08:00

gitea-actions bot commented 2026-07-19 20:45:26 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #612 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #612 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#612