Bug 2003178

Summary: [FLAKE][knative] The UI doesn't show updated traffic distribution after accepting the form
Product: OpenShift Container Platform Reporter: Chris Suszynski <ksuszyns>
Component: Dev ConsoleAssignee: Mohammed Saud <msaud>
Status: CLOSED ERRATA QA Contact: spathak <spathak>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.6.zCC: aos-bugs, cjerolim, jakumar, msaud, nmukherj, wsun
Target Milestone: ---   
Target Release: 4.10.0   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-03-10 16:09:11 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:

Description Chris Suszynski 2021-09-10 14:16:19 UTC
Description of problem:


The UI doesn't show updated traffic distribution, despite accepting the form.

For now, we have one evidence of this - failed OCP 4.6.43 run: https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/periodic-ci-openshift-knative-serverless-operator-main-4.6-e2e-aws-ocp-46-continuous/1436117460133613568

screenshot: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/logs/periodic-ci-openshift-knative-serverless-operator-main-4.6-e2e-aws-ocp-46-continuous/1436117460133613568/artifacts/e2e-aws-ocp-46-continuous/serverless-e2e/artifacts/build-jmUzaF/j/ui/screenshots/serving.spec.js/OCP%20UI%20for%20Serverless%20--%20can%20route%20traffic%20to%20multiple%20revisions%20(failed).png

This is for sure flaky.

Version-Release number of selected component (if applicable):

4.6.43


How reproducible:

Flaky


Steps to Reproduce:
1. Try applying the knative service traffic distribution form?!?

Actual results:

The UI should show traffic distribution between two revisions, rev.1 and rev.2. The routing should have 49% and 51% chances respectively.


Expected results:

The UI shows 100% traffic to rev.1, and you could see the rev.2 just started to terminate.

Comment 1 Chris Suszynski 2021-09-10 14:22:01 UTC
JS error:

   1) OCP UI for Serverless
       can route traffic to multiple revisions:
     AssertionError: Timed out retrying after 60000ms: Expected to find content: '51%' but never did.

Comment 5 Mohammed Saud 2021-09-28 15:46:56 UTC
The issue occurred due to filtered revisions being calculated using a function inside the component which caused it to use stale data, it was fixed by memoizing the revisions data.

The fix is in 4.10 (master). Sidebar should now update traffic distributions as soon as the resource is changed, although there is still a small delay since the UI waits for the resource to be updated rather than immediately showing the changed values on form submit.

Please let us know if there are further issues.

Comment 10 Christoph Jerolimov 2022-01-04 10:12:39 UTC
Hey Chris @ksuszyns, happy new year. :)

Can you verify that this flaky issue doesn't exist on a nightly build any more or is it okay if we verify this issue on our own? Thanks

Comment 11 Chris Suszynski 2022-01-17 11:21:26 UTC
Please verify it on your end. Hopefully, the issue will be resolved.

Comment 14 spathak@redhat.com 2022-02-25 16:04:33 UTC
Verified on build version: 4.10.0-0.nightly-2022-02-23-193238
Browser: Google Chrome 98`

Comment 16 errata-xmlrpc 2022-03-10 16:09:11 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 (Moderate: OpenShift Container Platform 4.10.3 security update), 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/RHSA-2022:0056