-
Notifications
You must be signed in to change notification settings - Fork 113
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
Configurable MaxLockCycles #3429
Conversation
Signed-off-by: jkoberg <jkoberg@owncloud.com>
381d011
to
11069bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@kobergj codacy is not happy with the Changelog markdown |
It is. I fixed the issue already, it's just still shown in red because the pipeline is red. |
Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: jkoberg <jkoberg@owncloud.com>
Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
is this still needed, now that we debounce for a longer time? in the experimental branch @fschade still see the 500 errors in clients, which he can fix by crinking up this ... or was that only the case on edge? |
When a file is locked the flock library will retry a given amount of times (with a increasing sleep time inbetween each round) | ||
Until now the max amount of such rounds was hardcoded to `10`. Now it is configurable, falling back to a default of `25` | ||
|
||
https://github.com/owncloud/ocis/pull/4959 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kobergj this changelog entry is broken. We need the reva pull request in the first line of the link list because it provides the primary issue id which needs to resolve inside the repo.
When a file is locked the flock library will retry a given amount of times (with a increasing sleep time inbetween each round)
Until now the max amount of such rounds was hardcoded to
10
. Now it is configurable, falling back to a default of25