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 HTTPBasicAuth for supporting MLFlow authentication #193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hunhoon21
Copy link

What does this PR do?
This PR adds support for the authentication features introduced in MLflow 2.12 to the mlflow-export-import tool. It includes handling authentication tokens and necessary configurations to work with authenticated MLflow servers.

Why is this PR needed?
With the introduction of authentication in MLflow 2.12, it is essential for tools like mlflow-export-import to support these features. This addition will enable users to use the tool in secure environments where authentication is required, thus enhancing its usability and versatility.

What are the changes made in this PR?

  • Added support for passing authentication tokens when using mlflow-export-import.
  • Updated documentation to guide users on how to configure and use authentication with the tool.
  • Ensured backward compatibility for users not using the authentication feature.

How to test the changes?

  1. Set up an MLflow server with authentication enabled. (over 2.12.1)
  2. Configure the mlflow-export-import tool with the necessary authentication tokens.
  3. Perform export and import operations to verify that authentication is handled correctly.

Related Issues
Closes #192

@hunhoon21
Copy link
Author

Can you guys check it for me?
@amesar @smurching @dbczumar @mingyu89

@hunhoon21
Copy link
Author

Are you guys there?

@amesar @smurching @dbczumar @mingyu89

@adamantivm
Copy link

ping @amesar @smurching @dbczumar @mingyu89

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

Successfully merging this pull request may close these issues.

Add support for MLflow 2.12 authentication in mlflow-export-import
2 participants