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

support bringing own AKS cluster to deploy #26

Open
ashishonce opened this issue Aug 4, 2020 · 10 comments
Open

support bringing own AKS cluster to deploy #26

ashishonce opened this issue Aug 4, 2020 · 10 comments
Assignees
Labels
enhancement New feature or request

Comments

@ashishonce
Copy link
Member

this links to this issue
https://github.com/github/ms-ecosystem/issues/85

@ashishonce ashishonce added the enhancement New feature or request label Aug 4, 2020
@marvinbuss
Copy link
Contributor

@ashishonce I cannot open the link. What is the issue about?

@vivishno
Copy link
Contributor

vivishno commented Aug 6, 2020

@marvinbuss the issue is linked to https://github.com/github/ms-ecosystem/issues/85
We want to deploy our model to a azure kubernetes service outside the ml workspace. For now we will support deploying to a aks inside the same subscription and same resource group.

@ashishonce
Copy link
Member Author

to expand :
today user has to provide an AKS cluster which is already attached to a workspace, else workspace is not able to identify the cluster.
The PM task has an ask to support the AKS Cluster which is outside the workspace, so for that, it can be first attached to ws and then can be used to deploy.
attaching is a one time job, so we can decide , how we can to support this

@marvinbuss
Copy link
Contributor

If we want to support attaching AKS, we should add this functionality to Azure/aml-compute. This task is compute related.
If the compute is not attached to AML, I would suggest to use these Actions: https://github.com/Azure/actions#deploy-to-kubernetes .

@ashishonce
Copy link
Member Author

@pulkitaggarwl for visibility , can you please put your thoughts?

@pulkitaggarwl
Copy link
Contributor

@marvinbuss We want customer be able to deploy to an existing cluster. Generally customers don't spin up a new K8S cluster for new services but instead use the same one.
Customer should not ideally need to use aml-compute in their workflow for this scenario.
As per, Deploy to K8s action - its a generic K8S action and would understand container images not Az ML models. Also, if we go that route we would need to build similar for ACI, FPGAs, Functions.

@marvinbuss
Copy link
Contributor

@pulkitaggarwl I do understand the use case. Existing clusters should either be attached to AML via ARM templates or via aml-compute. Handling compute related tasks in aml-deploy would be highly inconsistent, since this action is about creating docker images and deploying these to attached compute targets or just providing the image, so that it can be used with other actions, such as:

@awmatheson Do you have any recommendations?

@vivishno
Copy link
Contributor

vivishno commented Aug 7, 2020

@pulkitaggarwl can you give your inputs on this.
Following are the code changes made to aml-deploy action-
master...vivishno:master

With these changes the action will look for the aks cluster specified inside ml workspace and also see if it is attached.
If not found it will look for it in the resource group and will attach it if found.
If not found inside the ML workspace or in the resource group it will deploy the model to ACI.

@vivishno
Copy link
Contributor

vivishno commented Aug 7, 2020

We also need to decide on what behavior should be when the aks cluster specified in the config file is not found inside ML workspace or inside resource grp. Currently it deploys to ACI if cluster not found. Have raised a bug around this-
#27
If this is the expectation then we will have to document this as well.

@ashishonce
Copy link
Member Author

@vivishno just had a chat with @pulkitaggarwl and closed on that .. we can put this into aml-compute.

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

No branches or pull requests

4 participants