Skip to content

just a small change i forgot to push earlier - #37

Closed
sagoyal22 wants to merge 35 commits into
mainfrom
skills
Closed

sagoyal22 wants to merge 35 commits into
mainfrom
skills

Conversation

@sagoyal22

Copy link
Copy Markdown
Collaborator

What does this PR do?

A one-paragraph summary. Which component (operator / daemon / cli / shared / docs / chart)? What behavior changes?

Why?

Link the Issue this closes (Fixes #NNN) or explain the motivation if there's no filed Issue.

How was this tested?

  • Unit tests added or updated
  • make test passes locally
  • make integration passes locally (envtest)
  • Manually verified against a live cluster (describe how)

Breaking changes

  • No breaking changes
  • Breaking API change (CRD schema, admission webhook rule, CLI flag) — CHANGELOG updated with migration notes

Docs

  • Docs updated (README.md, docs/usage.md, or relevant per-component README)
  • CHANGELOG.md entry added under the next unreleased version heading
  • N/A (this PR has no user-visible change)

Checklist

  • Commits are signed off (git commit -s)
  • Code is formatted with gofmt -s
  • make vet passes
  • Any new source files carry the SPDX header

manasvik06 and others added 30 commits October 14, 2026 20:39
Agent skill that rotates a Principal's keytab with kubectl krb
rotate-keytab: read-only precheck (permissions, Principal, daemons,
keytab principal + KVNO), mandatory user approval, then independent
verification (every node Minted after the rotation, Ready, expected
KVNO, no new KerberosMintFailed events). Never prints key material.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Read-only inspection of Kerberator Tenants and Principals via kubectl-krb.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
… password

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
…T3 and 15 cases

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
…entials

CODEOWNERS (on the skills branch only): PRs touching .claude/ now request
review from the skills team instead of everything going to the default
owner. Everything else is unchanged.

.gitignore: never commit keytabs (keytabs/, *.keytab), kubeconfigs
(*.kubeconfig) or Kerberos ticket caches (krb5cc_*).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: manasvik06 <manasvi.spc@gmail.com>
Read-only triage for a Principal that can't get a Kerberos ticket.
scripts/collect.sh gathers the evidence in one pass: Principal and
Tenant realm, the KDC address from krb5.conf, per-node status, daemon
pods (and why one isn't running), Kerberos events with untruncated
messages, the daemon log lines carrying kinit's own error, every
Principal in the namespace, and a SIGNALS summary. SKILL.md classifies
from those signals with confidence labels and hands off to DG-03,
OP-04, OP-05 or OP-01 instead of fixing anything.
references/kinit-errors.md maps kinit errors to likely causes.

Tested collect.sh on the team cluster: a failing Principal (key
rejected on 2/2 nodes), a healthy one (no false alarm), and a UID
given instead of the CR name.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: manasvik06 <manasvi.spc@gmail.com>
…ng-key

From review:
- collect.sh sections 4, 5 and 7 tested $? after other assignments, so
  a failed kubectl call never printed UNAVAILABLE. Capture rc=$?
  immediately after each call.
- "Keytab contains no suitable keys" (e.g. a keytab for a different
  principal) fell through to unrecognized. Add keytab-no-matching-key
  for it and "Key table entry not found"; enctype errors become their
  own enctype-unsupported signature.

Also: when events have expired or can't be read, fall back to the
per-node status message for error_signature instead of reporting none.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: manasvik06 <manasvi.spc@gmail.com>
feat(skills): add kerberator-diagnose-auth-failure skill (DG-06)
Signed-off-by: Anvesha Girish <anveshagirish@gmail.com>
Signed-off-by: Saniya Goyal <saniyagoyal22@gmail.com>
Signed-off-by: Anvesha Girish <anveshagirish@gmail.com>
Signed-off-by: George Sukhotin <sukhotin@wisc.edu>
…; agent runs pending)

Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
…own files w instructions for special cases
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Vikram Varikooty <189786398+varikooty@users.noreply.github.com>
test(skills): Add automated agent evaluation framework and OP-03 skill
skills: adding create-tenant skill (OP-02)
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.

6 participants