Skip to content

Feat/mcp service#151

Open
avaya09 wants to merge 2 commits intomainfrom
feat/mcp-service
Open

Feat/mcp service#151
avaya09 wants to merge 2 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.

metadata:
name: {{ include "portkeyenterprise.fullname" . }}-mcp
labels:
{{- include "gateway.labels" . }}
Copy link

Copilot AI Feb 4, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The include statement is missing the 'nindent' directive, which will cause incorrect YAML indentation. This should be {{- include "gateway.labels" . | nindent 4 }}.

Copilot uses AI. Check for mistakes.
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