Is there a particular reason that named executions...
# announcements
e
Is there a particular reason that named executions are capped at 20 characters?
Copy code
Error: rpc error: code = InvalidArgument desc = size of name exceeded length 20 : test-named-workflow-2
This appears to be hard-coded here: https://github.com/flyteorg/flyteadmin/blob/7b48c856e3a1e12c57f1abbf535bf6c29fa59d74/pkg/manager/impl/validation/execution_validator.go#L19. Would folks be open to bumping that to 64 (to match the max length for an argo workflow's name)
t
cc: @freezing-airport-6809
f
Cc @high-park-82026 / @acceptable-policeman-57188
e
Is there any limitation w/ the database schema for flyteadmin that requires the execution ID is <= 20 chars? Argo's 64-char limitation is a K8s-level restriction vs an argo-specific implementation detail
f
I think it was done for k8s
nothing on the db
and also for legibility
e
The max length for k8s workloads/labels is 64 chars -- 20 chars is a bit tight in terms of named executions/can see that being a common user footgun. If it isn't a db-lvl limitation would you either be open to setting to 64 or making it configurable in flyteadmin?
f
we can but have to be careful
lets talk cc @acceptable-policeman-57188 / @high-park-82026
👍 1
e
Ya happy to discuss the tradeoffs later today @freezing-airport-6809 -- if shoehorning it into execution ID doesn't make sense, could also introduce a separate varchar field to include a human-readable name
that's separate from execution id
f
ya i was thinking we need a label anyways
👍 1
e
Especially if that could then lead to search in the frontend based on labels, think that'd be a great multi-tenancy improvement
👍 1
158 Views