-
Notifications
You must be signed in to change notification settings - Fork 283
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
Add option to execute Tests for release #178
Add option to execute Tests for release #178
Conversation
If PR is not complete, consider turning it into "draft". It's a handy feature in GH, exactly for that. |
Note that this will need to be ported to v1.0 |
@undera Please help review Will port to be compatible with release v1.0.0 once this is merged |
return | ||
} | ||
|
||
c.String(http.StatusAccepted, out) |
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.
The tests can actually have output and logs. Shouldn't we block until those are returned, and show the result in UI?
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 mean StatusAccepted
has meaning "the request is accepted for processing, but it is not complete". If it is complete, the status code should be 200 OK
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.
Handled in new commit
pkg/dashboard/subproc/data.go
Outdated
@@ -278,6 +278,17 @@ func (d *DataLayer) ChartInstall(namespace string, name string, repoChart string | |||
return out, nil | |||
} | |||
|
|||
func (d *DataLayer) RunTests(namespace string, name string) (string, error) { | |||
cmd := []string{"test", name, "--namespace", namespace} |
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.
add --logs
here?
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.
Handled in new commit
Fixes Issue
#115
Changes proposed
helm test
Check List (Check all the applicable boxes)
Screenshots