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 amqp docker compose setup #763

Merged
merged 8 commits into from
Nov 6, 2024

Conversation

atanmarko
Copy link
Member

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
@atanmarko atanmarko force-pushed the feat/prove-stdio-ampq branch from da510c0 to 78b1ee9 Compare October 31, 2024 12:00
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
Member 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).

@Nashtare Nashtare added this to the x Misc. milestone Nov 4, 2024
@atanmarko atanmarko changed the title feat: prove stdio using ampq docker compose setup feat: prove stdio using amqp docker compose setup Nov 6, 2024
@atanmarko atanmarko force-pushed the feat/prove-stdio-ampq branch from 78b1ee9 to 49ab68c Compare November 6, 2024 12:33
@github-actions github-actions bot added the ci label Nov 6, 2024
@atanmarko atanmarko merged commit 4646b2f into develop Nov 6, 2024
23 checks passed
@atanmarko atanmarko deleted the feat/prove-stdio-ampq branch November 6, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci crate: zero_bin Anything related to the zero-bin subcrates.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

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