-
Notifications
You must be signed in to change notification settings - Fork 130
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: error handling when creating TW #5631
Conversation
✅ Deploy Preview for testkube-docs-preview canceled.
|
@@ -67,7 +67,9 @@ func NewCreateTestWorkflowCmd() *cobra.Command { | |||
} | |||
} | |||
|
|||
workflow, _ := client.GetTestWorkflow(obj.Name) | |||
workflow, err := client.GetTestWorkflow(obj.Name) | |||
ui.ExitOnError("getting test workflow "+obj.Name+" in namespace "+obj.Namespace, err) |
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.
That should not be there, as will disallow creating new TestWorkflows, unless you will replace err = nil
in case of "404 Not Found" error
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 sure if debug message makes sense in case of lack of workflow during creation, but it's not critical issue so approving 👍
|
* fix: error handling when creating TW * fix: remove object.name update * fix: move error to debug * fix: handle error on update
Pull request description
Checklist (choose whats happened)
Breaking changes
Changes
Fixes