Skip to content

fix(ci): queue staging deployments#25

Merged
ailuckly merged 1 commit into
developfrom
fix/staging-deploy-queue
May 5, 2026
Merged

fix(ci): queue staging deployments#25
ailuckly merged 1 commit into
developfrom
fix/staging-deploy-queue

Conversation

@ailuckly
Copy link
Copy Markdown
Owner

@ailuckly ailuckly commented May 5, 2026

Do not cancel an in-progress staging deployment when a newer develop push or manual rerun starts.

This prevents GitHub Actions from terminating long remote Docker/Maven builds with 'The operation was canceled'.

Verification: git diff --check; ruby YAML.load_file('.github/workflows/cd-staging.yml')

📌 变更内容

  • 如:新增用户登录 API
  • 如:修复登录失败时错误提示

✅ 测试验证

  • 本地运行通过
  • 自测通过
  • CI 流水线通过

PR 提交规范提醒:

  • 确保提交主题信息符合约定式提交规范 (feat/fix/docs/style/refactor/test/chore)
  • 确保代码已经通过本地测试
  • 确保没有提交敏感信息(密码、密钥等)

Do not cancel an in-progress staging deployment when a newer develop push or manual rerun starts.

This prevents GitHub Actions from terminating long remote Docker/Maven builds with 'The operation was canceled'.

Verification: git diff --check; ruby YAML.load_file('.github/workflows/cd-staging.yml')
Copilot AI review requested due to automatic review settings May 5, 2026 07:51
@ailuckly ailuckly merged commit ca8a943 into develop May 5, 2026
2 checks passed
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts the staging CD workflow’s concurrency behavior so newer runs no longer cancel an in-progress staging deployment, preventing long remote Docker/Maven builds from being terminated mid-flight.

Changes:

  • Disable cancel-in-progress for the staging-deploy concurrency group so runs queue instead of canceling active deployments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ailuckly ailuckly deleted the fix/staging-deploy-queue branch May 7, 2026 12:03
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.

2 participants