Skip to content

Conversation

@yaelgen
Copy link
Member

@yaelgen yaelgen commented Jan 11, 2026

📝 Description

Fixes a deprecation warning that appeared when using project.run() with remote KFP workflows, even when no experiment parameter was provided. The warning was triggered because internal code paths were auto-generating experiment names.

This PR:
Prevents warnings when using project.run()
Moves experiment name enrichment and normalization to the server side only
Maintains backward compatibility with the old behavior (name or workflow_spec.name + project prefix)
Ensures the deprecation warning only appears when users explicitly pass the experiment parameter


🛠️ Changes Made

Changed _KFPRunner.run() to pass experiment=None instead of auto-generating experiment=name or workflow_spec.name
Added logic to pass workflow name (name or workflow_spec.name) as run parameter so server can use it as fallback for experiment name


✅ 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


🔗 References


🚨 Breaking Changes?

  • Yes (explain below)
  • No

yaelgen added 29 commits August 16, 2023 14:24
# Conflicts:
#	server/api/utils/builder.py
#	tests/api/utils/test_builder.py
@yaelgen yaelgen changed the title Fix experiment dep [Pipelines] Prevent deprecation warning when using project.run() for KFP workflows Jan 11, 2026
@yaelgen yaelgen marked this pull request as ready for review January 12, 2026 09:21
@yaelgen yaelgen requested a review from a team as a code owner January 12, 2026 09:21
@liranbg liranbg merged commit f2f564c into mlrun:development Jan 13, 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