-
Notifications
You must be signed in to change notification settings - Fork 39
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
feat: prove stdio using amqp docker compose setup #763
Conversation
da510c0
to
78b1ee9
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.
Shouldn't we test this in CI?
@0xaatif No, too big - too expensive. Requires at least 128GB RAM for 2 worker containers to work in parallel. This is only for manual development testing for scenarios where it is necessary to check if remote execution works (e.g. changes on paladin). |
78b1ee9
to
49ab68c
Compare
Resolves #749