Skip to content

🔨 add HTTPBasicAuth for supporting MLFlow authentication#193

Open
hunhoon21 wants to merge 1 commit into
mlflow:masterfrom
hunhoon21:enhance/add-support-for-authentication
Open

🔨 add HTTPBasicAuth for supporting MLFlow authentication#193
hunhoon21 wants to merge 1 commit into
mlflow:masterfrom
hunhoon21:enhance/add-support-for-authentication

Conversation

@hunhoon21
Copy link
Copy Markdown

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
Copy Markdown
Author

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

@hunhoon21
Copy link
Copy Markdown
Author

Are you guys there?

@amesar @smurching @dbczumar @mingyu89

@adamantivm
Copy link
Copy Markdown

ping @amesar @smurching @dbczumar @mingyu89

@robmarkcole
Copy link
Copy Markdown

Plus 1

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

3 participants