You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "aminghadersohi (via GitHub)" <gi...@apache.org> on 2025/12/11 23:02:35 UTC
[PR] docs(mcp): add Kubernetes deployment instructions to MCP service README [superset]
aminghadersohi opened a new pull request, #36547:
URL: https://github.com/apache/superset/pull/36547
### SUMMARY
This PR adds comprehensive Kubernetes deployment documentation to the MCP service README. The documentation enables users to deploy the MCP service in production Kubernetes environments with proper authentication, scaling, and routing.
**Key additions:**
- **Architecture diagrams**: Mermaid diagrams showing deployment topology and request flow
- **Two deployment options**:
1. Using the official Superset Helm chart with MCP as a sidecar container
2. Dedicated MCP service deployment for independent scaling
- **Complete Kubernetes manifests**: Deployment, Service, HPA, PodDisruptionBudget, and Ingress configurations
- **Configuration reference**: Environment variables and superset_config.py options
- **Production considerations**: Security (JWT, Network Policies, TLS), resource allocation, and Prometheus monitoring
- **Troubleshooting guide**: Common issues and debugging commands
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A - Documentation only change
### TESTING INSTRUCTIONS
1. Review the documentation in `superset/mcp_service/README.md`
2. Verify the Mermaid diagrams render correctly on GitHub
3. Optionally test the Kubernetes manifests in a local cluster (e.g., minikube, kind)
4. Verify the Helm values example works with the official Superset chart
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [x] Introduces new feature or API
- [ ] Removes existing feature or API
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] docs(mcp): add Kubernetes deployment instructions to MCP service README [superset]
Posted by "bito-code-review[bot] (via GitHub)" <gi...@apache.org>.
bito-code-review[bot] commented on PR #36547:
URL: https://github.com/apache/superset/pull/36547#issuecomment-3644156120
**Bito Automatic Review Skipped - Files Excluded**<br/><br/>Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type `/review` in a comment and save.<br/> You can change the excluded files settings [here](https://alpha.bito.ai/home/ai-agents/code-review-agent), or contact your Bito workspace admin at [evan@preset.io](mailto:evan@preset.io).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] docs(mcp): add Kubernetes deployment instructions to MCP service README [superset]
Posted by "mistercrunch (via GitHub)" <gi...@apache.org>.
mistercrunch merged PR #36547:
URL: https://github.com/apache/superset/pull/36547
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] docs(mcp): add Kubernetes deployment instructions to MCP service README [superset]
Posted by "michael-s-molina (via GitHub)" <gi...@apache.org>.
michael-s-molina commented on PR #36547:
URL: https://github.com/apache/superset/pull/36547#issuecomment-3646203541
@aminghadersohi @rusackas I think we should start thinking about MCP documentation in the Developer Portal similar to what we are doing with [Extensions](https://superset.apache.org/developer_portal/extensions/overview).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] docs(mcp): add Kubernetes deployment instructions to MCP service README [superset]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #36547:
URL: https://github.com/apache/superset/pull/36547#issuecomment-3644178544
## [Codecov](https://app.codecov.io/gh/apache/superset/pull/36547?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 47.89%. Comparing base ([`76d897e`](https://app.codecov.io/gh/apache/superset/commit/76d897eaa2f9e137102bc194c2e3109c29d0348f?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`098bc35`](https://app.codecov.io/gh/apache/superset/commit/098bc356bf17cd393409b23aff450faed8d1b9b1?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
:warning: Report is 3106 commits behind head on master.
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #36547 +/- ##
===========================================
- Coverage 60.48% 47.89% -12.60%
===========================================
Files 1931 637 -1294
Lines 76236 47101 -29135
Branches 8568 5130 -3438
===========================================
- Hits 46114 22560 -23554
+ Misses 28017 23886 -4131
+ Partials 2105 655 -1450
```
| [Flag](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [hive](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `43.52% <ø> (-5.64%)` | :arrow_down: |
| [javascript](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | |
| [presto](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `47.16% <ø> (-6.65%)` | :arrow_down: |
| [python](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `47.89% <ø> (-15.62%)` | :arrow_down: |
| [unit](https://app.codecov.io/gh/apache/superset/pull/36547/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `100.00% <ø> (+42.36%)` | :arrow_up: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
</details>
[:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/superset/pull/36547?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
:loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org