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

Concurrency issue in the end to end tests #830

Closed
zregvart opened this issue Jul 17, 2019 · 2 comments · Fixed by #834
Closed

Concurrency issue in the end to end tests #830

zregvart opened this issue Jul 17, 2019 · 2 comments · Fixed by #834
Assignees

Comments

@zregvart
Copy link
Member

The build #1150 failed with:

=== RUN   TestPlatformUpgrade
Camel K installed in namespace test-7ae995dd-f1fe-4a9f-b243-99b98ffaf955
--- FAIL: TestPlatformUpgrade (7.94s)
    upgrade_test.go:40: 
        Expected
            <*errors.StatusError | 0xc000434240>: {
                ErrStatus: {
                    TypeMeta: {Kind: "", APIVersion: ""},
                    ListMeta: {SelfLink: "", ResourceVersion: "", Continue: ""},
                    Status: "Failure",
                    Message: "Operation cannot be fulfilled on deployments.apps \"camel-k-operator\": the object has been modified; please apply your changes to the latest version and try again",
                    Reason: "Conflict",
                    Details: {
                        Name: "camel-k-operator",
                        Group: "apps",
                        Kind: "deployments",
                        UID: "",
                        Causes: nil,
                        RetryAfterSeconds: 0,
                    },
                    Code: 409,
                },
            }
        to be nil

Looks like a concurrency issue in the end to end test suite or in the operator.

@lburgazzoli
Copy link
Contributor

lburgazzoli commented Jul 17, 2019

@nicolaferraro I think Expect(scaleOperator) like here should be replaced by Eventually(scaleOperator) as the update operation may fail for conflict

@nicolaferraro
Copy link
Member

Yeah, right. Together with some currying on the scale function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants