Skip to content

Comments

Feat/mcp service#151

Merged
avaya09 merged 4 commits intomainfrom
feat/mcp-service
Feb 23, 2026
Merged

Feat/mcp service#151
avaya09 merged 4 commits intomainfrom
feat/mcp-service

Conversation

@avaya09
Copy link
Contributor

@avaya09 avaya09 commented Feb 4, 2026

No description provided.

@avaya09 avaya09 requested a review from Copilot February 4, 2026 09:55
Copy link
Contributor

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

This PR adds MCP (Model Context Protocol) service configuration support to the Portkey Gateway Helm chart. It introduces a new optional service that can be enabled alongside the existing gateway service.

Changes:

  • Added mcpService configuration block to values.yaml with default settings
  • Created a new Kubernetes Service resource template for MCP with conditional rendering

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
charts/portkey-gateway/values.yaml Adds mcpService configuration block with enabled flag, service type, port, and label/annotation options
charts/portkey-gateway/templates/gateway/service.yaml Adds conditional MCP service template that creates a separate Kubernetes service when enabled

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

Avanish Yadav added 2 commits February 23, 2026 18:16
- Introduced containerPort field under mcpService to allow customizable port settings.
- Updated helper template to prioritize mcpService.containerPort for MCP_PORT environment variable fallback.
Copy link
Contributor

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


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

@avaya09 avaya09 merged commit c504705 into main Feb 23, 2026
4 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.

1 participant