Re: [DISCUSS] Remove stress-new-test-openjdk11 requirement from PRs

2021-06-10 Thread Kirk Lund
Ok, I wanted to give this discussion another night and we still have consensus for making both stress-new-test non-required. I just filed PR #6602 to change stress-new-test from required to non-required. Please review! On Wed, Jun 9, 2021 at 2:11 PM Ant

Re: [DISCUSS] Remove stress-new-test-openjdk11 requirement from PRs

2021-06-10 Thread Nabarun Nag
Hi all, * We need to discuss how to prevent more flaky tests to be introduced now that stress-test is not mandatory for PRs to be merged? Reviewers checking the PR must check the tests failing in stress test and if it is a test that has been introduced or changed in the PR, the PR must be

Re: [DISCUSS] Remove stress-new-test-openjdk11 requirement from PRs

2021-06-10 Thread Owen Nichols
@Naba, two places to find recent flaky failures are: * Latest mass-test report[1] * Recently-sighted "CI Failure" Jira tickets[2] [1] https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-mass-test-run/jobs/create-mass-test-run-report/builds/19 [2] https://issues.apache.org/j

Re: [DISCUSS] Remove stress-new-test-openjdk11 requirement from PRs

2021-06-10 Thread Mark Hanson
+1 to new report from Owen +1 to re-introduce the stress-test Great ideas Naba! On 6/10/21, 9:50 AM, "Nabarun Nag" wrote: Hi all, * We need to discuss how to prevent more flaky tests to be introduced now that stress-test is not mandatory for PRs to be merged? Reviewers checking

Re: [DISCUSS] Remove stress-new-test-openjdk11 requirement from PRs

2021-06-10 Thread Dan Smith
I'm also -0 on this one. I personally kinda like having the merge button disabled on my PRs until all the checks are passing so I don't have to think as much, and I'm not sure this problem is going to come up again soon. But I'm willing go with disabling it. -Dan ___

Canceled: apache/geode-examples#543 (support/1.12 - a18cf3f)

2021-06-10 Thread Travis CI
Build Update for apache/geode-examples - Build: #543 Status: Canceled Duration: ? Commit: a18cf3f (support/1.12) Author: Owen Nichols Message: temporarily point to staging repo for CI purposes View the changeset: https://github.com/apache/geode-examples/compa

Errored: apache/geode-examples#545 (rel/v1.12.3.RC1 - 85ffd22)

2021-06-10 Thread Travis CI
Build Update for apache/geode-examples - Build: #545 Status: Errored Duration: 1 min and 3 secs Commit: 85ffd22 (rel/v1.12.3.RC1) Author: Owen Nichols Message: Bumping version to 1.12.3 View the changeset: https://github.com/apache/geode-examples/compare/rel/

Failed: apache/geode-native#3079 (rel/v1.12.3.RC1 - 717bbf7)

2021-06-10 Thread Travis CI
Build Update for apache/geode-native - Build: #3079 Status: Failed Duration: 2 mins and 30 secs Commit: 717bbf7 (rel/v1.12.3.RC1) Author: Owen Nichols Message: update Dockerfile and other variables to apache-geode 1.12.2 View the changeset: https://github.com

Canceled: apache/geode-examples#546 (support/1.12 - 3176db2)

2021-06-10 Thread Travis CI
Build Update for apache/geode-examples - Build: #546 Status: Canceled Duration: ? Commit: 3176db2 (support/1.12) Author: Owen Nichols Message: temporarily point to staging repo for CI purposes View the changeset: https://github.com/apache/geode-examples/compa

Errored: apache/geode-examples#548 (rel/v1.12.3.RC2 - 078f3b8)

2021-06-10 Thread Travis CI
Build Update for apache/geode-examples - Build: #548 Status: Errored Duration: 1 min and 0 secs Commit: 078f3b8 (rel/v1.12.3.RC2) Author: Owen Nichols Message: Revert "temporarily point to staging repo for CI purposes" View the changeset: https://github.com/a

Failed: apache/geode-native#3080 (rel/v1.12.3.RC2 - 717bbf7)

2021-06-10 Thread Travis CI
Build Update for apache/geode-native - Build: #3080 Status: Failed Duration: 1 min and 44 secs Commit: 717bbf7 (rel/v1.12.3.RC2) Author: Owen Nichols Message: update Dockerfile and other variables to apache-geode 1.12.2 View the changeset: https://github.com/