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

[Feature:Prometheus][Conformance] Prometheus when installed to the cluster should start and expose a secured proxy and unsecured metrics [Suite:openshift/conformance/parallel] #17693

Closed
enj opened this issue Dec 8, 2017 · 3 comments
Assignees
Labels
component/metrics kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1

Comments

@enj
Copy link
Contributor

enj commented Dec 8, 2017

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/17617/test_pull_request_origin_extended_conformance_gce/12593/#featureprometheusconformance-prometheus-when-installed-to-the-cluster-should-start-and-expose-a-secured-proxy-and-unsecured-metrics-suiteopenshiftconformanceparallel

/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/test/extended/prometheus/prometheus.go:40
Expected error:
    <*util.ExitError | 0xc42171b4a0>: {
        Cmd: "oc create --config=/tmp/cluster-admin.kubeconfig -f -",
        StdErr: "Error from server (AlreadyExists): serviceaccounts \"prometheus\" already exists\nError from server (AlreadyExists): clusterrolebindings.authorization.openshift.io \"prometheus-cluster-reader\" already exists\nError from server (AlreadyExists): routes.route.openshift.io \"prometheus\" already exists\nError from server (AlreadyExists): services \"prometheus\" already exists\nError from server (AlreadyExists): secrets \"prometheus-proxy\" already exists\nError from server (AlreadyExists): statefulsets.apps \"prometheus\" already exists\nError from server (AlreadyExists): configmaps \"prometheus\" already exists\nError from server (AlreadyExists): routes.route.openshift.io \"alerts\" already exists\nError from server (AlreadyExists): services \"alerts\" already exists\nError from server (AlreadyExists): secrets \"alerts-proxy\" already exists\nError from server (AlreadyExists): configmaps \"prometheus-alerts\" already exists",
        ExitError: {
            ProcessState: {
                pid: 18223,
                status: 256,
                rusage: {
                    Utime: {Sec: 0, Usec: 289106},
                    Stime: {Sec: 0, Usec: 44477},
                    Maxrss: 53140,
                    Ixrss: 0,
                    Idrss: 0,
                    Isrss: 0,
                    Minflt: 14765,
                    Majflt: 0,
                    Nswap: 0,
                    Inblock: 0,
                    Oublock: 0,
                    Msgsnd: 0,
                    Msgrcv: 0,
                    Nsignals: 0,
                    Nvcsw: 1530,
                    Nivcsw: 30,
                },
            },
            Stderr: nil,
        },
    }
    exit status 1
not to have occurred
/go/src/github.com/openshift/origin/_output/local/go/src/github.com/openshift/origin/test/extended/prometheus/prometheus.go:313
@enj enj added the kind/test-flake Categorizes issue or PR as related to test flakes. label Dec 8, 2017
@bparees bparees assigned mfojtik and smarterclayton and unassigned bparees Dec 8, 2017
@bparees
Copy link
Contributor

bparees commented Dec 8, 2017

@enj not sure who owns prometheus, but it's not devex (we do have build metrics in prometheus but that's not what this test is).

@smarterclayton and @mfojtik show up in the git blame on the test file.

@smarterclayton
Copy link
Contributor

This means that the build and the prometheus tests ran in parallel at the same time.

@bparees
Copy link
Contributor

bparees commented Dec 11, 2017

@gabemontero @smarterclayton so it sounds like both tests need to be updated to tolerate the existence of the service account/prometheus installation. (Unless the build test already does).

openshift-merge-robot added a commit that referenced this issue Dec 16, 2017
Automatic merge from submit-queue (batch tested with PRs 16281, 17293, 17717, 17753, 17830).

sync prometheus ext tests running in parallel

Fixes #17693

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

No branches or pull requests

6 participants