Bug 1701041 - Proxy never returns response when watch fails to get established
Summary: Proxy never returns response when watch fails to get established
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Service Broker
Version: 4.1.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 4.2.0
Assignee: Shawn Hurley
QA Contact: Cuiping HUO
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-04-17 21:23 UTC by Fabian von Feilitzsch
Modified: 2019-10-16 06:28 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-10-16 06:28:06 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github operator-framework operator-sdk pull 1638 0 'None' closed Bug 1701041: Adding timeout on dependent watch establishment 2020-01-31 08:59:05 UTC
Red Hat Product Errata RHBA-2019:2922 0 None None None 2019-10-16 06:28:21 UTC

Description Fabian von Feilitzsch 2019-04-17 21:23:14 UTC
Description of problem:
When requesting a resource without proper RBAC (resulting in a 403), the proxy attempts to establish a watch, and will continuously retry without returning a response. This causes Ansible to appear to hang, although it is just waiting for a response from the proxy

How reproducible:
100%, and against bare k8s as well

Steps to Reproduce:
1. Create an ansible operator that attempts to interact with a resource it does not have proper RBAC permissions for
2. Run the operator/create a
3. Create a CR to trigger a run of the operator

There's a reproducer operator and environment here: https://github.com/fabianvf/reproducer-operator
The travis CI logs will have debug output from the operator.

Actual results:
The Ansible playbook hangs indefinitely, error status is not reported on the CR

Expected results:
The playbook fails with a 403 and the CR reports the failure.


Additional info:

Comment 1 Dan Geoffroy 2019-09-05 15:46:44 UTC
This is merged and should be built in the latest operator.  As this was a base image fix, automation did not move this to ON_QA itself.

Comment 2 Zhang Cheng 2019-09-19 10:03:01 UTC
Tried with quay.io/operator-framework/ansible-operator:master , it is work for me.
Changing status to verified.

Comment 3 errata-xmlrpc 2019-10-16 06:28:06 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.

https://access.redhat.com/errata/RHBA-2019:2922


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