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

e2e test TestGithubSecondPullRequestConcurrencyMultiplePR is failing sometime #1691

Open
chmouel opened this issue May 27, 2024 · 1 comment
Assignees

Comments

@chmouel
Copy link
Member

chmouel commented May 27, 2024

the artifacts are not available anymore but when it shows we need to dig into the logs of e2e to see if there is any clues why cleanup didn't work properly

out of 18, waiting 10s more, 15/35"}
{"level":"info","ts":1716791990.2163646,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 8 out of 18, waiting 10s more, 16/35"}
{"level":"info","ts":1716792000.256401,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 7 out of 18, waiting 10s more, 17/35"}
{"level":"info","ts":1716792010.2988453,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 6 out of 18, waiting 10s more, 18/35"}
{"level":"info","ts":1716792020.3391192,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 6 out of 18, waiting 10s more, 19/35"}
{"level":"info","ts":1716792030.374956,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 5 out of 18, waiting 10s more, 20/35"}
{"level":"info","ts":1716792040.4162462,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 4 out of 18, waiting 10s more, 21/35"}
{"level":"info","ts":1716792050.4529085,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 4 out of 18, waiting 10s more, 22/35"}
{"level":"info","ts":1716792060.4852042,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 3 out of 18, waiting 10s more, 23/35"}
{"level":"info","ts":1716792070.5413985,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 2 out of 18, waiting 10s more, 24/35"}
{"level":"info","ts":1716792080.5747092,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 2 out of 18, waiting 10s more, 25/35"}
{"level":"info","ts":1716792090.6038084,"caller":"test/github_pullrequest_concurrency_multiplepr_test.go:130","msg":"number of unsuccessful PR 1 out of 18, waiting 10s more, 26/35"}
    github_pullrequest_concurrency_multiplepr_test.go:140: assertion failed: 10 (int) != 9 (allPipelinesRunAfterCleanUp int): we should have had 9 kept after cleanup, we got 10
@savitaashture
Copy link
Member

{"level":"info","ts":1716789214.0118024,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 4 out of 5, waiting 10s more in the waiting loop: 2/30"}
{"level":"info","ts":1716789224.0287883,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 4 out of 5, waiting 10s more in the waiting loop: 3/30"}
{"level":"info","ts":1716789234.0453036,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 4 out of 5, waiting 10s more in the waiting loop: 4/30"}
{"level":"info","ts":1716789244.0635755,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 3 out of 5, waiting 10s more in the waiting loop: 5/30"}
{"level":"info","ts":1716789254.0832686,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 2 out of 5, waiting 10s more in the waiting loop: 6/30"}
{"level":"info","ts":1716789264.1016324,"caller":"test/github_pullrequest_concurrency_test.go:129","msg":"number of unsuccessful PR 2 out of 5, waiting 10s more in the waiting loop: 7/30"}
{"level":"info","ts":1716789274.1189997,"caller":"test/github_pullrequest_concurrency_test.go:125","msg":"the 5 pipelineruns has successfully finished"}
    github_pullrequest_concurrency_test.go:145: assertion failed: expression is false: strings.HasPrefix(prActualName, prExpectedName): prActualName: no-match-f7jv4 does not start with expected prefix prlongrunnning-6, was is ordered properly at start time
{"level":"info","ts":1716789274.1404073,"caller":"github/pr.go:186","msg":"Closing PR 2228"}
{"level":"info","ts":1716789274.6032455,"caller":"repository/teardown.go:13","msg":"Deleting Repository in pac-e2e-ns-xmjxq"}
{"level":"info","ts":1716789274.6077428,"caller":"repository/teardown.go:16","msg":"Deleting NS pac-e2e-ns-xmjxq"}
{"level":"info","ts":1716789274.6104174,"caller":"github/pr.go:199","msg":"Deleting Ref refs/heads/pac-e2e-test-z92q4"}

@chmouel chmouel self-assigned this Jun 12, 2024
chmouel added a commit to chmouel/pipelines-as-code that referenced this issue Jun 12, 2024
Try to add some debugs for the tests and more logs collected by
namespaces.

Related to bug openshift-pipelines#1691
chmouel added a commit to chmouel/pipelines-as-code that referenced this issue Jun 12, 2024
Try to add some debugs for the tests and more logs collected by
namespaces.

Related to bug openshift-pipelines#1691

Signed-off-by: Chmouel Boudjnah <[email protected]>
chmouel added a commit to chmouel/pipelines-as-code that referenced this issue Jun 12, 2024
Try to add some debugs for the tests and more logs collected by
namespaces.

Related to bug openshift-pipelines#1691

Signed-off-by: Chmouel Boudjnah <[email protected]>
chmouel added a commit that referenced this issue Jun 12, 2024
Try to add some debugs for the tests and more logs collected by
namespaces.

Related to bug #1691

Signed-off-by: Chmouel Boudjnah <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants