I recently had to fix at least two problems that could've been prevented by running tests and checks that are about to be turned off by default ([1] and [2]).
I'd like to point out that this has happened while checks were theoretically enabled, and both problems could've been prevented. This doesn't seem to be a merge problem, or something that showed up only after the merge. I might be misunderstanding motivation for this, but my impression was that we, as a community, are striving to be able to have working version on every commit merged to master, and possibly even block merging in case tests don't pass. It'd be great to hear more about why this could be helpful. [1] Ninja fix: fix eclipse warnings that were broken during CASSANDRA-15528 <https://github.com/apache/cassandra/commit/a01d05d9a73211fb91c068e133d78ef8ccf34b4e> [2] Ninja fix: Fix unit tests that were broken during CASSANDRA-15303. <https://github.com/apache/cassandra/commit/b29af2925cddacb4ab8b429b31917748781fbe5d> On Tue, Mar 24, 2020 at 9:01 PM Joshua McKenzie <jmcken...@apache.org> wrote: > Am I understanding correctly - this isn't disabling tests, just changing > when they're triggered (i.e. automatic to manual)? > > So, for instance, smaller interim commits don't trigger a CI run and thus > costs? > > On Tue, Mar 24, 2020 at 3:34 PM David Capwell <dcapw...@gmail.com> wrote: > > > > > > > I want to change it so it could be a manual choice whether to do it or > > not. > > > > > > Could you explain the motivations for disabling the tests by default? My > > personal stance is all tests should run (we disable a lot, at least > HIGHER > > should enable all...), not a fan of disabling tests. > > > > On Tue, Mar 24, 2020 at 12:14 PM Ekaterina Dimitrova < > > ekaterina.dimitr...@datastax.com> wrote: > > > > > Hello everyone, > > > Hope this email finds you well! > > > > > > Just a heads up that I plan to open a Jira and change the CircleCI > > config. > > > Currently unit tests and in-jvm tests are triggered automatically on > > every > > > commit. > > > I want to change it so it could be a manual choice whether to do it or > > not. > > > > > > Anyone against that who really needs the current setup? Is there any > > > background information I miss? > > > > > > Ekaterina Dimitrova | Software Engineer > > > ekaterina.dimitr...@datastax.com | datastax.com > > > < > > > > > > http://datastax.com/?utm_campaign=FY20Q2_CONSTELLATION&utm_+medium=email&utm_source=signature > > > > > > > > > > -- alex p
