Skip to content

Commit

Permalink
[API] Add default artifact path to frontend spec endpoint response (m…
Browse files Browse the repository at this point in the history
  • Loading branch information
Tankilevitch authored Jan 26, 2022
1 parent 30af717 commit b3b7f74
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions mlrun/api/api/endpoints/frontend_spec.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ def get_frontend_spec(
function_deployment_mlrun_command=mlrun.builder.resolve_mlrun_install_command(),
auto_mount_type=config.storage.auto_mount_type,
auto_mount_params=config.get_storage_auto_mount_params(),
default_artifact_path=config.artifact_path,
)


Expand Down
1 change: 1 addition & 0 deletions mlrun/api/schemas/frontend_spec.py
Original file line number Diff line number Diff line change
Expand Up @@ -32,3 +32,4 @@ class FrontendSpec(pydantic.BaseModel):
function_deployment_mlrun_command: typing.Optional[str]
auto_mount_type: typing.Optional[str]
auto_mount_params: typing.Dict[str, str] = {}
default_artifact_path: str
1 change: 1 addition & 0 deletions tests/api/api/test_frontend_spec.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ def test_get_frontend_spec(
)
assert frontend_spec.default_function_image_by_kind is not None
assert frontend_spec.function_deployment_mlrun_command is not None
assert frontend_spec.default_artifact_path is not None
# fields UI expects to be in the template
assert (
mlrun.mlconf.httpdb.builder.docker_registry
Expand Down

0 comments on commit b3b7f74

Please sign in to comment.