Skip to content

fix(ci): harden staging deploy over ssh#24

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

fix(ci): harden staging deploy over ssh#24
ailuckly merged 1 commit into
developfrom
fix/staging-deploy-resilience

Conversation

@ailuckly
Copy link
Copy Markdown
Owner

@ailuckly ailuckly commented May 5, 2026

Add SSH keepalive options to the staging deploy and failure log collection steps so long remote builds do not drop idle connections.

Stop forcing Docker Compose no-cache builds on every deploy; use --pull while preserving build and Maven cache layers to reduce staging deploy time.

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

📌 变更内容

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

✅ 测试验证

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

PR 提交规范提醒:

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

Add SSH keepalive options to the staging deploy and failure log collection steps so long remote builds do not drop idle connections.

Stop forcing Docker Compose no-cache builds on every deploy; use --pull while preserving build and Maven cache layers to reduce staging deploy time.

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:29
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

Hardens the staging CD GitHub Actions workflow to make long-running SSH-based deployments more reliable, and reduces deploy time by avoiding forced no-cache Docker Compose builds.

Changes:

  • Added SSH keepalive options (ServerAliveInterval, ServerAliveCountMax, TCPKeepAlive) to both the deploy and failure-log-collection SSH steps.
  • Switched docker compose build from --no-cache to --pull to preserve build cache while still refreshing base images for built services.

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

@ailuckly ailuckly merged commit 59f5d96 into develop May 5, 2026
9 checks passed
@ailuckly ailuckly deleted the fix/staging-deploy-resilience 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