Github user jrwest commented on a diff in the pull request:

    https://github.com/apache/cassandra-dtest/pull/37#discussion_r218529082
  
    --- Diff: repair_tests/incremental_repair_test.py ---
    @@ -207,6 +208,7 @@ def test_manual_session_fail(self):
             self.fixture_dtest_setup.setup_overrides.cluster_options = ImmutableMapping({'hinted_handoff_enabled': 'false',
                                                                                          'num_tokens': 1,
                                                                                          'commitlog_sync_period_in_ms': 500})
    +        self.fixture_dtest_setup.init_default_config()
    --- End diff --
    
    Yea the proxying to the fixture is a bit non-intuitive. iirc it was added to make the transition from nosetest -> pytest easier. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org