Skip to content

Conversation

@ashlkv
Copy link
Contributor

@ashlkv ashlkv commented Jan 5, 2026

We're going to use SA as is if it's specified, so it has to be a full name as shown in kubectl get sa

Copilot AI review requested due to automatic review settings January 5, 2026 12:24
@vercel
Copy link

vercel bot commented Jan 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
archestra Ignored Ignored Jan 5, 2026 0:24am

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 corrects the ServiceAccount name specification in the Kubernetes MCP configuration to use the full ServiceAccount name as it appears in kubectl get sa, rather than a shortened version.

Key changes:

  • Updated the service_account field from "operator" to "archestra-platform-mcp-k8s-operator"
  • Simplified the instructions to directly state the default ServiceAccount name instead of describing a naming pattern

@ashlkv ashlkv merged commit 8af2667 into main Jan 5, 2026
17 checks passed
@ashlkv ashlkv deleted the ashlkv/sa-in-mcp-dialog branch January 5, 2026 12:25
github-merge-queue bot pushed a commit to archestra-ai/archestra that referenced this pull request Jan 6, 2026
If catalog entry has a `serviceAccount` field, like in Kubernetes MCP,
we are using this field value as is. It can also be customized.
No more `ARCHESTRA_ORCHESTRATOR_MCP_K8S_SERVICE_ACCOUNT_NAME`.

Related PR in the catalog
archestra-ai/website#314

MCP install modal:
<img width="1512" height="858" alt="image"
src="https://github.com/user-attachments/assets/f9a5812c-327a-418a-8225-b2ba0528280d"
/>

MCP edit modal:
<img width="1512" height="854" alt="image"
src="https://github.com/user-attachments/assets/15cd9eae-88d5-4812-9dc9-c9e21f79a9ff"
/>

---------

Co-authored-by: Copilot <[email protected]>
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