-
Notifications
You must be signed in to change notification settings - Fork 14
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
[patch] handle return codes #928
Conversation
@racree Can you add a description of this change into the PR text? |
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.
I've taken a look through and I can't see any obvious problems. However, this is the sort of thing we really need to put through an end-to-end test before merging into the gitops
branch (and risk breaking fvtsaas). Rick's going to raise a story for next sprint to make sure that gets done 👍
NOTE: please make sure this branch pulls in the latest from the gitops
branch before doing any further testing.
Check return codes of commands executed in the Tekton tasks and supporting scripts/functions and exit on failure.