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
K8S: RFC-1123 compliant job names #712
Conversation
""" Generate RFC 1123 compatible job name """ | ||
long_name = "-".join( | ||
[ | ||
flow_name, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technically flow_name
can start with an underscore, which will make the output of this function non-compliant.
Also, we would need to sanitize the strings for labels as well - we can (at a later stage) make this a utility method.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed. Also I realized that the previous version could generate identical names for HelloFlow
and Helloflow
, fixed that too
48abada
to
1a6c4a6
Compare
1a6c4a6
to
5e0c497
Compare
be2f658
to
78d66e9
Compare
We can sunset this PR in favour of #719? |
either works, #719 has this commit and one more for labels |
78d66e9
to
20a684d
Compare
Great! Merged #719. We can sunset this PR. |
This fixes a known TODO item on K8S API to make job names always-valid