Skip to content
This repository has been archived by the owner on Sep 13, 2023. It is now read-only.

Commit

Permalink
Sagemaker deployments (#366)
Browse files Browse the repository at this point in the history
* WIP

* its alive (kinda)

* it works but it's ugly

* little less ugly

* lil fix

* fix lint

* fix lint

* fix tests

* fix tests

* fix windows bugs

* fix tests

* fix tests
  • Loading branch information
mike0sv authored and madhur-tandon committed Sep 14, 2022
1 parent 3c5dd97 commit d836ba4
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
5 changes: 5 additions & 0 deletions mlem/contrib/heroku/meta.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,11 @@ def _get_client(self, state: HerokuState) -> Client:
host=urlparse(state.ensured_app.web_url).netloc, port=80
)

def _get_client(self, state: HerokuState) -> Client:
return HTTPClient(
host=urlparse(state.ensured_app.web_url).netloc, port=80
)


class HerokuEnv(MlemEnv[HerokuDeployment]):
"""Heroku Account"""
Expand Down
3 changes: 3 additions & 0 deletions tests/cli/test_deployment.py
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,9 @@ class Config:
def _get_client(self, state) -> Client:
return HTTPClient(host="", port=None)

def _get_client(self, state) -> Client:
return HTTPClient(host="", port=None)


class MlemEnvMock(MlemEnv):
"""mock"""
Expand Down

0 comments on commit d836ba4

Please sign in to comment.