Skip to content
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

Splitting traffic under routes is a bit difficult to understand #1070

Closed
benjaminapetersen opened this issue Dec 22, 2016 · 5 comments
Closed
Assignees
Labels
area/usability kind/bug Categorizes issue or PR as related to a bug. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. priority/P2

Comments

@benjaminapetersen
Copy link
Contributor

Pulled from original comment on this PR.
A few observations:

  • Checking "split traffic" is a bit jumpy. Couple things I struggled with here:
  • The slider works well, but once I click "edit weights as integers" I couldn't go back to the slider without refreshing the page.
  • When I switch to "edit weights as integers" it took me a while to figure out why I had a dropdown with only one service, even though the slider shows 2 services:
    screen shot 2016-12-22 at 2 47 03 pm
    I think its because the first service shows up above and the "weight" input now appears next to it. I had to think about that a bit as I expected everything I was interacting with to appear below the heading. I wonder if we could leave the above selector to be exclusive to "service to route to" and just list them all again under "split traffic" with the weight inputs (somehow show the primary service linked to the above "service to route to").
  • Once I added additional services beyond 2, it was a little easier to understand.

@spadgett @jwforres

@spadgett spadgett self-assigned this Dec 22, 2016
@spadgett
Copy link
Member

The interaction is probably less obvious now that we added the slider. Before, you picked a service and entered a weight in the input beside it as one step. Now if the page switches from the slider to integer weights it's not obvious what changed since the inputs are above where you clicked and might even be off screen.

@benjaminapetersen
Copy link
Contributor Author

Yup, I think thats what confused me, now that it jumps a bit its hard to follow which inputs are actually new.

@spadgett spadgett added area/usability kind/bug Categorizes issue or PR as related to a bug. priority/P2 labels Dec 22, 2016
@openshift-bot
Copy link

Issues go stale after 90d of inactivity.

Mark the issue as fresh by commenting /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
Exclude this issue from closing by commenting /lifecycle frozen.

If this issue is safe to close now please do so with /close.

/lifecycle stale

@openshift-ci-robot openshift-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 8, 2020
@jwforres
Copy link
Member

/close

@openshift-ci-robot
Copy link

@jwforres: Closing this issue.

In response to this:

/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/usability kind/bug Categorizes issue or PR as related to a bug. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. priority/P2
Projects
None yet
Development

No branches or pull requests

5 participants