Add missing _delete_trace_tag_v3 api#18813
Conversation
ddf7f04 to
a642017
Compare
|
Hi team, may I get some eyes on this one? it is resolving an api issue from our community. Also I am a new developer for this project, please let me know if my workflow / code needs more work here, thansk! @B-Step62 @serena-ruan @harupy |
B-Step62
left a comment
There was a problem hiding this comment.
The change looks good overall, can you fix the conflict and address nit comments? Thanks!
|
Documentation preview for 9b49e05 is available at: More info
|
a472de8 to
aabfcc8
Compare
Signed-off-by: Tian Lan <sky.blue266000@gmail.com>
299b0cd to
35cd6f0
Compare
Signed-off-by: Tian Lan <sky.blue266000@gmail.com>
|
Hi @B-Step62 , feel free to take another look of this one, already resolved your last comment and resolved conlicts. ( I believe there are some workflows can only be triggered after your approval. |
|
@Tian-Sky-Lan Can you fix the linter issues? https://github.com/mlflow/mlflow/blob/master/CONTRIBUTING.md#manual-python-development-environment-configuration |
|
❌ Validation Failed: This workflow can only be triggered by a repository owner, member, or collaborator. @Tian-Sky-Lan (NONE) does not have sufficient permissions. |
|
❌ Validation Failed: This workflow can only be triggered on PRs filed by a repository owner, member, or collaborator. @Tian-Sky-Lan (NONE) does not have sufficient permissions. |
Signed-off-by: Tian Lan <sky.blue266000@gmail.com>
@B-Step62 Hi I pushed a new commit, and it passed (on my fork repo) all previous failed checks like protos, lint and patch notes, please take a look, thanks!
|
|
Hi @B-Step62 , I found that all the checks passed (yeah), but I didn't see a |
|
@Tian-Sky-Lan No problem, just merged the PR. Thank you so much for the contribution! |

🛠 DevTools 🛠
Install mlflow from this PR
For Databricks, use the following command:
Related Issues/PRs
Resolve #18721
What changes are proposed in this pull request?
_delete_trace_tag_v3handler for Flask router, to be consistent with_set_trace_tag_v3.DELETErequest fordelete trace tagfrom using url to body.delete_trace_tagandset_trace_tagfor both v2 and v3 versionHow is this PR tested?
Screen.Recording.2025-11-09.at.10.29.32.PM.mp4
Does this PR require documentation update?
Release Notes
Is this a user-facing change?
Fix the bug of unable to delete the trace tags. User may experience HTTP 400 bug when trying to delete the trace tags previously.
What component(s), interfaces, languages, and integrations does this PR affect?
Components
area/tracking: Tracking Service, tracking client APIs, autologgingarea/models: MLmodel format, model serialization/deserialization, flavorsarea/model-registry: Model Registry service, APIs, and the fluent client calls for Model Registryarea/scoring: MLflow Model server, model deployment tools, Spark UDFsarea/evaluation: MLflow model evaluation features, evaluation metrics, and evaluation workflowsarea/gateway: MLflow AI Gateway client APIs, server, and third-party integrationsarea/prompts: MLflow prompt engineering features, prompt templates, and prompt managementarea/tracing: MLflow Tracing features, tracing APIs, and LLM tracing functionalityarea/projects: MLproject format, project running backendsarea/uiux: Front-end, user experience, plotting, JavaScript, JavaScript dev serverarea/build: Build and test infrastructure for MLflowarea/docs: MLflow documentation pagesHow should the PR be classified in the release notes? Choose one:
rn/none- No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" sectionrn/breaking-change- The PR will be mentioned in the "Breaking Changes" sectionrn/feature- A new user-facing feature worth mentioning in the release notesrn/bug-fix- A user-facing bug fix worth mentioning in the release notesrn/documentation- A user-facing documentation change worth mentioning in the release notesShould this PR be included in the next patch release?
Yesshould be selected for bug fixes, documentation updates, and other small changes.Noshould be selected for new features and larger changes. If you're unsure about the release classification of this PR, leave this unchecked to let the maintainers decide.Yes
What is a minor/patch release?
Bug fixes, doc updates and new features usually go into minor releases.
Bug fixes and doc updates usually go into patch releases.