Bug 1441776

Summary: [RFE][overcloud-inspector] Split the service into API and worker processes
Product: Red Hat OpenStack Reporter: mkovacik
Component: openstack-ironic-inspectorAssignee: mkovacik
Status: CLOSED DEFERRED QA Contact: Dan Yasny <dyasny>
Severity: unspecified Docs Contact:
Priority: high    
Version: 7.0 (Kilo)CC: dyasny, mburns, nlevinki, racedo, racedoro, sclewis, slinaber, tvignaud
Target Milestone: ---Keywords: FutureFeature, Triaged
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-12 13:49:08 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1288035, 1442136, 1480137    

Description mkovacik 2017-04-12 17:06:04 UTC
Inspector is currently a monolithic, single-process service that in order to become HA one day as well as to support upstream effort of WSGI services should be split into API and worker processes.

See also:
[1] https://governance.openstack.org/tc/goals/pike/deploy-api-in-wsgi.html
[2] https://etherpad.openstack.org/p/ptg-pike-wsgi

Comment 1 Dmitry Tantsur 2017-04-13 07:02:42 UTC
Thanks! Adding this to 12, as this is a de-scoped version of previously approved https://bugzilla.redhat.com/show_bug.cgi?id=1288035

Comment 8 mkovacik 2017-06-30 09:17:53 UTC
This seems less likely to land before Pike M3; just the base patch landed so far, the second patch in the chain is receiving reviews atm, the third however still needs substantial work.

Comment 10 mkovacik 2017-10-02 16:57:46 UTC
So there's still discussions pending upstream about the fate of inspector being merged with ironic. Setting target release to 14 to be revised after a virtual planning meetup due in ~2weeks from now.

Comment 11 mkovacik 2017-10-23 14:03:03 UTC
just attaching an alternative implementation of the same, for the record

Comment 13 mkovacik 2017-12-12 13:49:08 UTC
This was deferred upstream; to be reopened if necessary