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

Report errors when starting background balance #167

Open
kdave opened this issue Apr 17, 2019 · 0 comments
Open

Report errors when starting background balance #167

kdave opened this issue Apr 17, 2019 · 0 comments

Comments

@kdave
Copy link
Owner

@kdave kdave commented Apr 17, 2019

When there's already running balance, starting another one on the background does not report the errors (EINPROGRESS).

To fix this, the parent process should wait a bit if the forked background process still runs and report errors otherwise. There are no blocking calls when the 2nd ioctl is called, so a few seconds should be enough.

marcosps added a commit to marcosps/btrfs-progs that referenced this issue Oct 30, 2019
Introduce a sleep of 3 seconds after triggering balance to check if
isn't there another balance already being executed.

Fix: kdave#167

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
marcosps added a commit to marcosps/btrfs-progs that referenced this issue Oct 30, 2019
Introduce a sleep of 3 seconds after triggering balance to check if
isn't there another balance already being executed.

Fix: kdave#167

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.