-
Notifications
You must be signed in to change notification settings - Fork 166
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
Core implementation for test runs in PLZ mode #239
Conversation
4dc740c
to
9983f58
Compare
cbc9a61
to
757bd97
Compare
77b8250
to
837e325
Compare
837e325
to
2ec5788
Compare
2ec5788
to
d0530df
Compare
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.
As far as I can tell, things seem to be fine. Only a couple minor comments based upon quick review.
Not really sure how to fully test this functionality.
controllers/k6_controller.go
Outdated
|
||
isAborted := status.Aborted() | ||
|
||
// if isAborted { |
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.
Nit: remove commented code.
6ab0a1a
to
9ea5035
Compare
Branch updated and rebased with volumes as init container configuration as described in this comment. |
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.
🎉 LGTM!
1924725
to
be149bd
Compare
Issue: #236
This builds up on top of PR #238