Skip to content

feat(talos): enable the pod certificate and cluster trust bundle features - #5896

Merged
devantler merged 1 commit into
mainfrom
claude/kagent-pod-certificates
Nov 7, 2026
Merged

devantler merged 1 commit into
mainfrom
claude/kagent-pod-certificates

Conversation

@devantler

Copy link
Copy Markdown
Contributor

🤖 Generated by the Agentic Engineer

Why

The kagent Agentic Engineer lane runs on kagent 1.0, whose sandbox runtime (Substrate) gives every workload its identity as a pod certificate. Kubernetes 1.36 ships pod certificates and cluster trust bundles switched off, so Substrate cannot install on the cluster today.

What

Switches on the beta pod certificate and cluster trust bundle features in the control plane and on every node. The deploy rolls the control-plane components one at a time; nothing else changes until Substrate is installed in a follow-up PR.

🤖 Generated with Claude Code

Part of #5892

Substrate, the sandbox runtime under kagent 1.0's Claude Code harness,
issues workload identities as Pod Certificates and publishes its CAs as
ClusterTrustBundles. Kubernetes 1.36 ships both APIs as beta and off by
default, so serve certificates.k8s.io/v1beta1 and enable the
ClusterTrustBundle, ClusterTrustBundleProjection and PodCertificateRequest
gates on the API server, controller manager, scheduler and kubelet, as
Substrate's reference cluster does. The kubelet readback fixture now
declares the same gates.

Part of #5892

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this to 🫴 Ready in 🌊 Project Board Nov 7, 2026
@devantler
devantler marked this pull request as ready for review November 7, 2026 16:17
@devantler

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor
Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai

coderabbitai Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

Both Talos cluster configuration patches enable ClusterTrustBundle, ClusterTrustBundleProjection, and PodCertificateRequest for the API server, controller manager, scheduler, and kubelet. They also enable certificates.k8s.io/v1beta1 through API server runtime configuration. The healthy kubelet configuration fixture now includes all three feature gates set to true.

Possibly related issues

Merge Risk: Minimal · up to 2af89

The certificate configuration is ready to merge after normal checks; no blocking issue was identified.

Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed The title describes the pod certificate and cluster trust bundle features enabled by the changes.
Description check Passed The description relates to the feature gates enabled in the Talos cluster configurations.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files. (2 skipped: 2 unsupported.)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between ee5911d and 2af897a.

Files selected for processing: 3

  • scripts/tests/test-kubelet-config-readback.sh
  • talos-local/cluster/enable-pod-certificates.yaml
  • talos/cluster/enable-pod-certificates.yaml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Recent review details

Additional context used

Code guidelines: 1

AGENTS.md — auto-discovered
Path-based instructions: 1

Source excerpt: **Talos machine-config patches** (`talos/`, `talos-local/`) follow the same spirit: **one YAML document per file** and intent-describing `‹verb›-‹purpose›.yaml` names (e.g.

CodeRabbit inference engine (AGENTS.md)

Files:

  • talos/cluster/enable-pod-certificates.yaml
  • talos-local/cluster/enable-pod-certificates.yaml
Additional comments: 3

talos/cluster/enable-pod-certificates.yaml: 1

1-31: LGTM!

talos-local/cluster/enable-pod-certificates.yaml: 1

1-31: LGTM!

scripts/tests/test-kubelet-config-readback.sh: 1

88-88: LGTM!

@devantler
devantler added this pull request to the merge queue Nov 7, 2026
Merged via the queue into main with commit dacd072 Nov 7, 2026
51 checks passed
@devantler
devantler deleted the claude/kagent-pod-certificates branch November 7, 2026 22:37
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board Nov 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant