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

flake: deploymentconfigs with minimum ready seconds set [Conformance] should not transition the deployment to Complete before satisfied #16235

Closed
jsafrane opened this issue Sep 8, 2017 · 3 comments
Closed

flake: deploymentconfigs with minimum ready seconds set [Conformance] should not transition the deployment to Complete before satisfied #16235

jsafrane opened this issue Sep 8, 2017 · 3 comments
Assignees
@tnozicka
Labels
component/apps kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1

Comments

@jsafrane
Copy link
Contributor

jsafrane commented Sep 8, 2017

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/16232/test_pull_request_origin_extended_conformance_gce/7490/

: deploymentconfigs with minimum ready seconds set [Conformance] should not transition the deployment to Complete before satisfied 2m41s

/tmp/openshift/build-rpm-release/tito/rpmbuild-origin0il1_K/BUILD/origin-3.7.0/_output/local/go/src/github.com/openshift/origin/test/extended/deployments/deployments.go:972
Expected
    <int32>: 1
to be zero-valued
/tmp/openshift/build-rpm-release/tito/rpmbuild-origin0il1_K/BUILD/origin-3.7.0/_output/local/go/src/github.com/openshift/origin/test/extended/deployments/deployments.go:930
@jsafrane jsafrane mentioned this issue Sep 8, 2017
Merged
@mfojtik
Copy link
Contributor

mfojtik commented Sep 8, 2017

/assign tnozicka

@mfojtik mfojtik added component/apps kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1 labels Sep 8, 2017
@tnozicka tnozicka mentioned this issue Sep 8, 2017
Merged
openshift-merge-robot added a commit that referenced this issue Sep 9, 2017
@openshift-merge-robot
Merge pull request #16258 from tnozicka/fix-flake-16235
9bbec3f 
Automatic merge from submit-queue (batch tested with PRs 16258, 16214)

Fix deployment minReadySecond check for availableReplicas after all pods are ready

Fixes #16235
@bparees
Copy link
Contributor

bparees commented Oct 10, 2017

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/16737/test_pull_request_origin_extended_conformance_install_update/7620/

@bparees bparees reopened this Oct 10, 2017
@bparees
Copy link
Contributor

bparees commented Oct 10, 2017

sorry, looks like my failure is a new reason, will open a new issue.

@bparees bparees closed this as completed Oct 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tnozicka tnozicka

Labels
component/apps kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mfojtik @jsafrane @bparees @tnozicka