Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add function to test for existence for webservice #386

Open
revodavid opened this issue Aug 26, 2020 · 1 comment
Open

Add function to test for existence for webservice #386

revodavid opened this issue Aug 26, 2020 · 1 comment
Labels

Comments

@revodavid
Copy link
Member

For CI/CD setups, it a standard pattern would be:

  • If a webservice doesn't exist, deploy it with deploy_model
  • If the model has already been deployed and the webservice already exists, update it with update_aci_webservice

However, there is no function to detect the existence of a webservice. The workaround is messy:

try({
    existing_service <- get_webservice(ws, 'model_name')
})

It would be easier to write CI/CD flows to update webservices if either:

  • get_webservice returned NULL if the named webservice is not found instead of throwing an error, or:
  • a new function to check for the existence of a webservice were added
@mx-iao
Copy link
Contributor

mx-iao commented Aug 27, 2020

First option makes sense - we'll look into catching the error and returning NULL in get_webservice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants