Bug 1445510

Summary: [3.5] panic in github.com/openshift/origin/pkg/proxy/hybrid
Product: OpenShift Container Platform Reporter: Ben Bennett <bbennett>
Component: NetworkingAssignee: Ben Bennett <bbennett>
Status: CLOSED ERRATA QA Contact: Meng Bo <bmeng>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.5.0CC: aloughla, aos-bugs, bbennett, bmeng, dakini, dcbw, eparis, smunilla, zzhao
Target Milestone: ---   
Target Release: 3.5.z   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Cause: Panic due to concurrent access shows in the logs Consequence: Scary log messages (the thread is restarted, so there is no functional problem other than a small cost to rebuild the thread state) Fix: Add the appropriate locking. Result: Clean logs, no panics.
Story Points: ---
Clone Of: 1444125 Environment:
Last Closed: 2017-05-18 09:28:57 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:
Bug Depends On: 1444125    
Bug Blocks:    

Description Ben Bennett 2017-04-25 20:20:20 UTC
+++ This bug was initially created as a clone of Bug #1444125 +++


panic is in the attachment

Comment 2 zhaozhanqi 2017-04-28 03:10:38 UTC
@Ben Bennett

Could you give me some help to verify this bug since I have no idea how to reproduce this bug?

Comment 3 Ben Bennett 2017-04-28 12:46:57 UTC
@zhaozhanqi: Sadly, I can't tell you how to verify this fix.  It happens rarely, and was only seen in the logs on a busy, long-running system.  It would be a tricky race-condition to manage to trigger.  The best we can do is test that it doesn't break anything else, and that the code review caught all of the places the lock was needed.

Definitely run the idling and unidling test cases since that will be where the code is exercised.

Comment 4 Meng Bo 2017-05-03 07:43:16 UTC
Do regression testing on include parallel idling/unidling operations.

No regression issue found and no panic error found in the logs.

Move the bug to verified.

Comment 6 errata-xmlrpc 2017-05-18 09:28:57 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.