test(bdd): align local GPU instance types with two-GPU topology
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 218
- Forks
- 72
- Avg merge
- 1d 12h
- Merged PRs (30d)
- 427
Description
Description
Follow up after #999 merges.
PR #999 intentionally changes the shared local fake GPU topology from gpuCount: 128 to gpuCount: 2 so NVCA registers the one-GPU instance type required by the LLM routing scenario. Several local k3d BDDs still deploy NCP.GPU.H100_8x, which is not compatible with the adjusted topology.
Update the local single-cluster, multi-cluster, and NVCT task-smoke BDD fixtures to select an instance type actually registered by the two-GPU topology. Keep the gpuCount: 2 adjustment from #999. Do not change the EKS BDD, which uses a real cluster topology.
Definition of Done
- Update local single-cluster Helmfile BDD deployments that request
NCP.GPU.H100_8x. - Update local multi-cluster Helmfile BDD deployments that request
NCP.GPU.H100_8x. - Update the local NVCT task-smoke default instance type.
- Preserve
topology.nodePools.default.gpuCount: 2. - Verify the selected instance type is registered before deployment.
- Run the affected local BDD scenarios and document the results.
Resources (optional)
- #999
tools/ncp-local-cluster/apps/fake-gpu-operator/values.yamltests/bdd/features/single-cluster-helmfile.featuretests/bdd/features/multi-cluster-helmfile.featuretests/bdd/scripts/run-nvct-task-smoke.sh
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with tools/ncp-local-cluster/apps/fake-gpu-operator/values.yaml to confirm the instance types registered by the two-GPU topology. Update the local deployments in tests/bdd/features/single-cluster-helmfile.feature and multi-cluster-helmfile.feature, plus the default in tests/bdd/scripts/run-nvct-task-smoke.sh; leave EKS unchanged. Verify registration, run the affected local BDD scenarios, and document the results while preserving gpuCount: 2.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- bash, helm, kubernetes
- Domain
- devops, infrastructure, testing
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Clearly specified
- Newbie friendliness
- 75/100