Skip to content

ci: make manual pipeline deploy-only - #398

Merged
delete-cloud merged 1 commit into
mainfrom
ci/deploy-only-manual
May 29, 2026
Merged

delete-cloud merged 1 commit into
mainfrom
ci/deploy-only-manual

Conversation

@delete-cloud

@delete-cloud delete-cloud commented May 29, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Remove manual triggers from the CI/build workflow.
  • Keep manual pipeline focused on deploy and raise rollout timeout to 420s.

Verification

  • git diff --check -- .
  • python3 assertion for Woodpecker trigger contract

Summary by CodeRabbit

  • Chores
    • Updated continuous integration workflow to restrict automated job execution to the main branch.
    • Adjusted deployment rollout timeout configuration for improved stability during deployments.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 29, 2026 •

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a330e2a0-d02e-4981-a450-969b5a95bda9

📥 Commits

Reviewing files that changed from the base of the PR and between 7b87769 and 1ed396d.

📒 Files selected for processing (2)
  • .woodpecker/ci.yml
  • .woodpecker/deploy.yml
💤 Files with no reviewable changes (1)
  • .woodpecker/ci.yml

📝 Walkthrough

Walkthrough

CI workflow trigger conditions are simplified by removing manual execution and restricting push events to the main branch. Deployment workflow receives an explicit rollout timeout of 420 seconds for orchestrated Kubernetes deployments.

Changes

Deployment Pipeline Configuration

Layer / File(s) Summary
CI trigger constraint
.woodpecker/ci.yml
Manual trigger option removed from the top-level event list and push-specific when clause; push events now constrained to branch: main only.
Deployment rollout timeout
.woodpecker/deploy.yml
ROLLOUT_TIMEOUT environment variable added to the deploy-o6n step, set to 420s for Kubernetes rollout configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Workflows now flow with purpose true,
Manual triggers bid adieu,
Main branch only, timeout set at last,
This rabbit dances, CI runs fast!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: removing manual triggers from CI and keeping the manual pipeline deploy-only, which aligns with the file modifications in ci.yml and deploy.yml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/deploy-only-manual

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@delete-cloud
delete-cloud merged commit b498c33 into main May 29, 2026
1 check passed
@delete-cloud
delete-cloud deleted the ci/deploy-only-manual branch May 29, 2026 21:10
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