NVIDIA / holodeck / 22024718082

48%

main: 48%

Build Type

push

github

Committed by ArangoGutierrez

Commit Message

fix(aws): infer AMI architecture from instance type when unset

When image.architecture is not explicitly set, holodeck defaults to
x86_64 regardless of the instance type. This causes EC2 RunInstances
to fail with "Unsupported configuration" when an arm64-only instance
type (e.g., g5g, m7g, c7g) is paired with an x86_64 AMI.

Add inferArchFromInstanceType() that queries DescribeInstanceTypes to
determine the instance type's supported architectures. When the type
only supports arm64, architecture is automatically set to "arm64"
before AMI resolution. This is wired into all three resolution paths:
resolveOSToAMI (single-node), setLegacyAMI (legacy default), and
createInstances (cluster mode).

Backward compatible: dual-arch or x86_64-only instance types still
default to x86_64. Explicit image.architecture always takes precedence.

Ref: https://github.com/NVIDIA/gpu-driver-container/actions/runs/22012665274/job/63611032634
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>

Run Details

30 of 40 new or added lines in 2 files covered. (75.0%)

2 existing lines in 1 file now uncovered.

2609 of 5395 relevant lines covered (48.36%)

0.54 hits per line