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

e2e test flakes: traffic split tests #694

Closed
navidshaikh opened this issue Feb 29, 2020 · 3 comments
Closed

e2e test flakes: traffic split tests #694

navidshaikh opened this issue Feb 29, 2020 · 3 comments
Assignees
Labels
area/e2e kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@navidshaikh
Copy link
Collaborator

What version of Knative Client?

HEAD

What version of Knative Serving running on your cluster?

seen against nightly build of serving

Expected Behavior

Traffic split tests pass.

Actual Behavior

Flakes in traffic split e2e tests.

        traffic_split_test.go:417: assertion failed: error is not nil: expecting 3 targets, got 2 targets targets: ,echo12-rev-2,100,true|latest,echo12-rev-1,0,false separator: |

ref: logs

Steps to Reproduce the Problem

@navidshaikh navidshaikh added kind/bug Categorizes issue or PR as related to a bug. area/e2e labels Feb 29, 2020
@MIBc
Copy link
Contributor

MIBc commented Feb 29, 2020

The test log showed the result: ,echo12-rev-2,100,true|latest,echo12-rev-1,0,false.

It seems that the cmd kn service update echo12 --untag latest --tag echo12-rev-1=old,echo12-rev-2=latest --namespace kne2etests2 is useless.

I alse have ran the test in knative-serving 0.11.0. It is OK.
Maybe you can retest it.

@dsimansk
Copy link
Contributor

dsimansk commented Mar 5, 2020

Fixed by #701.

/close

@knative-prow-robot
Copy link
Contributor

@dsimansk: Closing this issue.

In response to this:

Fixed by #701.

/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.

@navidshaikh navidshaikh added this to the v0.13.0 milestone Mar 9, 2020
dsimansk added a commit to dsimansk/client that referenced this issue May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/e2e kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

5 participants