-
Notifications
You must be signed in to change notification settings - Fork 810
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(check-pre-condition): CheckPrecondition doesn't evaluate expressi… #4682
Conversation
…on correctly after upstream stages get restarted
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.
Thanks! :)
@jasonmcintosh Can we backport this for 1.33, 1.32, and 1.31? |
@Mergifyio backport release-1.33.x release-1.32.x release-1.31.x |
❌ No backport have been created
GitHub error:
|
✅ Backports have been created
|
This PR includes the changes to fix the spinnaker/spinnaker#6933
Root cause:
If pipeline created from template, CheckPrecondition doesn't evaluate expression correctly after upstream stages get restarted
Fix:
Replace precondition List with precondition Map containing stage refId and preconditions. Modify the replace condition to replace pre-condition context with corresponding preCondition stage context.