NVIDIA / NVIDIA/nvcf

test(bdd): align local GPU instance types with two-GPU topology

Open
#1,078 0 comments 0 reactions 0 assignees View on GitHub

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.yaml
  • tests/bdd/features/single-cluster-helmfile.feature
  • tests/bdd/features/multi-cluster-helmfile.feature
  • tests/bdd/scripts/run-nvct-task-smoke.sh

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.