Skip to content

fix(kagent): remove duplicate default-model-config from kagent-setup#10

Open
allamand wants to merge 1 commit into
mainfrom
fix/kagent-model-config-conflict
Open

fix(kagent): remove duplicate default-model-config from kagent-setup#10
allamand wants to merge 1 commit into
mainfrom
fix/kagent-model-config-conflict

Conversation

@allamand

Copy link
Copy Markdown

Resolves ArgoCD resource conflict where both kagent-hub and kagent-setup-hub apps manage the same ModelConfig/default-model-config resource.

Changes:

  • Remove default-model-config from kagent-setup chart (was a duplicate)
  • Configure kagent-hub's upstream chart to use Bedrock/LiteLLM via providers.openAI values

This eliminates the perpetual OutOfSync state caused by both apps fighting over the same resource.

Move the Bedrock/LiteLLM model configuration to kagent-hub values
(providers.openAI) so the upstream chart's default-model-config is
configured correctly without a conflicting resource from kagent-setup.
@elamaran11

Copy link
Copy Markdown
Contributor

@allamand Same comment. Is this required still. we have removed Kagent from our design

elamaran11 added a commit that referenced this pull request Jul 10, 2026
…-to-end

Live test success: pod under kata-clh ran with guest kernel 6.18.35 vs host
6.12.90 = real VM isolation. The real crux (NOT containerd restart / nydus —
those were red herrings): self-managed MNG nodes on Auto Mode need BOTH vpc-cni
AND kube-proxy addons, else kata-deploy can't reach kubernetes.default.svc and
crashloops. Plus the runtime-not-ready startup-taint gate (openclaw PR #10) and
an IAM access-entry-recreate gotcha. Full working recipe documented.
elamaran11 added a commit that referenced this pull request Jul 10, 2026
…int)

Ported from openclaw PR #10 — the missing piece our chart lacked. Waits for
kata-deploy Ready on each kata node, then removes the runtime-not-ready startup
taint so kata sandboxes can schedule. Without it, Sandbox pods stay Pending
(live-test finding). Verified: renders + chart lints clean.
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.

2 participants