Update kdp documentation#2083
Conversation
Signed-off-by: julioc-p <14-10820@usb.ve>
kubermatic-bot
added
do-not-merge/work-in-progress
|
Skipping CI for Draft Pull Request. |
kubermatic-bot
added
the
size/L
Signed-off-by: julioc-p <14-10820@usb.ve>
Signed-off-by: julioc-p <14-10820@usb.ve>
kubermatic-bot
added
size/XL
Signed-off-by: julioc-p <14-10820@usb.ve>
kubermatic-bot
removed
the
do-not-merge/work-in-progress
There was a problem hiding this comment.
Pull request overview
Updates KDP documentation to reflect newer component versions (notably api-syncagent v0.3.x and newer kcp docs), expands the Quickstart with Gateway API and AI Agent deployment paths, and refreshes several service-provider examples to match updated api-syncagent behavior.
Changes:
- Bump referenced versions/links (api-syncagent 0.3.x, kcp docs links, kcp Helm chart version, Crossplane/cert-manager/provider-kubernetes versions).
- Extend the Quickstart to include optional Gateway API-based exposure and a dedicated “Deploy KDP AI Agent” step, plus add Gateway/HTTPRoute example manifests.
- Update PublishedResource documentation/examples for api-syncagent v0.3.x semantics (multi-version publishing, explicit group projection, Go-template naming).
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| content/developer-platform/tutorials/your-first-service/_index.en.md | Update api-syncagent version guidance to 0.3.x and add image tag instruction. |
| content/developer-platform/tutorials/kcp-command-line/_index.en.md | Update upstream kcp kubectl plugin docs link to v0.30. |
| content/developer-platform/tutorials/agent-without-kdp/_index.en.md | Update Sync Agent install snippet to use chart/image v0.3.0. |
| content/developer-platform/setup/quickstart/data/kdp-dashboard.values.yaml | Enable AI agent feature in dashboard values and set generator URL. |
| content/developer-platform/setup/quickstart/data/http-routes.yaml | Add Gateway API HTTPRoute examples for Dex and Dashboard. |
| content/developer-platform/setup/quickstart/data/gateway.yaml | Add shared Gateway example with HTTPS listeners for login/dashboard. |
| content/developer-platform/setup/quickstart/data/ai-agent.values.yaml | Add AI Agent Helm values example for Quickstart. |
| content/developer-platform/setup/quickstart/data/ai-agent.http-route.yaml | Add Gateway API HTTPRoute example for AI Agent with URL rewrite. |
| content/developer-platform/setup/quickstart/_index.en.md | Expand Quickstart (Gateway API alternative + AI Agent step + updated versions/next steps). |
| content/developer-platform/service-providers/publish-resources/_index.en.md | Update PublishedResource docs for api-syncagent v0.3.x (templates, projection, versions, related schema text). |
| content/developer-platform/service-providers/crossplane/_index.en.md | Refresh Crossplane/cert-manager/provider docs + update PublishedResource example to syncagent api/v0.3 template syntax. |
| content/developer-platform/platform-users/rbac/_index.en.md | Update upstream kcp RBAC docs link to v0.30. |
| content/developer-platform/_index.en.md | Clarify org/project workspace hierarchy wording and add architecture image. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: julioc-p <14-10820@usb.ve>
Signed-off-by: julioc-p <14-10820@usb.ve>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 15 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
content/developer-platform/service-providers/publish-resources/_index.en.md
Outdated
Show resolved
Hide resolved
content/developer-platform/setup/quickstart/data/ai-agent.values.yaml
Outdated
Show resolved
Hide resolved
content/developer-platform/setup/quickstart/data/kdp-dashboard.values.yaml
Outdated
Show resolved
Hide resolved
content/developer-platform/service-providers/publish-resources/_index.en.md
Outdated
Show resolved
Hide resolved
Signed-off-by: julioc-p <14-10820@usb.ve>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 15 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
neolit123
left a comment
neolit123
left a comment
There was a problem hiding this comment.
i went trough the diff and didn't find any thing out of place.
i might be lacking some understanding on the architecture and usage of external components. this looks good to me though. best to have a second review from another human too.
|
/lgtm |
|
LGTM label has been added. DetailsGit tree hash: 2336eb1ea35fd69fcf7729b3da2c95a6d492db62 |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: wojciech12 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
kubermatic-bot
added
the
approved
4 participants
No description provided.