Skip to content

Conversation

@royischoss
Copy link
Contributor

@royischoss royischoss commented Jan 5, 2026

📝 Description

Enable http async mode using serving graph in mlrun.
Bump nuclio-jupyter to 0.13.0


🛠️ Changes Made

  1. Make sync mode be the default mode of http trigger as before, by using with_http without AsyncSpec or as default http trigger created by mlrun
  2. Change the number of workers to 1 if user use with_http with multiple workers
  3. Test serving with new handler as async def signature as added in nuclio-jupyter v0.13.0
  4. Bump nuclio-jupyter to 0.13.0

✅ 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

Added system test test_async_http_mode_serving_graph


🔗 References


🚨 Breaking Changes?

  • Yes (explain below)
  • No

🔍️ Additional Notes

Both of this changes require a nuclio version of 1.15.3 and above

@royischoss royischoss marked this pull request as ready for review January 5, 2026 15:06
@royischoss royischoss requested review from a team and liranbg as code owners January 5, 2026 15:06
@royischoss royischoss requested a review from TomerShor as a code owner January 6, 2026 11:38
@TomerShor TomerShor requested a review from rokatyy January 6, 2026 15:40
Copy link
Contributor

@davesh0812 davesh0812 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rokatyy rokatyy left a comment

Choose a reason for hiding this comment

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

LGTM

@davesh0812 davesh0812 merged commit 8242863 into mlrun:development Jan 7, 2026
13 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.

4 participants