Project

Profile

Help

Issue #6724

Some consumer profiles are missed when calculating batch regenerate applicability for repo

Added by hyu 3 months ago. Updated 13 days ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
Estimated time:
Severity:
3. High
Version:
Platform Release:
2.21.3
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello, Pulp 2
Sprint:
Sprint 74

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 = []

Associated revisions

Revision 2d032c6f View on GitHub
Added by hyu 3 months ago

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

Revision 79f0c2ac View on GitHub
Added by hyu about 1 month ago

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)

History

#1 Updated by fao89 3 months ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 73

#2 Updated by ipanova@redhat.com 3 months ago

  • Status changed from NEW to POST

#4 Updated by rchan 3 months ago

  • Sprint changed from Sprint 73 to Sprint 74

#5 Updated by hyu 2 months ago

  • Status changed from POST to MODIFIED

#6 Updated by ttereshc about 2 months ago

  • Platform Release set to 2.21.3

#7 Updated by hyu about 1 month ago

#8 Updated by ttereshc 30 days ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF