Skip to content

Feat/idle workload detector#32

Merged
btwshivam merged 2 commits into
mainfrom
feat/idle-workload-detector
May 18, 2026
Merged

Feat/idle workload detector#32
btwshivam merged 2 commits into
mainfrom
feat/idle-workload-detector

Conversation

@btwshivam
Copy link
Copy Markdown
Member

@btwshivam btwshivam commented May 18, 2026

What

Why

Fixes #

How

Testing

  • make lint test passes locally
  • go test -race ./... passes
  • If a new rule: added a chart fixture in testdata/ and a golden output
  • Tested locally with:

Checklist

  • PR title follows Conventional Commits (feat(scope): subject)
  • All commits are DCO-signed (git commit -s)
  • No unrelated changes pulled in
  • Documentation updated where user-visible behavior changed
  • No LLM calls or telemetry introduced
  • No proprietary backend imports

btwshivam added 2 commits May 18, 2026 05:53
@btwshivam
chore(docs): expand CLAUDE.md to full operating manual
2de15f1 
Signed-off-by: btwshivam <shivam200446@gmail.com>
@btwshivam
feat(rules): add idle-workload detector (replicas=0, no HPA)
c799f78 
Signed-off-by: btwshivam <shivam200446@gmail.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation testing Test coverage and golden fixtures area/rules Cost / security rules engine size/XL 501+ lines (consider splitting) labels May 18, 2026
@btwshivam btwshivam enabled auto-merge (squash) May 18, 2026 00:24
@btwshivam btwshivam disabled auto-merge May 18, 2026 00:46
@btwshivam btwshivam merged commit 53a2e20 into main May 18, 2026
11 of 13 checks passed
btwshivam added a commit that referenced this pull request May 24, 2026
@btwshivam
Merge branch 'main' into feat/openapi-spec
bbf9e81 
Resolved conflicts by taking our branch's TDT-consolidated tests and
godoc-restored source files (needed for revive); merged the two
CLAUDE.md edits to keep the no-`_TableDriven`-suffix rule + main's
'every detector needs positive+negative' and 90% pkg/ coverage bullets.

Bumped TestAll_DetectorCountAndUniqueIDs from 30 to 31 to absorb
the new idle-workload detector (#32).

Signed-off-by: btwshivam <shivam200446@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area/rules Cost / security rules engine documentation Improvements or additions to documentation size/XL 501+ lines (consider splitting) testing Test coverage and golden fixtures

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@btwshivam