kestra-io / kestra

:zap: Workflow Automation Platform. Orchestrate & Schedule code in any language, run anywhere, 500+ plugins. Alternative to Zapier, Rundeck, Camunda, Airflow...
https://kestra.io
Apache License 2.0
13.25k stars 1.15k forks source link

Not possible to Set Labels — Illegal argument: System labels can only be set by Kestra itself, offending label: system.app=xxx #5992

Closed smunteankestra closed 18 hours ago

smunteankestra commented 6 days ago

Adding execution labels returns, after refresh they disappear image

Demo: https://share.descript.com/view/FVaLVn8UMvY

Environment

MilosPaunovic commented 6 days ago

Related to https://github.com/kestra-io/kestra-ee/issues/2001.

loicmathieu commented 5 days ago

This is a UI issue, existing labels are now sent to the backend which should not happen

MilosPaunovic commented 5 days ago

After a brief discussion we've came to a conclusion current behavior is definitely not ideal. We need to find a way to either ignore system labels using the existing endpoint or to create separate endpoints for adding and for removing labels.

anna-geller commented 19 hours ago

@MilosPaunovic are you on it or should we ask Bart or Yann? this one seems critical to get before 0.20

MilosPaunovic commented 19 hours ago

I am not on it, had a discussion on Slack with @loicmathieu, he said he'll look at it from BE side.