Repository navigation
[Automated] Draft docs (kagent): core: support cross-namespace ModelConfig and RemoteMCPServer with… - #666
Closed
github-actions[bot] wants to merge 1 commit into
Closed
[Automated] Draft docs (kagent): core: support cross-namespace ModelConfig and RemoteMCPServer with…#666github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
…onfig and RemoteMCPServer with… Signed-off-by: GitHub Action <action@github.com>
Contributor
|
Folded into #672. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
kagent-dev/kagent#3382 — core: support cross-namespace ModelConfig and RemoteMCPServer with allowedNamespaces
Docs versions:
1.x(first shipped in v1.0.0-rc1)No code-path rule covers 11 touched file(s). The planner could route these only by identifier search, so the candidate pages may miss what they changed, including in the other mode:
docs/architecture/configuration-and-compilation.mddocs/architecture/credential-injection.mddocs/architecture/cross-namespace-references.mddocs/architecture/egress-policies.mddocs/architecture/runtime-and-lifecycle.mdgo/core/internal/controller/collections.goWhat changes for users: AgentTemplates and inline Agent templates can reference shared ModelConfigs and RemoteMCPServers in another namespace by setting a separate
namespacefield. Harness memory and summarizer settings can also reference shared ModelConfigs. The target ModelConfig or RemoteMCPServer controls access withallowedNamespaces, and a denied update preserves the Agent's last successful revision for new Sessions.What the docs now say: The model provider, MCP tool, Harness memory, context management, operations, debug, and tools ecosystem pages now describe namespace-qualified references,
allowedNamespaces, target-local Secrets, and the last-successful-revision behavior. Provider setup pages now say that provider Secrets belong in the ModelConfig namespace, while their examples keep ModelConfigs and AgentTemplates together.Not verified: No cluster; the configuration is unrun, and internal links are unchecked.
How
kagent-3382was drafted, and what was not verifiedPlan, and what changed it
diff-truncated.mdsays the CRD templates and generated deepcopy file are outside the code diff, andreference/api-ref.mdis generated, so I left generated reference rows to the update-ref-docs workflow.Why a documentation change is necessary
The release note says that Agents can reference shared ModelConfigs and RemoteMCPServers across namespaces by using
allowedNamespaces. The diff changesAgentTemplate.spec.modelConfig, MCP server bindings, and HarnessmodelConfigReffields from local references to references with optional namespace fields. Existing pages said that ModelConfigs, MCP servers, memory models, and summarizer models had to be in the same namespace, and provider pages told readers to place Secrets with the AgentTemplates rather than with the ModelConfig that reads them.What changed on disk
docs-site/content/kagent/1.x/setup/model-providers/about-model-providers.md— AddedallowedNamespaces, cross-namespacemodelConfig.namespace, and separatename/namespaceguidance.docs-site/content/kagent/1.x/setup/model-providers/amazon-bedrock.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/anthropic.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/azure-ai-foundry.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/azure-openai.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/byo-agentgateway.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/byo-openai.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/gemini.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/mistral.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/ollama.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/openai.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/setup/model-providers/xai.md— Corrected Secret placement and same-namespace example wording.docs-site/content/kagent/1.x/skills-and-mcp/about-tools.md— Updated MCP binding scope and documentedmcp.server.namespace.docs-site/content/kagent/1.x/get-started/your-first-mcp-tool.md— Updated MCP field descriptions and reference-resolution failure guidance.docs-site/content/kagent/1.x/agents/declarative-agents.md— Updated Harness memory model reference guidance.docs-site/content/kagent/1.x/agents/agent-memory.md— Updated memory ModelConfig placement andmemory.modelConfigRef.namespace.docs-site/content/kagent/1.x/agents/context-management.md— Updated summarizer ModelConfig namespace and failure guidance.docs-site/content/kagent/1.x/operations/operational-considerations.md— Added recompilation and last-successful-revision behavior for namespace policy changes.docs-site/content/kagent/1.x/operations/debug.md— AddedallowedNamespacesdenial as aResolvedRefsfailure and explained the preserved revision.docs-site/content/kagent/1.x/reference/tools-ecosystem.md— Clarified cross-namespace binding tokagent-tool-server.How to verify this change
spec.allowedNamespaces.from: All, then create an AgentTemplate in another namespace withspec.modelConfig.nameandspec.modelConfig.namespaceset to that shared ModelConfig.spec.allowedNamespaces.from: Selector, label the AgentTemplate namespace to match, and bind it from the AgentTemplate withspec.tools[].mcp.server.namespace.kubectl get agent <agent-name> -n <agent-namespace> -o jsonpath='{.status.desiredRevision}{"\t"}{.status.latestSuccessfulRevision}{"\n"}'and expect both revision values to match after compilation succeeds.allowedNamespaces.fromtoSame, then inspect the Agent conditions.ResolvedRefsshould reportReferenceResolutionFailed, andlatestSuccessfulRevisionshould remain populated with the prior prepared revision.Proposed release note
No
proposed-release-note.md:markers.notes_placeableis false. Suggested note: Agents can reference shared ModelConfigs and RemoteMCPServers across namespaces by setting the referencenamespacefield and permitting the source namespace withallowedNamespaces. If a later namespace policy change denies the reference, kagent keeps the Agent's last successful revision available for new Sessions until another revision prepares successfully.What was not verified
go/api/v1alpha3/zz_generated.deepcopy.go,helm/kagent-crds/templates/api.kagent.dev_agents.yaml,helm/kagent-crds/templates/api.kagent.dev_agenttemplates.yaml,helm/kagent-crds/templates/api.kagent.dev_harnesses.yaml, andhelm/kagent-crds/templates/api.kagent.dev_modelconfigs.yamlwere beyond the truncated diff.ui/playwright/tests/agents/agent-crud.spec.ts,ui/src/components/agent-template-form/agentTemplateDraft.test.ts, andui/src/components/model-form/modelDraft.test.tswere beyond the truncated diff.How the pages were chosen (triage report)
Feature clusters
Kept apart on purpose. Merging wrongly produces a guide that documents two things and explains neither, so these are reported for a person to merge rather than grouped automatically:
docs-site/content/kagent/1.x/observability/launch-ui.mdas a candidate, but for one of them it is one page among more than 3, so they are kept apart; merge them if both drafts do land thereDraft a documentation change
kagent-dev/kagent#3382 — core: support cross-namespace ModelConfig and RemoteMCPServer with allowedNamespaces
kagent-dev/websitedocs/architecture/configuration-and-compilation.mddocs/architecture/credential-injection.mddocs/architecture/cross-namespace-references.mddocs/architecture/egress-policies.mddocs/architecture/runtime-and-lifecycle.mdgo/core/internal/controller/collections.goDraft branch:
pr-tracker-draft-kagent-3382