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
fix(controller): Get correct Step or DAG name. Fixes #2244 #2304
Conversation
if strings.Contains(nodeName, ".") { | ||
name := nodeName[strings.LastIndex(nodeName, ".")+1:] | ||
// Check retry scenario |
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.
Not only retry
scenario has (
, but also withItems
and withParam
. Simply removed the retry
condition check, and do it for all - This is based on the fact that neither template name nor step/DAG name is allowed to contain (
.
Codecov Report
@@ Coverage Diff @@
## master #2304 +/- ##
==========================================
- Coverage 11.44% 11.43% -0.01%
==========================================
Files 71 71
Lines 27860 27868 +8
==========================================
Hits 3188 3188
- Misses 24268 24275 +7
- Partials 404 405 +1
Continue to review full report at Codecov.
|
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.
Conditional approval: can you add new tests for this behavior? Perhaps from the reference issue?
Thanks! will add a functional test for that example. |
…rgoproj#2304) * fix(controller): Get correct Step or DAG name * Add functional test * Increase wait time
Fixes #2244
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.