-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat: Support Arguments in Exit Handler #5455
Conversation
Signed-off-by: Saravanan Balasubramanian <sarabala1979@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #5455 +/- ##
==========================================
+ Coverage 47.03% 47.36% +0.32%
==========================================
Files 246 247 +1
Lines 15485 15614 +129
==========================================
+ Hits 7283 7395 +112
Misses 7290 7290
- Partials 912 929 +17
Continue to review full report at Codecov.
|
This is really cool. We need something like this to pass parameters to exit handlers from certain steps. |
Signed-off-by: Saravanan Balasubramanian <sarabala1979@gmail.com>
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.
see comments
@@ -750,6 +749,9 @@ func (ctx *templateValidationCtx) validateSteps(scope map[string]interface{}, tm | |||
if err != nil { | |||
return errors.Errorf(errors.CodeBadRequest, "templates.%s.steps[%d].%s %s", tmpl.Name, i, step.Name, err.Error()) | |||
} | |||
if step.HasExitHook() { |
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.
do we need to guard this at all?
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.
yes, Including the current step/task variables should be only for exithook
.
🎉 |
Signed-off-by: Saravanan Balasubramanian sarabala1979@gmail.com
closes #5452
closes #4631
Checklist: