Skip to content

Conversation

@lpenet
Copy link
Contributor

@lpenet lpenet commented Apr 11, 2022

To test:

  • build a kit with this branch
    • edit make-studio-package.sh to set the API_CLIENT_VERSION to this branch name
    • smtg like make && NOGEO=1 NOGEOIP=1 ./packagers/studio/make-studio-package.sh 11.0.0-20220410 ~/dss-releases
  • install it
  • test in a notebook (without encrypted RPC)
  • enable encrypted RPC: ./bin/dssadmin install-encrypted-rpc
  • test in a notebook

Simple code like:

import dataiku
import mlflow
import os

client = dataiku.api_client()
project = client.get_project("POUET")
mf = project.get_managed_folder("w6adRQEB")

#os.environ["MLFLOW_TRACKING_INSECURE_TLS"]="true"

project.setup_mlflow(managed_folder=mf)

with mlflow.start_run():
        mlflow.log_param("alpha", 1)
        mlflow.log_param("l1_ratio", 2)
        mlflow.log_metric("rmse", 3)
        mlflow.log_metric("r2", 4)
        mlflow.log_metric("mae", 5)

should be ok.

@lpenet lpenet added the team-mielpops Team MieL pOps label Apr 11, 2022
@lpenet lpenet added this to the V 11.0.0 milestone Apr 11, 2022
@lpenet lpenet requested a review from vthorey April 11, 2022 14:41
@shortcut-integration
Copy link

Copy link
Contributor

@vthorey vthorey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and LGTM

OK with custom certificate
Screenshot 2022-04-13 at 11 13 03
NOK if custom certificate and disable insecure_TLS
Screenshot 2022-04-13 at 11 12 38

@lpenet lpenet merged commit c84bec6 into master Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-mielpops Team MieL pOps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants