Bug 2003178 - [FLAKE][knative] The UI doesn't show updated traffic distribution after accepting the form
Summary: [FLAKE][knative] The UI doesn't show updated traffic distribution after accep...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Dev Console
Version: 4.6.z
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
: 4.10.0
Assignee: Mohammed Saud
QA Contact: spathak@redhat.com
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-09-10 14:16 UTC by Chris Suszynski
Modified: 2022-03-10 16:09 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-03-10 16:09:11 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift console pull 10147 0 None open Bug 2003178: Memoize knative filtered revisions for sidebar 2021-09-28 12:13:00 UTC
Github openshift console pull 10187 0 None open Bug 2003178: Use topology sidebar details as a component 2021-10-07 06:26:56 UTC
Red Hat Issue Tracker SRVCOM-1517 0 None None None 2021-09-10 14:16:18 UTC
Red Hat Product Errata RHSA-2022:0056 0 None None None 2022-03-10 16:09:58 UTC

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


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