Skip to content

Deployment: Stop automate deployment in ECS - #876

Merged
Ayush8923 merged 1 commit into
mainfrom
fix/deploy-ecs-manually
May 21, 2026
Merged

Deployment: Stop automate deployment in ECS#876
Ayush8923 merged 1 commit into
mainfrom
fix/deploy-ecs-manually

Conversation

@Ayush8923

@Ayush8923Ayush8923 commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Stop automate deployment in ECS after merging the PR into main, because now we are using the EC2 instance for staging instead of ECS.

Checklist

Before submitting a pull request, please ensure that you mark these task.

  • Ran fastapi run --reload app/main.py or docker compose up in the repository root and test.
  • If you've fixed a bug or added code that is tested and has test cases.

Summary by CodeRabbit

  • Chores
    • Modified staging deployment workflow configuration to require manual triggering instead of automatic deployment on branch updates.

Review Change Stack

@coderabbitai

coderabbitaiBot commented May 21, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6af70773-6f9d-4c33-b3f9-e03540d1d4c2

📥 Commits

Reviewing files that changed from the base of the PR and between f1880b5 and 33cdfd5.

📒 Files selected for processing (1)
  • .github/workflows/cd-staging-ecs.yml

📝 Walkthrough

Walkthrough

The staging ECS deployment workflow is reconfigured to clarify its purpose through an updated display name and switch from automatic push-based triggering to manual invocation via workflow_dispatch.

Changes

Staging ECS CD Workflow

Layer / File(s)Summary
Workflow metadata and trigger
.github/workflows/cd-staging-ecs.yml
Workflow display name updated to "Deploy Kaapi to ECS Staging" and trigger mechanism changed from push on main branch to workflow_dispatch for manual invocation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • vprashrex
  • AkhileshNegi

Poem

🐰 A workflow needs a gentle hand,
No auto-push across the land,
Now "staging" shines in the display,
Manual control saves the day! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check nameStatusExplanation
Description Check✅ PassedCheck skipped - CodeRabbit’s high-level summary is enabled.
Title check✅ PassedThe title 'Deployment: Stop automate deployment in ECS' clearly describes the main change: disabling automated ECS deployments. It directly relates to the workflow modification that switches from automatic pushes to manual dispatch triggering.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck 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 fix/deploy-ecs-manually

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.

@sentry

sentryBot commented May 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Ayush8923
Ayush8923 merged commit 34b3a7b into mainMay 21, 2026
3 checks passed
@Ayush8923
Ayush8923 deleted the fix/deploy-ecs-manually branch May 21, 2026 13:40
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Ayush8923@Prajna1999