Skip to content
This repository was archived by the owner on Aug 11, 2020. It is now read-only.

Commit 9cf1e05

Browse files
bbathaBartoszCki
authored andcommitted
fix: unsupported machineTypes listed in deployment flags PS-10082
1 parent d91504f commit 9cf1e05

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

paperspace/cli/deployments.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ def deployments():
2121
)
2222
)
2323

24+
DEPLOYMENT_MACHINE_TYPES = ("G1", "G6", "G12",
25+
"K80", "P100", "V100")
2426

2527
@deployments.command("create", help="Create new deployment")
2628
@click.option(
@@ -45,7 +47,7 @@ def deployments():
4547
@click.option(
4648
"--machineType",
4749
"machineType",
48-
type=click.Choice(constants.MACHINE_TYPES),
50+
type=click.Choice(DEPLOYMENT_MACHINE_TYPES),
4951
required=True,
5052
help="Type of machine for new deployment",
5153
)

tests/functional/test_deployments.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class TestDeploymentsCreate(object):
1818
"--deploymentType", "tfserving",
1919
"--modelId", "some_model_id",
2020
"--name", "some_name",
21-
"--machineType", "Air",
21+
"--machineType", "G1",
2222
"--imageUrl", "https://www.latlmes.com/breaking/paperspace-now-has-a-100-bilion-valuation",
2323
"--instanceCount", "666",
2424
]
@@ -27,13 +27,13 @@ class TestDeploymentsCreate(object):
2727
"--deploymentType", "tfserving",
2828
"--modelId", "some_model_id",
2929
"--name", "some_name",
30-
"--machineType", "Air",
30+
"--machineType", "G1",
3131
"--imageUrl", "https://www.latlmes.com/breaking/paperspace-now-has-a-100-bilion-valuation",
3232
"--instanceCount", "666",
3333
"--apiKey", "some_key",
3434
]
3535
BASIC_OPTIONS_REQUEST = {
36-
"machineType": u"Air",
36+
"machineType": u"G1",
3737
"name": u"some_name",
3838
"imageUrl": u"https://www.latlmes.com/breaking/paperspace-now-has-a-100-bilion-valuation",
3939
"deploymentType": "Tensorflow Serving on K8s",

0 commit comments

Comments
 (0)