Bug 1559442 - Consolidated RefreshWorkers may cause job starvation
Summary: Consolidated RefreshWorkers may cause job starvation
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Providers
Version: 5.8.0
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: GA
: 5.10.0
Assignee: Tomas Coufal
QA Contact: Tasos Papaioannou
URL:
Whiteboard:
Depends On:
Blocks: 1560699
TreeView+ depends on / blocked
 
Reported: 2018-03-22 14:55 UTC by Tomas Coufal
Modified: 2019-02-11 14:06 UTC (History)
6 users (show)

Fixed In Version: 5.10.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1560699 (view as bug list)
Environment:
Last Closed: 2019-02-11 14:06:59 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

Description Tomas Coufal 2018-03-22 14:55:11 UTC 
Description of problem:
Job starvation can occur when :drb strategy is used for RefreshWorkers


Version-Release number of selected component (if applicable):


How reproducible:
Always


Steps to Reproduce:
1. Start EVM
2. Add Azure or Amazon provider (any provider with consolidated workers)
3. Start queueing the provider refresh (over long period of time, in some interval shorter than the time it takes to refresh Cloud manager)

Actual results:
Refresh of dependent managers (like Network or Storage) never happens

Expected results:
Dependent managers should be refreshed in order of job arrival to MiqQueue

Additional info:
Required to be fixed by the same time as the Azure targeted refresh lands in downstream [1]

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1487602
Comment 2 Tomas Coufal 2018-03-22 14:57:08 UTC 
Also affects other providers with targeted refresh like OpenStack and AWS.

Fixed by PR: https://github.com/ManageIQ/manageiq/pull/17187
Comment 3 Tomas Coufal 2018-03-26 09:15:01 UTC 
PR merged.
Comment 5 Tasos Papaioannou 2018-10-05 18:46:46 UTC 
Verified on 5.10.0.18.
Note You need to log in before you can comment on or make changes to this bug.