Skip to content

Conversation

@mimarcel
Copy link
Contributor

@mimarcel mimarcel commented Oct 30, 2025

Why

  • CLARIFAI_ROOT_CERTIFICATES_PATH env variable is not used when starting a runner server

How

  • When initializing an API client (BaseClient), set arguments from environment (ClarifaiAuthHelper.from_env)
  • Update ClarifaiAuthHelper.from_env to set user-provided arguments and default to environment arguments

@mimarcel mimarcel requested a review from Copilot October 30, 2025 16:35
@github-actions
Copy link

Code Coverage

Package Line Rate Health
clarifai 45%
clarifai.cli 44%
clarifai.cli.templates 33%
clarifai.client 67%
clarifai.client.auth 67%
clarifai.constants 100%
clarifai.datasets 100%
clarifai.datasets.export 80%
clarifai.datasets.upload 75%
clarifai.datasets.upload.loaders 37%
clarifai.models 100%
clarifai.modules 0%
clarifai.rag 72%
clarifai.runners 53%
clarifai.runners.models 59%
clarifai.runners.pipeline_steps 41%
clarifai.runners.pipelines 70%
clarifai.runners.utils 63%
clarifai.runners.utils.data_types 72%
clarifai.schema 100%
clarifai.urls 60%
clarifai.utils 60%
clarifai.utils.evaluation 67%
clarifai.workflows 95%
Summary 62% (8208 / 13324)

Minimum allowed line rate is 50%

@mimarcel mimarcel merged commit fb78938 into master Oct 30, 2025
22 of 23 checks passed
@mimarcel mimarcel deleted the mimarcel/DE-203-base-client-use-env-variables branch October 30, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants