Skip to content

Conversation

@weilerN
Copy link
Contributor

@weilerN weilerN commented Jan 11, 2026

📝 Description

Fix probes test by removing redundant function deployment that supposed to fail, functionality can be tested with only 1 deployment


🛠️ Changes Made

  • remove the first deployment from the test_application_probes test

✅ 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

  • Tested on developer's vmdev

🔗 References


🚨 Breaking Changes?

  • Yes (explain below)
  • No

🔍️ Additional Notes

@weilerN weilerN requested review from a team and TomerShor and removed request for TomerShor January 11, 2026 14:42
@weilerN weilerN marked this pull request as ready for review January 11, 2026 14:43
@TomerShor TomerShor merged commit 67ec873 into mlrun:development Jan 11, 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.

2 participants