Add kubectl bootstrap script #31

Merged

tepichord merged 5 commits from feat/30-kubectl-bootstrap-script into main

2026-09-08 16:44:04 -07:00

tepichord commented

2026-09-08 16:21:02 -07:00

Owner

Closes #30

Automated PR created by forgejo-done adapter.

Closes #30 Automated PR created by forgejo-done adapter.

tepichord added 4 commits

2026-09-08 16:21:02 -07:00

feat(control-plane): add use-kubectl.sh bootstrap script b1e3cb8935

Mirrors use-talos.sh to wire kubectl to the Milner Talos cluster.
Extracts kubeconfig from terraform output, handles fresh and existing
~/.kube/config with idempotent stale-entry cleanup, and uses the
KUBECONFIG=a:b kubectl config view --flatten workaround for merging
(kubectl has no built-in merge command, see kubectl#1868).

- chmod +x, shellcheck-clean
- Includes the script in the test_shellcheck test list

Refs: #30

docs: update runbook step 3 to use use-kubectl.sh e4fbee0dd5

Replace the manual kubeconfig extraction and --kubeconfig flag with
the new use-kubectl.sh script and plain 'kubectl get nodes'.

Refs: #30

docs(control-plane): document use-kubectl.sh in AGENTS.md, trim verbose inline comments 3f3331bbc0

Refs: #30

fix: handle sole-context edge case in use-kubectl.sh a21bf42ae8

When the target kubeconfig contains only our cluster's contexts, the
delete-then-merge path can fail because kubectl refuses to delete
entities referenced by the active context. Fall back to wholesale
replacement — matching the established pattern in use-talos.sh.

tepichord added the

reviewing

label

2026-09-08 16:21:02 -07:00

tepichord added this to the MVP project

2026-09-08 16:26:27 -07:00

tepichord added 1 commit

2026-09-08 16:43:26 -07:00

fix(control-plane): parse kubeconfig names with kubectl, not sed fa8a980cc8

use-kubectl.sh extracted context/cluster/user names by matching
map-style YAML keys ('  name:'), but every real kubeconfig here is
list-style ('- name: ...'): kubectl config set-*, Talos/terraform
output, and kubectl config view --flatten (the script's own output).
The parser therefore found nothing — routing configs down the
wholesale-replace path and silently deleting unrelated clusters — or
matched phantom keys such as the users section's 'user:' line and
skipped the stale-entry cleanup entirely.

- Extract section names via kubectl config view jsonpath, which parses
  both styles.
- Delete stale clusters/users by the freshly generated file's names:
  Talos names the cluster '<cluster>' (milner) but its context and
  user '<user>@<cluster>' (admin@milner), so name-equality with the
  context never matched the cluster entry and old CA/certs/server won
  the first-file-wins merge after a rebuild.
- Correct README wording: the kubectl context is admin@milner, not
  milner.

tepichord merged commit 57d06cb02b into main

2026-09-08 16:44:04 -07:00

tepichord deleted branch feat/30-kubectl-bootstrap-script

2026-09-08 16:44:04 -07:00

tepichord referenced this pull request from a commit

2026-09-08 16:44:05 -07:00

Add kubectl bootstrap script (#31)

tepichord referenced this pull request

2026-09-10 23:09:16 -07:00

Automate Cluster Deployments with Flux #46