Project

Profile

Help

Issue #6724

closed

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

Added by hyu almost 4 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
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
Quarter:

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

Actions #1

Updated by fao89 almost 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 73
Actions #2

Updated by ipanova@redhat.com almost 4 years ago

  • Status changed from NEW to POST

Added by hyu almost 4 years ago

Revision 2d032c6f | 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

Actions #4

Updated by rchan almost 4 years ago

  • Sprint changed from Sprint 73 to Sprint 74
Actions #5

Updated by hyu almost 4 years ago

  • Status changed from POST to MODIFIED
Actions #6

Updated by ttereshc almost 4 years ago

  • Platform Release set to 2.21.3

Added by hyu almost 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)

Actions #7

Updated by hyu almost 4 years ago

Actions #8

Updated by ttereshc almost 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF