Skip to content

Rework integration docs & add mcp#3825

Draft
mjudeikis wants to merge 3 commits intokcp-dev:mainfrom
mjudeikis:integration.docs
Draft

Rework integration docs & add mcp#3825
mjudeikis wants to merge 3 commits intokcp-dev:mainfrom
mjudeikis:integration.docs

Conversation

@mjudeikis
Copy link
Contributor

Summary

What Type of PR Is This?

Related Issue(s)

Fixes #

Release Notes

NONE

Signed-off-by: Mangirdas Judeikis <mangirdas@judeikis.lt>
On-behalf-of: SAP <mangirdas.judeikis@sap.com
@kcp-ci-bot kcp-ci-bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none Denotes a PR that doesn't merit a release note. labels Feb 7, 2026
@kcp-ci-bot
Copy link
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kcp-ci-bot kcp-ci-bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Feb 7, 2026
@kcp-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign embik for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 7, 2026
@@ -0,0 +1,132 @@
# Kubernetes MCP Server Helm Values for KCP
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Kubernetes MCP Server Helm Values for KCP
# Kubernetes MCP Server Helm Values for kcp


4. Create the database secret for Keycloak:
```sh
kubectl apply -f keycloak-db-secret.yaml
Copy link
Member

Choose a reason for hiding this comment

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

Instead of a fixed file a create secret command with --from-literal values instead?
Same in postgres-cluster. I think that's clearer and nicer for users going through the docs and we won't run into the possibility of some codescanning recognizing that as secrets.

These assets are referenced by the production deployment documentation in `docs/content/setup/production/`.

Each deployment type (dekker, vespucci, comer) has its own subdirectory with complete configuration files and deployment manifests.
Each deployment type (dekker, vespucci, comer, zheng) has its own subdirectory with complete configuration files and deployment manifests.
Copy link
Member

Choose a reason for hiding this comment

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

I think we can scrap or shorten listing the types, if we add more it will get very long.

Suggested change
Each deployment type (dekker, vespucci, comer, zheng) has its own subdirectory with complete configuration files and deployment manifests.
Each deployment type (dekker, vespucci, ...) has its own subdirectory with complete configuration files and deployment manifests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. do-not-merge/needs-kind Indicates a PR lacks a `kind/foo` label and requires one. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants