add kueue components as an option

[kannon92]

Summary

Add kueue recipe.

Motivation / Context

Fixes: #486
Related:

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Refactoring (no functional changes)
• Build/CI/tooling

Component(s) Affected

• CLI (cmd/aicr, pkg/cli)
• API server (cmd/aicrd, pkg/api, pkg/server)
• Recipe engine / data (pkg/recipe)
• Bundlers (pkg/bundler, pkg/component/*)
• Collectors / snapshotter (pkg/collector, pkg/snapshotter)
• Validator (pkg/validator)
• Core libraries (pkg/errors, pkg/k8s)
• Docs/examples (docs/, examples/)
• Other: ____________

Implementation Notes

Testing

# Commands run (prefer `make qualify` for non-trivial changes)
make qualify

Risk Assessment

• Low — Isolated change, well-tested, easy to revert
• Medium — Touches multiple components or has broader impact
• High — Breaking change, affects critical paths, or complex rollout

Rollout notes:

Checklist

• Tests pass locally (make test with -race)
• Linter passes (make lint)
• I did not skip/disable tests to make CI green
• I added/updated tests for new functionality
• I updated docs if user-facing behavior changed
• Changes follow existing patterns in the codebase
• Commits are cryptographically signed (git commit -S) — GPG signing info

[copy-pr-bot]

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

[ArangoGutierrez]

Hey Kevin, thanks for adding this. A couple things to fix before this is ready.

[ArangoGutierrez]

This path is wrong. The bundler constructs {Repository}/{ChartName}, so this resolves to oci://registry.k8s.io/kueue/kueue but the chart actually lives at oci://registry.k8s.io/kueue/charts/kueue.

Should be:

defaultRepository: oci://registry.k8s.io/kueue/charts

You can verify with:

helm show chart oci://registry.k8s.io/kueue/charts/kueue --version v0.17.0

[ArangoGutierrez]

Worth checking what the upstream chart defaults are for resource requests and security context. If they look good, a short comment here saying so would help (see kai-scheduler/values.yaml for an example of documenting why values are minimal). If not, we should pin them explicitly.

[kannon92]

I'm having a hard time figuring out how to test this on a kind cluster.

I was thinking that I could have a bundle and verify simple kind cluster for Kueue but can't figure out how to do that.

[ArangoGutierrez]

I'm having a hard time figuring out how to test this on a kind cluster.

I was thinking that I could have a bundle and verify simple kind cluster for Kueue but can't figure out how to do that.

In response to this comment, I have created #508 , with it, is easier for contributors to fully test the component in a controlled environment, even if the contributor doesn't have access to a GPU by using the NVML-Mock project.

Hope this helps @kannon92

[yuanchen8911]

Cross-Review Summary for PR #490

Reviewers: Codex, CodeRabbit + Integration Analysis
Rounds: 1 + Codex follow-up
Consensus reached: Yes — no confirmed defects

Confirmed Issues

None. The PR is correct as written.

Positive Observations

• Registry entry follows established patterns — displayName, version quoting, OCI URL, nodeScheduling structure all consistent with existing components
• Health check mirrors kubeflow-trainer/kai-scheduler pattern (deployment exists + no unhealthy pods in namespace)
• controllerManager.tolerations: [{operator: Exists}] matches kai-scheduler/nvsentinel/network-operator convention
• Kueue webhook runs in the same controller-manager pod (verified against upstream chart v0.17.0), so wiring only controllerManager.* scheduling paths is correct
• All 3 files automatically embedded via existing //go:embed directives — no Go code changes needed
• No existing kueue references in codebase — clean integration

---

Cross-review by Claude Code + Codex + CodeRabbit

[yuanchen8911]

/lgtm