-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 async i/o error and move startup script #19736
Conversation
Thanks! The PR looks good to me. Let's see what the CI has to say! |
Thanks! Is this PR supposed to resolve #19445 too completely? |
@gastaldi yes, but we need to change the title of the issue. The PR fixes potential startup issues in some environments. |
@gastaldi just updated the issue title. |
Hi @joyrex2001, this change (specifically the addition of Interestingly, if I add I'm on Linux and have 32 GB of RAM. The container sees all those 32 GB of RAM (because we don't apply any memory cgroup limits), which is probably where the memory sizing logic gets confused. Would that change (adding |
According to the cli documentation not specifying a memory limit would actually imply that it will use all available memory, which makes me also suspect the automatic sizing logic fails. I tried various combinations of the cli arguments they suggest in the docker quick start and also played with the The redpanda docker quick start sets it to 1G, and also sets the |
So in addition to |
Indeed :-) As this PR is already closed/merged, do you want me to create a new PR with the changes, or will you? |
If you can submit a PR, that would be great, otherwise I'll do it tomorrow or the day after that. Thanks! |
This PR starts the redpanda server with
--smp 1
which resolves aCould not setup Async I/O: Resource temporarily unavailable.
error when running tests using dind in k8s. This PR also moves the startup script to the/var/lib/redpanda/
folder which is owned by the redpanda user that runs the container.See also: #19445