Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-16323

Circle CI python upgrade test build includes non-upgrade tests

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Normal
    • Resolution: Fixed
    • Fix Version/s: 4.0, 4.0-beta4
    • Component/s: CI
    • Labels:
      None

      Description

      https://app.circleci.com/pipelines/github/dcapwell/cassandra/841/workflows/bcadf6e6-8d04-4010-8a47-99f7f9b5ac1d/jobs/4949/parallel-runs/23?filterBy=ALL

      Jenkins uses the following for finding tests

      export RUN_STATIC_UPGRADE_MATRIX=true
      ./run_dtests.py \
          --execute-upgrade-tests-only \
          --dtest-print-tests-only \
          --dtest-print-tests-output=${WORKSPACE}/test_list.txt \
          --cassandra-dir=$CASSANDRA_DIR
      

      Where as circle ci uses

      export RUN_STATIC_UPGRADE_MATRIX=true
      ./run_dtests.py \
          --execute-upgrade-tests \
          --dtest-print-tests-only \
          --dtest-print-tests-output=/tmp/all_dtest_tests_j8_upgradetests_without_vnodes_raw \
          --cassandra-dir=../cassandra
      

      The difference here is that —execute-upgrade-tests-only will block non-upgrade tests, where —execute-upgrade-tests will include upgrade tests; for this reason non-upgrade tests are scheduled as part of the build.

      Tests such as the following were found in the allow list:
      transient_replication_test.py::TestTransientReplicationSpeculativeQueries::test_custom_speculate
      ttl_test.py::TestTTL::test_collection_set_ttl
      auth_test.py::TestAuth::test_alter_ks_auth

      many more were found

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                dcapwell David Capwell
                Reporter:
                dcapwell David Capwell
                Authors:
                David Capwell
                Reviewers:
                Brandon Williams
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: