Issue #6724
closedSome consumer profiles are missed when calculating batch regenerate applicability for repo
Description
Pulp calculates the applicability for repo in the batch size of 10 but the profiles of the 10th consumer in the batch will always be skipped.
For example, if you have 20 consumers, the 10th and 20th consumers will always get 0 applicability because the calculations are skipped.
Below is the offending code:
server/managers/consumer/applicability.py if len(profiles_to_process) < batch_size: <========= The last profiles in the batch is no including profiles_to_process.append((repo_id, all_profiles_hash, profiles)) else: batch_regenerate_applicability_task.apply_async( (profiles_to_process,), **{'group_id': task_group_id}) profiles_to_process = []
Updated by fao89 over 4 years ago
- Triaged changed from No to Yes
- Sprint set to Sprint 73
Updated by ipanova@redhat.com over 4 years ago
- Status changed from NEW to POST
Added by hyu over 4 years ago
Updated by hyu over 4 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulp|2d032c6f63fb83e78f3bb309037691c458f24e82.
Added by hyu over 4 years ago
Revision 79f0c2ac | View on GitHub
Fix batch regenerate applicability for repo
Pulp calculates the applicability for repo in the batch size of 10 but the profiles of the 10th consumer in the batch will always be missed. This commit fixed it.
closes: #6724 https://pulp.plan.io/issues/6724 (cherry picked from commit 2d032c6f63fb83e78f3bb309037691c458f24e82)
Updated by hyu over 4 years ago
Applied in changeset pulp|79f0c2ac075e867d9e0a175558ed4b62d7bc9625.
Updated by ttereshc over 4 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Fix batch regenerate applicability for repo
Pulp calculates the applicability for repo in the batch size of 10 but the profiles of the 10th consumer in the batch will always be missed. This commit fixed it.
closes: #6724 https://pulp.plan.io/issues/6724