Skip to content
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 ampq docker compose setup #763

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

atanmarko
Copy link
Contributor

Resolves #749

@atanmarko atanmarko self-assigned this Oct 30, 2024
@github-actions github-actions bot added the crate: zero_bin Anything related to the zero-bin subcrates. label Oct 30, 2024
@atanmarko atanmarko marked this pull request as ready for review October 31, 2024 11:59
Copy link
Contributor

@0xaatif 0xaatif left a 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?

@atanmarko
Copy link
Contributor Author

atanmarko commented Oct 31, 2024

@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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crate: zero_bin Anything related to the zero-bin subcrates.
Projects
Status: Ready to Review
Development

Successfully merging this pull request may close these issues.

Crate docker compose setup to easily run leader/worker test using message queues
2 participants