Skip to content

chore: apply post-deploy fixes#12

Merged
webpiratt merged 1 commit intomainfrom
chore-post-deploy
Feb 27, 2026
Merged

chore: apply post-deploy fixes#12
webpiratt merged 1 commit intomainfrom
chore-post-deploy

Conversation

@4others
Copy link
Contributor

@4others 4others commented Feb 27, 2026

Summary by CodeRabbit

  • Chores
    • Updated Kubernetes deployment infrastructure with standardized service naming conventions applied consistently across development and production environments.
    • Removed ingress resource configurations that previously managed external service routing.

@webpiratt webpiratt merged commit 992857b into main Feb 27, 2026
1 of 2 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4dab892 and 5803530.

📒 Files selected for processing (5)
  • .github/workflows/deploy-dev.yaml
  • .github/workflows/deploy-prod.yaml
  • deploy/01_server.yaml
  • deploy/dev/02_ingress.yaml
  • deploy/prod/02_ingress.yaml

Walkthrough

The changes rename the Kubernetes server deployment from "server" to "mcp-server" across configuration and workflow files, and remove ingress configurations from both development and production environments.

Changes

Cohort / File(s) Summary
Workflow Rollout Checks
.github/workflows/deploy-dev.yaml, .github/workflows/deploy-prod.yaml
Updated rollout status checks from deployment/server to deployment/mcp-server to align with renamed deployment target.
Server Deployment Configuration
deploy/01_server.yaml
Renamed Kubernetes Service and Deployment resources from "server" to "mcp-server" in metadata, labels, selectors, and container names.
Ingress Configuration
deploy/dev/02_ingress.yaml, deploy/prod/02_ingress.yaml
Deleted Kubernetes Ingress resources that previously routed traffic to the "server" backend service.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore-post-deploy

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.

RaghavSood added a commit that referenced this pull request Feb 27, 2026
Revert the deletion of deploy/dev/02_ingress.yaml from PR #12,
updated to reference the renamed mcp-server service.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@RaghavSood RaghavSood mentioned this pull request Feb 27, 2026
2 tasks
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