[CASSANDRA-16781] Fix upgrade dtests (after cassandra-4.0 branch bumped to 4.0.1)

• Type: Bug
• Status: Resolved
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 4.0.1, 4.1-alpha1, 4.1
• Component/s: Test/dtest/python
• Bug Category: Correctness - Test Failure
• Severity: Critical
• Complexity: Normal
• Discovered By: DTest
• Platform: All
• Since Version: 4.0-rc2

Source Control Link

https://github.com/apache/cassandra-dtest/commit/a1080b1ef3c77bc4c5e6a4fff154d1c17f115e22

Test and Documentation Plan

CI

• Assignee: Michael Semb Wever
• Reporter: Michael Semb Wever

Authors

Michael Semb Wever

Reviewers

Berenguer Blasi, Brandon Williams

• Created: 01/Jul/21 11:57
• Updated: 27/May/22 19:25
• Resolved: 04/Jul/21 10:55

Description

Regression
dtest-upgrade.upgrade_tests.paging_test.TestPagingDataNodes2RF1_Upgrade_current_4_0_x_To_indev_4_0_x.test_static_columns_paging (from Cassandra dtests)

Failing for the past 1 build (Since Unstable#116 )
Took 46 sec.
 Failed 1 times in the last 29 runs. Flakiness: 3%, Stability: 96%
Error Message
TypeError: '<' not supported between instances of 'str' and 'int'
Stacktrace
self = <abc.TestPagingDataNodes2RF1_Upgrade_current_4_0_x_To_indev_4_0_x object at 0x7f5ff1817c10>

    @since('2.0.6')
    def test_static_columns_paging(self):
        """
            Exercises paging with static columns to detect bugs
            @jira_ticket CASSANDRA-8502.
            """
        cursor = self.prepare(row_factory=named_tuple_factory)
        cursor.execute("CREATE TABLE test (a int, b int, c int, s1 int static, s2 int static, PRIMARY KEY (a, b))")
    
        for is_upgraded, cursor in self.do_upgrade(cursor, row_factory=named_tuple_factory):
>           min_version = min(self.get_node_versions())

upgrade_tests/paging_test.py:661: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
/usr/lib/python3.8/distutils/version.py:52: in __lt__
    c = self._cmp(other)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = LooseVersion ('4.0-rc1'), other = LooseVersion ('4.0.1')

    def _cmp (self, other):
        if isinstance(other, str):
            other = LooseVersion(other)
    
        if self.version == other.version:
            return 0
>       if self.version < other.version:
E       TypeError: '<' not supported between instances of 'str' and 'int'

/usr/lib/python3.8/distutils/version.py:337: TypeError

from https://ci-cassandra.apache.org/job/Cassandra-4.0/116/testReport/junit/dtest-upgrade.upgrade_tests.paging_test/TestPagingDataNodes2RF1_Upgrade_current_4_0_x_To_indev_4_0_x/test_static_columns_paging/

ref: https://the-asf.slack.com/archives/CK23JSY2K/p1625125464283300

Issue Links

• relates to: CASSANDRA-16648 Add 4_0_x to Python DTest's upgrade_manifest (after cassandra-4.0 branch was created)