Skip to content

Conversation

@jond01
Copy link
Member

@jond01 jond01 commented Jan 8, 2026

📝 Description

This PR fixes the import mlrun errors introduced from 1.11.0-rc14.
A new nuclio-jupyter version removed the boto3 (nuclio/nuclio-jupyter#194):

  1. Remove boto3 from required deps - Moved to optional [boto3] extra for backward compatibility

And it revealed two bugs:

  1. import mlrun boils down to import boto3 and import botocore in other imported mlrun modules.
  2. The package test is faulty. This is a separate issue, to be handled in ML-11948.

This PR moved the boto3/botocore imports to the mlrun.datastore.s3 module, where they should be (and only there).
It also adds unit tests, removed unneeded imports, and moves some code the the proper module.


✅ Checklist

  • I have tested the changes in this PR

🧪 Testing

I packaged mlrun locally with

make package-wheel

Created a clean Python 3.11 venv, installed the mlrun wheel in it, and ran

python -c "import mlrun"

🔗 References


🚨 Breaking Changes?

There is a small breaking change in the error returned from S3Store.get. When the object does not exist, a FileNotFound error will be raised.

@assaf758 assaf758 merged commit 8fab4b0 into mlrun:development Jan 10, 2026
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants