Is there a particular reason that named executions...
# announcements
r
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)
s
cc: @Ketan (kumare3)
k
Cc @Haytham Abuelfutuh / @katrina
r
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
k
I think it was done for k8s
nothing on the db
and also for legibility
r
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?
k
we can but have to be careful
lets talk cc @katrina / @Haytham Abuelfutuh
👍 1
r
Ya happy to discuss the tradeoffs later today @Ketan (kumare3) -- 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
k
ya i was thinking we need a label anyways
👍 1
r
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