Skip to content

Conversation

@gtopper
Copy link
Collaborator

@gtopper gtopper commented Dec 17, 2025

Following regression introduced in #9006 / v1.10.0-rc1.

ML-11732

Includes some small tangential improvements / fixes.

A system test was added to test_aws_s3.py. I ran it against MinIO and it

  • failed without the fix in this PR
  • passed with the fix

@gtopper gtopper merged commit 12b85c5 into mlrun:development Dec 17, 2025
14 checks passed
gtopper added a commit to gtopper/mlrun that referenced this pull request Dec 17, 2025
…tastores [1.10.x]

Backport of mlrun#9072.

Following regression introduced in mlrun#9006 / v1.10.0-rc1.

[ML-11732](https://iguazio.atlassian.net/browse/ML-11732)

Includes some small tangential improvements / fixes.

A system test was added to `test_aws_s3.py`. I ran it against MinIO and
it
* failed without the fix in this PR
* passed with the fix

[ML-11732]:
https://iguazio.atlassian.net/browse/ML-11732?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Co-authored-by: Gal Topper <[email protected]>
assaf758 pushed a commit that referenced this pull request Dec 17, 2025
…tastores [1.10.x] (#9078)

Backport of #9072.

Following regression introduced in #9006 / v1.10.0-rc1.

[ML-11732](https://iguazio.atlassian.net/browse/ML-11732)

Includes some small tangential improvements / fixes.

A system test was added to `test_aws_s3.py`. I ran it against MinIO and
it
* failed without the fix in this PR
* passed with the fix

[ML-11732]:
https://iguazio.atlassian.net/browse/ML-11732?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

Co-authored-by: Gal Topper <[email protected]>
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