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

fix(config): aqua-pool-size argument #1606

Merged
merged 1 commit into from
May 6, 2023
Merged

fix(config): aqua-pool-size argument #1606

merged 1 commit into from
May 6, 2023

Conversation

gurinderu
Copy link
Contributor

@gurinderu gurinderu commented May 5, 2023

Description

Fix aqua-pool-size argument

@linear
Copy link

linear bot commented May 5, 2023

NET-465 --aqua-pool-size=2 has no effect

2023-05-01T19:55:25.493447Z  INFO tokio-runtime-worker ThreadId(05) aquamarine::vm_pool: All 16 AquaVMs created.

You can run docker compose up -d in rust-peer-distro/deploy/docker-compose and see it yourself

@gurinderu gurinderu added the e2e Run e2e workflow label May 5, 2023
@kmd-fl
Copy link
Contributor

kmd-fl commented May 5, 2023

Do I understand correctly that the problem was in the variable's name and that the simple renaming fixes the issue?

Would be nice to add it to the PR description.

@gurinderu
Copy link
Contributor Author

Do I understand correctly that the problem was in the variable's name and that the simple renaming fixes the issue?

exactly, var names in deserialized structure should have the same name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
e2e Run e2e workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants