Skip to content

Conversation

@royischoss
Copy link
Contributor

@royischoss royischoss commented Dec 11, 2025

πŸ“ Description

Fix for evidently missing requirement sniffio


βœ… Checklist

  • I updated the documentation (if applicable)
  • I have tested the changes in this PR
  • I confirmed whether my changes are covered by system tests
    • If yes, I ran all relevant system tests and ensured they passed before submitting this PR
    • I updated existing system tests and/or added new ones if needed to cover my changes
  • If I introduced a deprecation:

πŸ§ͺ Testing

test_app_flow[True, True]


πŸ”— References


🚨 Breaking Changes?

  • Yes (explain below)
  • No

πŸ”οΈ Additional Notes

@assaf758 assaf758 merged commit 4c90a53 into mlrun:1.10.x Dec 11, 2025
18 checks passed
jond01 added a commit to jond01/mlrun that referenced this pull request Dec 16, 2025
assaf758 pushed a commit that referenced this pull request Dec 17, 2025
…ts [1.10.x] (#9067)

### πŸ“ Description

Revert #9045 following [`litestar`
2.19.0](https://github.com/litestar-org/litestar/releases/tag/v2.19.0).

---

### πŸ› οΈ Changes Made

Remove the explicit `sniffio` requirement from the Evidently app and
update the lock files.

---

### βœ… Checklist

- [x] I have tested the changes in this PR
- [x] I confirmed whether my changes are covered by system tests

---

### πŸ§ͺ Testing

I ran
`"tests/system/model_monitoring/test_app.py::TestMonitoringAppFlow::test_app_flow[True-True]"`
and it passed successfully.

---

### πŸ”— References

- Ticket link: [ML-11776](https://iguazio.atlassian.net/browse/ML-11776)

---

### 🚨 Breaking Changes?

- [x] No


[ML-11776]:
https://iguazio.atlassian.net/browse/ML-11776?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
jond01 added a commit to jond01/mlrun that referenced this pull request Dec 17, 2025
…ts [1.10.x] (mlrun#9067)

### πŸ“ Description

Revert mlrun#9045 following [`litestar`
2.19.0](https://github.com/litestar-org/litestar/releases/tag/v2.19.0).

---

### πŸ› οΈ Changes Made

Remove the explicit `sniffio` requirement from the Evidently app and
update the lock files.

---

### βœ… Checklist

- [x] I have tested the changes in this PR
- [x] I confirmed whether my changes are covered by system tests

---

### πŸ§ͺ Testing

I ran
`"tests/system/model_monitoring/test_app.py::TestMonitoringAppFlow::test_app_flow[True-True]"`
and it passed successfully.

---

### πŸ”— References

- Ticket link: [ML-11776](https://iguazio.atlassian.net/browse/ML-11776)

---

### 🚨 Breaking Changes?

- [x] No

[ML-11776]:
https://iguazio.atlassian.net/browse/ML-11776?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
# Conflicts:
#	dockerfiles/gpu/locked-requirements.txt
#	dockerfiles/jupyter/locked-requirements.txt
#	dockerfiles/mlrun-api/locked-requirements.txt
#	dockerfiles/mlrun-kfp/locked-requirements.txt
#	dockerfiles/mlrun/locked-requirements.txt
#	dockerfiles/test-system/locked-requirements.txt
#	dockerfiles/test/locked-requirements.txt
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