-
Notifications
You must be signed in to change notification settings - Fork 519
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 snark worker support in integration tests #10279
Fix snark worker support in integration tests #10279
Conversation
@@ -32,10 +32,14 @@ spec: | |||
image: {{ $.Values.mina.image }} | |||
{{ if $.Values.mina.useCustomEntrypoint -}} | |||
command: [{{ $.Values.mina.customEntrypoint }}] | |||
{{- else }} |
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.
Would this work for non-integration-test deployments as well? cc @QuiteStochastic
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 THINK george's change should be fine. the default daemon entrypoint is more or less just mina $INPUT_ARGS $EXTRA_FLAGS 2>mina-stderr.log 1>mina.log &
. so if there is no command
, then args will be passed straight to that. cc @lk86
d42c364
to
acc5483
Compare
90216e2
to
3440a4d
Compare
3440a4d
to
4231466
Compare
Integration test framework supports snark workers by design. This functionality wasn't used and well-maintained. This PR brings it up-to-date, read to be used.
Checklist: