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

Extended.[Feature:DeploymentConfig] deploymentconfigs won't deploy RC with unresolved images #19270

Closed
mfojtik opened this issue Apr 9, 2018 · 3 comments
Assignees
Labels
component/apps kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1 sig/master

Comments

@mfojtik
Copy link
Contributor

mfojtik commented Apr 9, 2018

Happens after 1.10 rebase landed, seems like the scaler need PATCH or retry mechanism:

Apr  9 10:10:12.291: INFO: Running 'oc logs --config=/tmp/extended-test-cli-deployment-l6kht-zktkc-user.kubeconfig --namespace=extended-test-cli-deployment-l6kht-zktkc pod/example-1-deploy --timestamps=true'
Apr  9 10:10:12.610: INFO: --- pod example-1-deploy logs
2018-04-09T10:10:10.487919892Z --> Scaling example-1 to 1
2018-04-09T10:10:10.535412137Z error: couldn't scale example-1 to 1: Scaling the resource failed with: could not update the scale for replicationcontrollers example-1: Operation cannot be fulfilled on replicationcontrollers "example-1": the object has been modified; please apply your changes to the latest version and try again; Current resource version 20855---

Logs: https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/19246/test_pull_request_origin_extended_conformance_gce/18766/

@mfojtik mfojtik added priority/P1 component/apps kind/test-flake Categorizes issue or PR as related to test flakes. sig/master labels Apr 9, 2018
@mfojtik mfojtik self-assigned this Apr 9, 2018
@mfojtik
Copy link
Contributor Author

mfojtik commented Apr 9, 2018

@bparees all the build flakes I saw today seems to be related to this (unable to push to registry, I guess because registry failed to scale up, just FYI)

@bparees
Copy link
Contributor

bparees commented Apr 9, 2018

@wozniakjan fyi in case this is showing up in our extended builds tests too.

@mfojtik
Copy link
Contributor Author

mfojtik commented Apr 11, 2018

This should be now fixes as @deads2k #19275 merged.

@mfojtik mfojtik closed this as completed Apr 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/apps kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1 sig/master
Projects
None yet
Development

No branches or pull requests

2 participants