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.

Sign up for GitHub

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

Jump to bottom

[k8s.io] Deployment deployment should support rollover {Kubernetes e2e suite} #39879

Closed
k8s-github-robot opened this issue Jan 13, 2017 · 10 comments
Closed

[k8s.io] Deployment deployment should support rollover {Kubernetes e2e suite} #39879

k8s-github-robot opened this issue Jan 13, 2017 · 10 comments
Assignees
@janetkuo
Labels
kind/flake Categorizes issue or PR as related to a flaky test.

Comments

@k8s-github-robot
Copy link

https://k8s-gubernator.appspot.com/build/kubernetes-jenkins/logs/ci-kubernetes-e2e-gke-release-1.5/3341/
Failed: [k8s.io] Deployment deployment should support rollover {Kubernetes e2e suite}

/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/deployment.go:77
Expected error:
    <*errors.errorString | 0xc420d3d760>: {
        s: "error waiting for deployment \"test-rollover-deployment\" status to match expectation: timed out waiting for the condition",
    }
    error waiting for deployment "test-rollover-deployment" status to match expectation: timed out waiting for the condition
not to have occurred
/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/deployment.go:598

Previous issues for this test: #26509 #26834 #29780 #35355
@k8s-github-robot k8s-github-robot added kind/flake Categorizes issue or PR as related to a flaky test. priority/P2 labels Jan 13, 2017
@pwittrock pwittrock assigned janetkuo and unassigned pwittrock Jan 13, 2017
@pwittrock
Copy link
Member

@kubernetes/sig-apps-misc

This was referenced Jan 14, 2017
Closed
Closed
Closed
Closed
Closed
@0xmichalis
Copy link
Contributor

Seems like AvailableReplicas in one of the underlying replica sets is not updated correctly but it's quite rare to hit. Either a clock-skew issue or the cache is dropping a watch event or ...

This was referenced Jan 18, 2017
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
This was referenced Jan 25, 2017
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@yujuhong
Copy link
Contributor

yujuhong commented Feb 7, 2017

Can we replace watch.Until with something else (polling?) in the tests until we fix it properly?

@yujuhong yujuhong mentioned this issue Feb 7, 2017
Closed
k8s-github-robot pushed a commit that referenced this issue Feb 7, 2017
Merge pull request #40900 from janetkuo/deployment-e2e-pod-running
d82e51e 
Automatic merge from submit-queue (batch tested with PRs 40345, 38183, 40236, 40861, 40900)

Remove checks for pods responding in deployment e2e tests

Fixes #39879

Remove it because it caused deployment e2e tests sometimes timed out waiting for pods responding, and pods responding isn't related to deployment controller and is not a prerequisite of deployment e2e tests.

@Kargakis
@janetkuo janetkuo mentioned this issue Feb 8, 2017
Merged
k8s-github-robot pushed a commit that referenced this issue Feb 10, 2017
Merge pull request #41112 from janetkuo/no-watch-until
558c37a 
Automatic merge from submit-queue (batch tested with PRs 41112, 41201, 41058, 40650, 40926)

e2e test flakes: remove some uses of watch.Until in e2e tests

`watch.Until` is somewhat broken and is causing quite a lot of test flakes. See #39879 (comment) and #31345 for more context.

@wojtek-t @yujuhong @Kargakis
This was referenced Feb 26, 2017
Closed
Closed
This was referenced Mar 5, 2017
Closed
Closed
Closed
Closed
Closed
This was referenced Mar 14, 2017
Closed
Closed
Closed
Closed
Closed
Closed
Closed
This was referenced Mar 23, 2017
Closed
Closed
Closed
Closed
This was referenced Mar 31, 2017
Closed
Closed
This was referenced Apr 26, 2017
Closed
Closed
@tnozicka tnozicka mentioned this issue Aug 3, 2017
Closed
5 tasks
akhilerm pushed a commit to akhilerm/apimachinery that referenced this issue Sep 20, 2022
@k8s-publishing-bot
Merge pull request #41112 from janetkuo/no-watch-until
f758f8b 
Automatic merge from submit-queue (batch tested with PRs 41112, 41201, 41058, 40650, 40926)

e2e test flakes: remove some uses of watch.Until in e2e tests

`watch.Until` is somewhat broken and is causing quite a lot of test flakes. See kubernetes/kubernetes#39879 (comment) and kubernetes/kubernetes#31345 for more context.

@wojtek-t @yujuhong @Kargakis

Kubernetes-commit: 558c37aee3ae62356bb16068af9973e5489aa86a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janetkuo janetkuo

Labels
kind/flake Categorizes issue or PR as related to a flaky test.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@mml @0xmichalis @janetkuo @pwittrock @yujuhong @wojtek-t @k8s-github-robot