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

oc create -f test/testdata/app-scenarios fails to POST #10008

Closed
juanvallejo opened this issue Jul 25, 2016 · 13 comments
Closed

oc create -f test/testdata/app-scenarios fails to POST #10008

juanvallejo opened this issue Jul 25, 2016 · 13 comments
Assignees
Labels
kind/test-flake Categorizes issue or PR as related to test flakes. priority/P2

Comments

@juanvallejo
Copy link
Contributor

juanvallejo commented Jul 25, 2016

Link to last failure: https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_check/4612/consoleFull#-200384109356c25df4e4b05b79524e5d52

Version
oc v1.3.0-alpha.0+ea204df-dirty
kubernetes v1.3.0+57fb9ac
features: Basic-Auth
Steps To Reproduce
  1. [test]
Current Result
FAILURE after 2.917s: test/cmd/admin.sh:350: executing 'oc create -f test/testdata/app-scenarios' expecting success: the command returned the wrong error code

Standard error from the command:
The Service "frontend" is invalid.
spec.clusterIP: Invalid value: "172.30.17.154": provided IP is already allocated
Expected Result

Should succeed

EDIT: updated link to last failure

@juanvallejo juanvallejo added the kind/test-flake Categorizes issue or PR as related to test flakes. label Jul 25, 2016
@stevekuznetsov
Copy link
Contributor

@juanvallejo why the assign to me?

@stevekuznetsov stevekuznetsov removed their assignment Jul 25, 2016
@juanvallejo
Copy link
Contributor Author

@stevekuznetsov Ah was not sure who to assign to this

@stevekuznetsov
Copy link
Contributor

@juanvallejo if unsure, leave assignment and priority label unset, someone like @danmcp will come by to set them soon

@mfojtik
Copy link
Contributor

mfojtik commented Jul 26, 2016

@sgallagher can you please spend some time figuring out what is the issue here?

@sgallagher
Copy link
Contributor

@juanvallejo Can you point me at a recent example? The original link has expired.

@juanvallejo
Copy link
Contributor Author

@sgallagher Hm, the build seems to have been deleted, after checking on ci.openshift.redhat.com/jenkins/. Unfortunately this was the only build I can find this error happening

@juanvallejo
Copy link
Contributor Author

@stevekuznetsov
Copy link
Contributor

The failures seem to be this always:

Error from server: The POST operation against Pod could not be completed at this time, please try again.

@stevekuznetsov stevekuznetsov changed the title oc create -f test/testdata/app-scenarios test case keeps failing on jenkins oc create -f test/testdata/app-scenarios fails to POST Aug 5, 2016
@liggitt
Copy link
Contributor

liggitt commented Aug 5, 2016

Looks like that message is coming from https://github.com/openshift/origin/blob/master/vendor/k8s.io/kubernetes/pkg/api/rest/create.go#L106

// CheckGeneratedNameError checks whether an error that occurred creating a resource is due
// to generation being unable to pick a valid name.
func CheckGeneratedNameError(strategy RESTCreateStrategy, err error, obj runtime.Object) error {
    if !errors.IsAlreadyExists(err) {
        return err
    }

    objectMeta, kind, kerr := objectMetaAndKind(strategy, obj)
    if kerr != nil {
        return kerr
    }

    if len(objectMeta.GenerateName) == 0 {
        return err
    }

    return errors.NewServerTimeoutForKind(kind.GroupKind(), "POST", 0)
}

that's a terrible error... it looks like it's converting an AlreadyExists error into a server timeout error if it has a generate name

@liggitt
Copy link
Contributor

liggitt commented Aug 5, 2016

@smarterclayton looks like the sample data for petset is the culprit... it's creating a petset and all the pods that go with it, so the pod creates will race with the petset controller, and are likely to fail with an AlreadyExists error

@liggitt
Copy link
Contributor

liggitt commented Aug 5, 2016

bumping priority, I think we'll hit this flake a lot

@liggitt
Copy link
Contributor

liggitt commented Aug 5, 2016

reopening until #10240 goes in as well

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

No branches or pull requests

7 participants