|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
From b781f06791afe27a780b51bc9a232ecac97a894b Mon Sep 17 00:00:00 2001
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
From: Mark Reynolds <mreynolds@redhat.com>
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
Date: Mon, 18 May 2015 14:51:55 -0400
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
Subject: [PATCH 325/327] Ticket 48158 - cleanAllRUV task limit not being
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
enforced correctly
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
Bug Description: The task limit check was only counting tasks that had
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
already begun the cleaning process. If the task was waiting
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
for a replica to be online before it starting the cleaning
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
process, then task count was not incremented, and limit
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
check was essentially bypassed.
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
Fix Description: Use the "pre-cleaned" rid list to determine the task count.
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
The pre-cleaned list is updated as soon a task request
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
comes in - not once the cleaning starts.
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
https://fedorahosted.org/389/ticket/48158
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
Reviwed by: nhosoi(Thanks!)
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
(cherry picked from commit d774b19fe6a1626cca3c92b125e56a22771bba5a)
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
(cherry picked from commit fd427d1db9820ce3bb81e2b72e20caa7e35bf6f9)
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
---
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
ldap/servers/plugins/replication/repl5_replica_config.c | 2 +-
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
diff --git a/ldap/servers/plugins/replication/repl5_replica_config.c b/ldap/servers/plugins/replication/repl5_replica_config.c
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
index a9aeed3..ae4c2ff 100644
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
--- a/ldap/servers/plugins/replication/repl5_replica_config.c
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
+++ b/ldap/servers/plugins/replication/repl5_replica_config.c
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
@@ -3152,7 +3152,7 @@ get_cleanruv_task_count()
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
slapi_rwlock_wrlock(rid_lock);
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
for(i = 0; i < CLEANRIDSIZ; i++){
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
- if(cleaned_rids[i] != 0){
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
+ if(pre_cleaned_rids[i] != 0){
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
count++;
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
}
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
}
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
--
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
1.9.3
|
|
![](https://seccdn.libravatar.org/avatar/f7c8374eebc27491f72ab99cc480e9d2456d6fed1ceedcdf8a1c5f40c91270c2?s=16&d=retro) |
dc8c34 |
|