Skip to content

chore: restore dev ingress#14

Merged
RaghavSood merged 1 commit intomainfrom
revert-dev-ingress
Feb 27, 2026
Merged

chore: restore dev ingress#14
RaghavSood merged 1 commit intomainfrom
revert-dev-ingress

Conversation

@RaghavSood
Copy link
Collaborator

@RaghavSood RaghavSood commented Feb 27, 2026

Summary

  • Restores deploy/dev/02_ingress.yaml which was deleted in PR chore: apply post-deploy fixes #12
  • Service name updated to mcp-server to match the rename from the same PR
  • Prod ingress remains removed as intended

Test plan

  • Verify dev ingress is applied on next dev deploy
  • Confirm mcp.dev.plugins.vultisig.com routes to the mcp-server service

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Chores
    • Added infrastructure configuration to enable secure HTTPS access to the mcp service through a new dedicated endpoint with automatic TLS certificate management.

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>
@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ 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 85aac04 and 0b39cdb.

📒 Files selected for processing (1)
  • deploy/dev/02_ingress.yaml

Walkthrough

A new Kubernetes Ingress resource is introduced to handle external traffic routing for the mcp service. The manifest configures TLS termination via cert-manager with Traefik as the ingress controller, routing requests to the mcp-server service.

Changes

Cohort / File(s) Summary
Kubernetes Ingress Configuration
deploy/dev/02_ingress.yaml
New Ingress manifest defining routing for mcp.dev.plugins.vultisig.com with Traefik ingress class, TLS certificate management via cert-manager, and backend service routing to mcp-server on port 80.
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: restore dev ingress' clearly and directly describes the main change: restoring a previously deleted Ingress configuration file for the dev environment.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-dev-ingress

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

@RaghavSood RaghavSood merged commit 11b6ef3 into main Feb 27, 2026
2 checks passed
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