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

deployment config controller doesn't sync, conformance flakes #10229

Closed
stevekuznetsov opened this issue Aug 4, 2016 · 4 comments
Closed

deployment config controller doesn't sync, conformance flakes #10229

stevekuznetsov opened this issue Aug 4, 2016 · 4 comments
Assignees
Labels
component/apps kind/bug Categorizes issue or PR as related to a bug. kind/test-flake Categorizes issue or PR as related to test flakes. priority/P2

Comments

@stevekuznetsov
Copy link
Contributor

seen here

/data/src/github.com/openshift/origin/test/extended/deployments/deployments.go:577
the current deployment needs to have finished before attempting to trigger a new deployment through configuration change
Expected error:
    <*errors.errorString | 0xc8200ce130>: {
        s: "timed out waiting for the condition",
    }
    timed out waiting for the condition
not to have occurred
@stevekuznetsov stevekuznetsov added kind/bug Categorizes issue or PR as related to a bug. component/apps kind/test-flake Categorizes issue or PR as related to test flakes. labels Aug 4, 2016
@mfojtik
Copy link
Contributor

mfojtik commented Aug 9, 2016

interesting, the seventh DC we triggered (and the one that failed) has this error:

I0804 16:09:00.952711   27136 controller.go:407] Error syncing deployment config extended-test-cli-deployment-qnffq-gqpmq/history-limit: Operation cannot be fulfilled on replicationcontrollers "history-limit-7": the object has been modified; please apply your changes to the latest version and try again

@0xmichalis
Copy link
Contributor

0xmichalis commented Aug 18, 2016

@mfojtik do we need the retries on conflict you have in #10341?

@mfojtik
Copy link
Contributor

mfojtik commented Aug 18, 2016

@Kargakis will check.

@0xmichalis
Copy link
Contributor

#10507 added retries on update conflicts in the deploymentconfig controller. Closing until we see it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/apps kind/bug Categorizes issue or PR as related to a bug. kind/test-flake Categorizes issue or PR as related to test flakes. priority/P2
Projects
None yet
Development

No branches or pull requests

4 participants