Bug 899273 (JBEWS-237) - Apache BZ 48735 "bybusyness does not balance after failed worker has recovered"
Summary: Apache BZ 48735 "bybusyness does not balance after failed worker has recovered"
Keywords:
Status: CLOSED NEXTRELEASE
Alias: JBEWS-237
Product: JBoss Enterprise Web Server 1
Classification: JBoss
Component: unspecified
Version: EWS 1.0.1
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: EWS 1.0.2
Assignee: Permaine Cheung
QA Contact:
URL: http://jira.jboss.org/jira/browse/JBE...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-06-29 03:30 UTC by Samuel Mendenhall
Modified: 2012-11-13 16:26 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-05-22 21:12:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
httpd-2.2.17-busyness.patch (4.38 KB, text/plain)
2011-03-04 15:21 UTC, Ivan Szanto
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JBEWS-237 0 Minor Closed Apache BZ 48735 "bybusyness does not balance after failed worker has recovered" 2013-08-16 09:12:40 UTC

Description Samuel Mendenhall 2010-06-29 03:30:49 UTC
Affects: Release Notes
Date of First Response: 2010-07-08 12:25:05
Help Desk Ticket Reference: https://c.na7.visual.force.com/apex/Case_View?id=500A0000003hTBx&sfdc.override=1
project_key: JBEWS

https://issues.apache.org/bugzilla/show_bug.cgi?id=48735

"after a failed worker has recovered, no request is fowarded to it although it is marked as OK in balancer-manager"

Comment 3 Rebecca Newton 2010-11-15 05:17:06 UTC
Release Notes Docs Status: Added: Not Yet Documented
Writer: Added: rebecca_newton


Comment 4 Permaine Cheung 2010-11-16 14:14:45 UTC
We'll check if this is in the latest httpd that we update to, if not, we can apply the patch attached to the Apache BZ.

Comment 5 Permaine Cheung 2011-01-28 17:15:23 UTC
The reporter of the ASF BZ confirmed that this is not reproducible in httpd 2.2.17, and that's what we're upgrading to in EWS 1.0.2, so I'm marking this as resolved.

Comment 6 Rebecca Newton 2011-02-23 06:19:51 UTC
Release Notes Docs Status: Removed: Not Yet Documented Added: Documented as Resolved Issue
Release Notes Text: Added: No request is forwarded to a worker node after it has failed, even though it is marked as available in the balance manager. This issue has been resolved with the inclusion of httpd 2.2.17.


Comment 7 Ivan Szanto 2011-03-04 14:17:39 UTC
Unfortunately it IS reproducible in httpd 2.2.17.
ASF somehow forgot to include their patch in 2.2.17.
Is it possible to include the patch in EWS 1.0.2?

Comment 8 Permaine Cheung 2011-03-04 14:28:43 UTC
Can you please point me to a patch for this issue for 2.2.17?

CR1 is to be handed off today, there's not enough time to rebuild and incorporate that into all the brew and hudson builds today, would it be ok to have this included in CR2?

Comment 9 Ivan Szanto 2011-03-04 15:21:01 UTC
Permaine, can you please reopen this bug? I could not seem to do it.

Although it displays a couple of warnings, the patch posted in the ASF BZ does just a fine job for 2.2.17 as well.

Here are the warnings:

[szivan@manipura httpd-2.2.17]$ patch -p1 < ../httpd-2.2.16.patch 
patching file modules/proxy/mod_proxy.c
Hunk #1 succeeded at 1038 (offset 22 lines).
patching file modules/proxy/mod_proxy.h
patching file modules/proxy/mod_proxy_balancer.c
Hunk #4 succeeded at 623 with fuzz 2 (offset 9 lines).
Hunk #5 succeeded at 848 (offset 9 lines).
Hunk #6 succeeded at 880 (offset 9 lines).
Hunk #7 succeeded at 1057 (offset 9 lines).
Hunk #8 succeeded at 1140 (offset 9 lines).
Hunk #9 succeeded at 1222 (offset 9 lines).

After building the server with this patch it fixed the problem.

If you prefer a patch built for 2.2.17, I will attach it in a minute.

Comment 10 Ivan Szanto 2011-03-04 15:21:33 UTC
this patch was made for 2.2.17 to solve the issue

Comment 11 Ivan Szanto 2011-03-04 15:21:33 UTC
Attachment: Added: httpd-2.2.17-busyness.patch


Comment 12 Permaine Cheung 2011-03-04 15:28:30 UTC
Reopening for incorporation in CR2.

Comment 13 Permaine Cheung 2011-03-04 16:07:05 UTC
Thanks for the patch, Ivan!

Comment 14 Permaine Cheung 2011-03-29 15:46:00 UTC
httpd22-2.2.17-9.ep5.el4
httpd-2.2.17-9.2.ep5.el6
httpd-2.2.17-9.1.ep5.el5
Built with the fix, will be in CR2.

Comment 15 Permaine Cheung 2011-03-29 15:48:28 UTC
Mladen, can you please update the Hudson build to use
httpd-2_2_17-9_1_ep5_el5 and apply the patch for this issue (httpd-2.2.17-bz48735.patch)? Thanks!

Comment 17 Permaine Cheung 2011-04-07 12:30:45 UTC
Marking it as resolved as Hudson machines are working properly now, will be in CR2.

Comment 18 Jiri Skrabal 2012-11-13 16:26:49 UTC
Release Notes Docs Status: Removed: Documented as Resolved Issue 
Writer: Removed: rebecca_newton 
Release Notes Text: Removed: No request is forwarded to a worker node after it has failed, even though it is marked as available in the balance manager. This issue has been resolved with the inclusion of httpd 2.2.17. 



Note You need to log in before you can comment on or make changes to this bug.