Skip to content

Commit e5618ea

Browse files
committed
Tests for AWS neuron paths
1 parent 855ded2 commit e5618ea

1 file changed

Lines changed: 19 additions & 0 deletions

File tree

api/test/test_aws_provider.py

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,25 @@ def test_cpu_picks_cpu_ami(self, provider):
9797
cpu.assert_called_once()
9898

9999

100+
class TestBuildUserDataNeuron:
101+
"""Neuron launches must activate the AMI's prebuilt venv + Neuron PATH; others must not."""
102+
103+
def test_neuron_activates_prebuilt_venv(self):
104+
ud = AWSProvider._build_user_data(ClusterConfig(accelerators="Trainium:1"), region="us-east-1")
105+
assert "/opt/aws/neuron/bin" in ud
106+
assert "aws_neuronx_venv_pytorch" in ud
107+
assert "activate" in ud
108+
109+
def test_nvidia_has_no_neuron_block(self):
110+
ud = AWSProvider._build_user_data(ClusterConfig(accelerators="A100:8"), region="us-east-1")
111+
assert "aws_neuronx_venv" not in ud
112+
assert "/opt/aws/neuron/bin" not in ud
113+
114+
def test_cpu_has_no_neuron_block(self):
115+
ud = AWSProvider._build_user_data(ClusterConfig(cpus=4, memory=16), region="us-east-1")
116+
assert "aws_neuronx_venv" not in ud
117+
118+
100119
class TestResolveCpuInstanceType:
101120
def test_no_requirements_returns_minimum(self):
102121
assert _resolve_cpu_instance_type(None, None) == "c5.large"

0 commit comments

Comments
 (0)