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 integration test for dss create to check MLflow access #70

Open
orfeas-k opened this issue Apr 10, 2024 · 1 comment
Open

Add integration test for dss create to check MLflow access #70

orfeas-k opened this issue Apr 10, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@orfeas-k
Copy link
Contributor

Why it needs to get done

This is needed in order to ensure that created pods can access MLflow. This was initially suggested here by @misohu in the dss create PR.

What needs to get done

Write an integration test which will create a notebook and then it will run the script to create experiement in MLflow. e.g.

import mlflow
c = mlflow.MlflowClient()  # You should not need to specify the tracking_uri - it should be populated automatically
print(c.tracking_uri)  # should print the in-cluster url for dss's mlflow
c.create_experiment("test-experiment")  # Should succeed

The idea is to create notebook and then run kubectl exec with that script. Or we could also execute a jupyter notebook from inside the pod (which could be git clone and run).

Something like this (from PR) could also be used to run command from the pod.

When is the task considered done

There is integration test checking MLflow access.

@orfeas-k orfeas-k added the enhancement New feature or request label Apr 10, 2024
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-5568.

This message was autogenerated

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

1 participant