Bug 1287219

Summary: [Backport] haproxy endpoing checks create a performance impacting loop for application restarts.
Product: OpenShift Container Platform Reporter: Eric Rich <erich>
Component: ImageStreamsAssignee: Timothy Williams <tiwillia>
Status: CLOSED NOTABUG QA Contact: Anping Li <anli>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 2.2.0CC: abhgupta, aos-bugs, bperkins, dma, erich, erjones, gpei, jkaur, jokerman, misalunk, mmccomas, nicholas_schuetz, pep, tiwillia, xtian
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1285084 Environment:
Last Closed: 2016-06-20 16:19:55 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: 1273542    

Comment 6 Anping Li 2016-02-14 11:19:25 UTC
After update, the cartridge version are same as before. I think we should assign a new version in manifest.yml.

Comment 7 Anping Li 2016-02-15 05:13:43 UTC
The performance impacting disappears once this patch was enabled. For existing applications, the customer need to call oo-admin-upgrade to replace the Haproxy inner applications (if there are other commands, please let me known).

For upgrade, there isn't any issue related to this patch. there are two little issue about nodejs and php packages. 
 1) nodejs010-nodejs-debug-0.7.2-1.el6op.noarch may be replace with non-ose packages '2.2.0-2.el6' from rhscl
 2) there are Dependency error if we upgrade php-common-5.3.3-40 to php-common-5.3.3-46

Comment 8 Timothy Williams 2016-02-16 19:35:47 UTC
Bumping the cartridge version is going to cause a minor issue. The compatible-versions list in the cartridge manifest must be maintained with prior versions that don't require a restart when upgraded - otherwise the users will experience unnecessary downtime. Since I can't retroactively add a '0.0.23.2' entry to the compatible versions list of the newer openshift enterprise haproxy cartridge packages, I've bumped the version to 0.0.24.

0.0.24 is in the list of compatible versions for the enterprise rpms and shouldn't cause the unnecessary downtime. Usually a 3-digit cartridge version would denote an upstream cartridge, but I think this should be acceptable in this case. Could you please re-test with the new rpm in the following build:

  https://brewweb.devel.redhat.com/taskinfo?taskID=10503866

Even though its unrelated, could you give us more details on the php-common dependency issue. We expect the nodejs package to be updated and will handle that appropriately in the next release (2.2.9). We have not aware of the php-common dependency issue though. Could you give us the specific dependency issue output from yum that you are seeing? Also, can you describe your yum repository configuration?

Comment 9 Anping Li 2016-02-17 03:03:26 UTC
Verified and pass with package in brew. Below is the step to apply this patch.

1. yum update openshift-origin-cartridge-haproxy-1.30.1.2-1.el6op.noarch.rpm
2. service ruby193-mcollective restart
on broker:
3. oo-admin-ctl-cartridge -c import-profile --activate
4. oo-admin-upgrade upgrade-node --version=0.0.24