-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update default task run name to exclude hash of task key #8292
Conversation
✅ Deploy Preview for prefect-orion ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Apparently, this is currently untested! I'm not sure I feel like we need to make assertions about the default task run names but it could be kind of nice to add coverage now? |
@ahuang11 this won't break the Dask task key right? |
It's separate but I wonder how this shows up in the graph. |
@ahuang11 it should display the same since Dask just eats up to the first hyphen. |
Supersedes #8287
The task run name isn't very user friendly by default and since we're showing task run names in lots of places we'd like the default to be focused on human readability. The task key will still be available in details. This will allow us to improve readability of task runs in the UI without short-term patches that would complicate design once we allow users to set their own task run names.
Checklist
<link to issue>
"fix
,feature
,enhancement