New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bug 1883737: Update Rotue section in Knative revision sidebar #6763
Bug 1883737: Update Rotue section in Knative revision sidebar #6763
Conversation
The routes section when the knative service is selected should just be the first box with the location the splits should not be shown. I believe this is what was agreed on in the serverless meeting, right @invincibleJai ? |
Yes @bgliwa01 , for KSVC side we'll show just route with location and revisions section will show split percentage with unique route if any |
@bgliwa01 @invincibleJai Updated the code and screenshots to remove the split details in the routes section of KSVC sidebar. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
4f70105
to
d54f1c3
Compare
@karthikjeeyar: This pull request references Bugzilla bug 1883737, which is invalid:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
const tData = trafficData.filter(filterTrafficBasedOnResource(resource)).reduce( | ||
(acc, traffic: Traffic) => { | ||
traffic.url && acc.urls.push(traffic.url); | ||
acc.percent += traffic.percent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit : good to have check here as well
d54f1c3
to
925fa82
Compare
/approve Verified the changes looks good!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
verified changes locally
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bgliwa01, invincibleJai, karthikjeeyar, sahil143 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/bugzilla refresh |
@karthikjeeyar: This pull request references Bugzilla bug 1883737, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@karthikjeeyar: All pull requests linked via external trackers have merged: Bugzilla bug 1883737 has been moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Fixes: https://issues.redhat.com/browse/ODC-4879
Problem:
Revision/Service Route section in sidebar shows incorrect traffic percent for unique url.
Solution:
Update Route section in the knative Revision sidebar and service sidebar to have unique and base url information.
Screenshots:
Revision with unique tag:
Knative Service sidebar:
Test cases:
RoutesOverviewList:
RoutesOverveiwListItem
RevisionsOverveiwListItem
resource-overview-utils:
/kind bug
cc: @bgliwa01 @invincibleJai @openshift/team-devconsole-ux